Fix

mui · Dec 2, 2024 · df85077 · df85077
1 parent d93c25b
commit df85077
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 38 deletions.
diff --git a/packages/x-date-pickers/src/MobileDatePicker/tests/MobileDatePicker.test.tsx b/packages/x-date-pickers/src/MobileDatePicker/tests/MobileDatePicker.test.tsx
@@ -11,7 +11,6 @@ import {
   expectFieldValueV7,
   buildFieldInteractions,
   openPicker,
-  getFieldSectionsContainer,
 } from 'test/utils/pickers';
 
 describe('<MobileDatePicker />', () => {
@@ -132,17 +131,6 @@ describe('<MobileDatePicker />', () => {
   });
 
   describe('picker state', () => {
-    it('should open when clicking the input', () => {
-      const onOpen = spy();
-
-      render(<MobileDatePicker onOpen={onOpen} />);
-
-      fireEvent.click(getFieldSectionsContainer());
-
-      expect(onOpen.callCount).to.equal(1);
-      expect(screen.queryByRole('dialog')).toBeVisible();
-    });
-
     it('should call `onAccept` even if controlled', () => {
       const onAccept = spy();
 

diff --git a/packages/x-date-pickers/src/MobileDateTimePicker/tests/MobileDateTimePicker.test.tsx b/packages/x-date-pickers/src/MobileDateTimePicker/tests/MobileDateTimePicker.test.tsx
@@ -8,7 +8,6 @@ import {
   createPickerRenderer,
   openPicker,
   getClockTouchEvent,
-  getFieldSectionsContainer,
 } from 'test/utils/pickers';
 
 describe('<MobileDateTimePicker />', () => {
@@ -83,17 +82,6 @@ describe('<MobileDateTimePicker />', () => {
   });
 
   describe('picker state', () => {
-    it('should open when clicking the input', () => {
-      const onOpen = spy();
-
-      render(<MobileDateTimePicker onOpen={onOpen} />);
-
-      fireEvent.click(getFieldSectionsContainer());
-
-      expect(onOpen.callCount).to.equal(1);
-      expect(screen.queryByRole('dialog')).toBeVisible();
-    });
-
     it('should call onChange when selecting each view', function test() {
       if (typeof window.Touch === 'undefined' || typeof window.TouchEvent === 'undefined') {
         this.skip();

diff --git a/packages/x-date-pickers/src/MobileTimePicker/tests/MobileTimePicker.test.tsx b/packages/x-date-pickers/src/MobileTimePicker/tests/MobileTimePicker.test.tsx
@@ -8,24 +8,12 @@ import {
   adapterToUse,
   openPicker,
   getClockTouchEvent,
-  getFieldSectionsContainer,
 } from 'test/utils/pickers';
 
 describe('<MobileTimePicker />', () => {
   const { render } = createPickerRenderer({ clock: 'fake' });
 
   describe('picker state', () => {
-    it('should open when clicking the input', () => {
-      const onOpen = spy();
-
-      render(<MobileTimePicker onOpen={onOpen} />);
-
-      fireEvent.click(getFieldSectionsContainer());
-
-      expect(onOpen.callCount).to.equal(1);
-      expect(screen.queryByRole('dialog')).toBeVisible();
-    });
-
     it('should fire a change event when meridiem changes', () => {
       const handleChange = spy();
       render(

diff --git a/test/utils/pickers/describeValue/testControlledUnControlled.tsx b/test/utils/pickers/describeValue/testControlledUnControlled.tsx
@@ -157,7 +157,7 @@ export const testControlledUnControlled: DescribeValueTestSuite<any, any> = (
       });
     });
 
-    it('should not allow editing with keyboard in mobile pickers', () => {
+    it('should allow editing with keyboard in mobile pickers', () => {
       if (componentFamily !== 'picker' || params.variant !== 'mobile') {
         return;
       }
@@ -170,7 +170,7 @@ export const testControlledUnControlled: DescribeValueTestSuite<any, any> = (
       });
       v7Response.selectSection(undefined);
       fireUserEvent.keyPress(v7Response.getActiveSection(0), { key: 'ArrowUp' });
-      expect(handleChange.callCount).to.equal(0);
+      expect(handleChange.callCount).to.equal(1);
     });
 
     it('should have correct labelledby relationship when toolbar is shown', () => {