add comment

mui · Nov 28, 2022 · 0faa8b3 · 0faa8b3
1 parent e2aba9a
commit 0faa8b3
Show file tree

Hide file tree

Showing 16 changed files with 53 additions and 16 deletions.
diff --git a/packages/mui-material/src/Alert/Alert.test.js b/packages/mui-material/src/Alert/Alert.test.js
@@ -23,7 +23,10 @@ describe('<Alert />', () => {
       closeButton: {},
       closeIcon: {},
     },
-    skip: ['componentsProp', 'slotPropsCallback'],
+    skip: [
+      'componentsProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   describe('prop: square', () => {

diff --git a/packages/mui-material/src/Autocomplete/Autocomplete.test.js b/packages/mui-material/src/Autocomplete/Autocomplete.test.js
@@ -68,7 +68,7 @@ describe('<Autocomplete />', () => {
         'componentsProp',
         'slotsProp',
         'reactTestRenderer',
-        'slotPropsCallback',
+        'slotPropsCallback', // not supported yet
       ],
     }),
   );

diff --git a/packages/mui-material/src/AvatarGroup/AvatarGroup.test.js b/packages/mui-material/src/AvatarGroup/AvatarGroup.test.js
@@ -24,7 +24,11 @@ describe('<AvatarGroup />', () => {
       slots: {
         additionalAvatar: { expectedClassName: classes.avatar },
       },
-      skip: ['componentsProp', 'slotsProp', 'slotPropsCallback'],
+      skip: [
+        'componentsProp',
+        'slotsProp',
+        'slotPropsCallback', // not supported yet
+      ],
     }),
   );
 

diff --git a/packages/mui-material/src/Backdrop/Backdrop.test.js b/packages/mui-material/src/Backdrop/Backdrop.test.js
@@ -26,7 +26,7 @@ describe('<Backdrop />', () => {
       'componentsProp',
       // react-transition-group issue
       'reactTestRenderer',
-      'slotPropsCallback',
+      'slotPropsCallback', // not supported yet
     ],
   }));
 

diff --git a/packages/mui-material/src/FilledInput/FilledInput.test.js b/packages/mui-material/src/FilledInput/FilledInput.test.js
@@ -22,7 +22,11 @@ describe('<FilledInput />', () => {
       root: { expectedClassName: classes.root, testWithElement: null },
       input: { expectedClassName: classes.input, testWithElement: null },
     },
-    skip: ['componentProp', 'componentsProp', 'slotPropsCallback'],
+    skip: [
+      'componentProp',
+      'componentsProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   it('should have the underline class', () => {

diff --git a/packages/mui-material/src/FormControlLabel/FormControlLabel.test.js b/packages/mui-material/src/FormControlLabel/FormControlLabel.test.js
@@ -22,7 +22,12 @@ describe('<FormControlLabel />', () => {
     slots: {
       typography: { expectedClassName: classes.label },
     },
-    skip: ['componentProp', 'componentsProp', 'slotsProp', 'slotPropsCallback'],
+    skip: [
+      'componentProp',
+      'componentsProp',
+      'slotsProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   describe('prop: label', () => {

diff --git a/packages/mui-material/src/Input/Input.test.js b/packages/mui-material/src/Input/Input.test.js
@@ -22,7 +22,11 @@ describe('<Input />', () => {
       root: { expectedClassName: classes.root, testWithElement: null },
       input: { expectedClassName: classes.input, testWithElement: null },
     },
-    skip: ['componentProp', 'componentsProp', 'slotPropsCallback'],
+    skip: [
+      'componentProp',
+      'componentsProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   it('should forward classes to InputBase', () => {

diff --git a/packages/mui-material/src/InputBase/InputBase.test.js b/packages/mui-material/src/InputBase/InputBase.test.js
@@ -27,7 +27,10 @@ describe('<InputBase />', () => {
       root: { expectedClassName: classes.root, testWithElement: null },
       input: { expectedClassName: classes.input, testWithElement: null },
     },
-    skip: ['componentProp', 'slotPropsCallback'],
+    skip: [
+      'componentProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   it('should render an <input /> inside the div', () => {

diff --git a/packages/mui-material/src/ListItem/ListItem.test.js b/packages/mui-material/src/ListItem/ListItem.test.js
@@ -26,7 +26,10 @@ describe('<ListItem />', () => {
     slots: {
       root: {},
     },
-    skip: ['componentsProp', 'slotPropsCallback'],
+    skip: [
+      'componentsProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   it('should render with gutters classes', () => {

diff --git a/packages/mui-material/src/Modal/Modal.test.js b/packages/mui-material/src/Modal/Modal.test.js
@@ -43,7 +43,7 @@ describe('<Modal />', () => {
         'themeDefaultProps', // portal, can't determine the root
         'themeStyleOverrides', // portal, can't determine the root
         'reactTestRenderer', // portal https://github.com/facebook/react/issues/11565
-        'slotPropsCallback',
+        'slotPropsCallback', // not supported yet
       ],
     }),
   );

diff --git a/packages/mui-material/src/OutlinedInput/OutlinedInput.test.js b/packages/mui-material/src/OutlinedInput/OutlinedInput.test.js
@@ -23,7 +23,11 @@ describe('<OutlinedInput />', () => {
       root: { expectedClassName: classes.root, testWithElement: null },
       input: { expectedClassName: classes.input, testWithElement: null },
     },
-    skip: ['componentProp', 'componentsProp', 'slotPropsCallback'],
+    skip: [
+      'componentProp',
+      'componentsProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   it('should render a NotchedOutline', () => {

diff --git a/packages/mui-material/src/PaginationItem/PaginationItem.test.js b/packages/mui-material/src/PaginationItem/PaginationItem.test.js
@@ -27,7 +27,7 @@ describe('<PaginationItem />', () => {
       // uses non-standard camel-case fields in `components`
       'slotsProp',
       'slotPropsProp',
-      'slotPropsCallback',
+      'slotPropsCallback', // not supported yet
     ],
   }));
 

diff --git a/packages/mui-material/src/Popper/Popper.test.js b/packages/mui-material/src/Popper/Popper.test.js
@@ -37,7 +37,7 @@ describe('<Popper />', () => {
       'themeVariants',
       // https://github.com/facebook/react/issues/11565
       'reactTestRenderer',
-      'slotPropsCallback',
+      'slotPropsCallback', // not supported yet
     ],
   }));
 

diff --git a/packages/mui-material/src/Slider/Slider.test.js b/packages/mui-material/src/Slider/Slider.test.js
@@ -64,7 +64,9 @@ describe('<Slider />', () => {
           expectedClassName: classes.markLabel,
         },
       },
-      skip: ['slotPropsCallback'],
+      skip: [
+        'slotPropsCallback', // not supported yet
+      ],
     }),
   );
 

diff --git a/packages/mui-material/src/StepLabel/StepLabel.test.js b/packages/mui-material/src/StepLabel/StepLabel.test.js
@@ -21,7 +21,12 @@ describe('<StepLabel />', () => {
     slots: {
       label: { expectedClassName: classes.label },
     },
-    skip: ['componentProp', 'componentsProp', 'slotsProp', 'slotPropsCallback'],
+    skip: [
+      'componentProp',
+      'componentsProp',
+      'slotsProp',
+      'slotPropsCallback', // not supported yet
+    ],
   }));
 
   describe('label content', () => {

diff --git a/packages/mui-material/src/Tooltip/Tooltip.test.js b/packages/mui-material/src/Tooltip/Tooltip.test.js
@@ -59,7 +59,7 @@ describe('<Tooltip />', () => {
         'themeVariants',
         // react-transition-group issue
         'reactTestRenderer',
-        'slotPropsCallback',
+        'slotPropsCallback', // not supported yet
       ],
     }),
   );