From f1ab1e05fe5cd2f71a220b0cfe0dae4a128e2bf1 Mon Sep 17 00:00:00 2001
From: Kermit <kermitlx@outlook.com>
Date: Thu, 13 May 2021 17:21:07 +0800
Subject: [PATCH] chore: add role to clear button (#250)

* chore: add role to clear button

* chore: update snapshot
---
 src/Picker.tsx                           | 15 +++++++++------
 tests/__snapshots__/picker.spec.tsx.snap |  1 +
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/Picker.tsx b/src/Picker.tsx
index 2463473ca..11b717bdc 100644
--- a/src/Picker.tsx
+++ b/src/Picker.tsx
@@ -98,9 +98,11 @@ type OmitPanelProps<Props> = Omit<
   'onChange' | 'hideHeader' | 'pickerValue' | 'onPickerValueChange'
 >;
 
-export type PickerBaseProps<DateType> = {} & PickerSharedProps<DateType> & OmitPanelProps<PickerPanelBaseProps<DateType>>;
+export type PickerBaseProps<DateType> = {} & PickerSharedProps<DateType> &
+  OmitPanelProps<PickerPanelBaseProps<DateType>>;
 
-export type PickerDateProps<DateType> = {} & PickerSharedProps<DateType> & OmitPanelProps<PickerPanelDateProps<DateType>>;
+export type PickerDateProps<DateType> = {} & PickerSharedProps<DateType> &
+  OmitPanelProps<PickerPanelDateProps<DateType>>;
 
 export type PickerTimeProps<DateType> = {
   picker: 'time';
@@ -109,7 +111,8 @@ export type PickerTimeProps<DateType> = {
    * since `defaultOpenValue` will confuse user of current value status
    */
   defaultOpenValue?: DateType;
-} & PickerSharedProps<DateType> & Omit<OmitPanelProps<PickerPanelTimeProps<DateType>>, 'format'>;
+} & PickerSharedProps<DateType> &
+  Omit<OmitPanelProps<PickerPanelTimeProps<DateType>>, 'format'>;
 
 export type PickerProps<DateType> =
   | PickerBaseProps<DateType>
@@ -194,9 +197,8 @@ function InnerPicker<DateType>(props: PickerProps<DateType>) {
   const [selectedValue, setSelectedValue] = React.useState<DateType | null>(mergedValue);
 
   // Operation ref
-  const operationRef: React.MutableRefObject<ContextOperationRefProps | null> = React.useRef<ContextOperationRefProps>(
-    null,
-  );
+  const operationRef: React.MutableRefObject<ContextOperationRefProps | null> =
+    React.useRef<ContextOperationRefProps>(null);
 
   // Open
   const [mergedOpen, triggerInnerOpen] = useMergedState(false, {
@@ -433,6 +435,7 @@ function InnerPicker<DateType>(props: PickerProps<DateType>) {
           triggerOpen(false);
         }}
         className={`${prefixCls}-clear`}
+        role="button"
       >
         {clearIcon || <span className={`${prefixCls}-clear-btn`} />}
       </span>
diff --git a/tests/__snapshots__/picker.spec.tsx.snap b/tests/__snapshots__/picker.spec.tsx.snap
index b5278a8bd..62782d3e5 100644
--- a/tests/__snapshots__/picker.spec.tsx.snap
+++ b/tests/__snapshots__/picker.spec.tsx.snap
@@ -20,6 +20,7 @@ exports[`Picker.Basic icon 1`] = `
   </span>
   <span
     class="rc-picker-clear"
+    role="button"
   >
     <span
       class="suffix-icon"