feat: add test, and change onInternalChange

Author: 刘欢

src/PickerInput/SinglePicker.tsx

@@ -273,7 +273,14 @@ function Picker<DateType extends object = any>(
   const onInternalChange: PickerProps<DateType>['onChange'] =
     onChange &&
     ((dates, dateStrings) => {
-      onChange(pickerParam(dates), pickerParam(dateStrings));
+      onChange(
+        pickerParam(dates),
+        pickerParam(
+          Array.isArray(dateStrings) && multiple && dateStrings.every((val) => !val)
+            ? null
+            : dateStrings,
+        ),
+      );
     });
 
   const [

src/PickerInput/hooks/useRangeValue.ts

@@ -283,11 +283,10 @@ export default function useRangeValue<ValueType extends DateType[], DateType ext
 
       // Trigger `onChange` if needed
       if (onChange && !isSameMergedDates) {
-        const everyEmpty = clone.every((val) => !val);
         onChange(
           // Return null directly if all date are empty
-          isNullValue && everyEmpty ? null : clone,
-          everyEmpty ? null : getDateTexts(clone),
+          isNullValue && clone.every((val) => !val) ? null : clone,
+          getDateTexts(clone),
         );
       }
     }

tests/multiple.spec.tsx

@@ -2,7 +2,7 @@
 import { fireEvent, render } from '@testing-library/react';
 import { resetWarned } from '@rc-component/util/lib/warning';
 import React from 'react';
-import { DayPicker, getDay, isOpen, openPicker, selectCell } from './util/commonUtil';
+import { clearValue, DayPicker, getDay, isOpen, openPicker, selectCell } from './util/commonUtil';
 
 const fakeTime = getDay('1990-09-03 00:00:00').valueOf();
 
@@ -57,6 +57,8 @@ describe('Picker.Multiple', () => {
     ]);
 
     expect(onChange.mock.calls[0][0]).toHaveLength(3);
+    clearValue();
+    expect(onChange).toHaveBeenCalledWith(null, null);
   });
 
   it('panel click to remove', () => {

tests/picker.spec.tsx

@@ -283,7 +283,7 @@ describe('Picker.Basic', () => {
         onChange.mockReset();
 
         clearValue();
-        expect(onChange).toHaveBeenCalledWith(null, null);
+        expect(onChange).toHaveBeenCalledWith(null, '');
         expect(isOpen()).toBeFalsy();
 
         openPicker(container);

tests/range.spec.tsx

@@ -202,7 +202,7 @@ describe('Picker.Range', () => {
     expect(onChange).toHaveBeenCalledWith([expect.anything(), null], ['1990-09-11', '']);
 
     clearValue();
-    expect(onChange).toHaveBeenCalledWith(null, null);
+    expect(onChange).toHaveBeenCalledWith(null, ['', '']);
     onChange.mockReset();
 
     // Not allow empty with startDate