From df850772c45b0d73449ac9400d5428c37769a703 Mon Sep 17 00:00:00 2001
From: flavien <flaviendelangle@gmail.com>
Date: Mon, 2 Dec 2024 14:32:13 +0100
Subject: [PATCH] Fix

---
 .../MobileDatePicker/tests/MobileDatePicker.test.tsx | 12 ------------
 .../tests/MobileDateTimePicker.test.tsx              | 12 ------------
 .../MobileTimePicker/tests/MobileTimePicker.test.tsx | 12 ------------
 .../describeValue/testControlledUnControlled.tsx     |  4 ++--
 4 files changed, 2 insertions(+), 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
index 74188094c7da..1c6255189b9e 100644
--- 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
index 51a62e3da389..474374f61806 100644
--- 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
index 4b052523d02e..6fd96148682b 100644
--- 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
index 272360c609c0..318f1baaafab 100644
--- 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', () => {