Skip to content

Commit 5d79cb8

Browse files
AmitShwartsadids1221
authored andcommitted
fix: [MADS-4071] clear picker value on web (#2986)
* fix: clear picker value on web * fix test
1 parent c225124 commit 5d79cb8

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/components/picker/__tests__/index.spec.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ describe('Picker', () => {
7878
describe('Test selection', () => {
7979
it('Should select a single item', () => {
8080
const driver = getDriver();
81-
expect(driver.getValue()).toEqual(undefined);
81+
expect(driver.getValue()).toEqual('');
8282
expect(driver.isOpen()).toBeFalsy();
8383
driver.open();
8484
expect(driver.isOpen()).toBeTruthy();
@@ -89,7 +89,7 @@ describe('Picker', () => {
8989

9090
it('Should select multiple items', () => {
9191
const driver = getDriver({mode: 'MULTI'});
92-
expect(driver.getValue()).toEqual(undefined);
92+
expect(driver.getValue()).toEqual('');
9393
expect(driver.isOpen()).toBeFalsy();
9494
driver.open();
9595
expect(driver.isOpen()).toBeTruthy();

src/components/picker/helpers/usePickerLabel.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ const usePickerLabel = (props: UsePickerLabelProps) => {
5757
getLabelsFromArray,
5858
getLabel: _getLabel,
5959
accessibilityInfo,
60-
label: _getLabel(value)
60+
label: _getLabel(value) ?? ''
6161
};
6262
};
6363

0 commit comments

Comments
 (0)