revert: fix: render counting

mui · Apr 8, 2020 · 497ed0f · 497ed0f
1 parent 62f9f63
commit 497ed0f
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 12 deletions.
diff --git a/packages/material-ui-styles/src/StylesProvider/StylesProvider.test.js b/packages/material-ui-styles/src/StylesProvider/StylesProvider.test.js
@@ -159,13 +159,11 @@ describe('StylesProvider', () => {
           <Test />
         </StylesProvider>,
       );
-      assert.strictEqual(consoleErrorMock.callCount(), 2);
+      assert.strictEqual(consoleErrorMock.callCount(), 1);
       assert.include(
         consoleErrorMock.messages()[0],
         'you cannot use the jss and injectFirst props at the same time',
       );
-      // one error per render (twice rendered in StrictMode)
-      assert.strictEqual(consoleErrorMock.messages()[0], consoleErrorMock.messages()[1]);
     });
   });
 });
diff --git a/packages/material-ui/src/GridList/GridList.test.js b/packages/material-ui/src/GridList/GridList.test.js
@@ -189,13 +189,11 @@ describe('<GridList />', () => {
         </GridList>,
       );
 
-      assert.strictEqual(consoleErrorMock.callCount(), 2);
+      assert.strictEqual(consoleErrorMock.callCount(), 1);
       assert.include(
         consoleErrorMock.messages()[0],
         "Material-UI: the GridList component doesn't accept a Fragment as a child.",
       );
-      // one error per render (twice rendered in StrictMode)
-      assert.strictEqual(consoleErrorMock.messages()[0], consoleErrorMock.messages()[1]);
     });
   });
 });
diff --git a/packages/material-ui/src/Hidden/HiddenCss.test.js b/packages/material-ui/src/Hidden/HiddenCss.test.js
@@ -160,13 +160,11 @@ describe('<HiddenCss />', () => {
         </HiddenCss>,
       );
 
-      assert.strictEqual(consoleErrorMock.callCount(), 2);
+      assert.strictEqual(consoleErrorMock.callCount(), 1);
       assert.include(
         consoleErrorMock.messages()[0],
         'Material-UI: unsupported props received by `<Hidden implementation="css" />`: xxlUp.',
       );
-      // one error per render (twice rendered in StrictMode)
-      assert.strictEqual(consoleErrorMock.messages()[0], consoleErrorMock.messages()[1]);
     });
   });
 });
diff --git a/packages/material-ui/src/InputAdornment/InputAdornment.test.js b/packages/material-ui/src/InputAdornment/InputAdornment.test.js
@@ -150,13 +150,11 @@ describe('<InputAdornment />', () => {
             />
           </FormControl>,
         );
-        expect(consoleErrorMock.callCount()).to.equal(2);
+        expect(consoleErrorMock.callCount()).to.equal(1);
         expect(consoleErrorMock.messages()[0]).to.equal(
           'Material-UI: The `InputAdornment` variant infers the variant ' +
             'prop you do not have to provide one.',
         );
-        // one error per render (twice rendered in StrictMode)
-        expect(consoleErrorMock.messages()[0]).to.equal(consoleErrorMock.messages()[1]);
       });
     });
   });