swallow errors instead

Author: oliviertassinari

package.json

@@ -33,13 +33,13 @@
     "size:why": "size-limit --why packages/material-ui/build/index.js",
     "start": "yarn docs:dev",
     "test": "yarn lint && yarn typescript && yarn test:coverage",
-    "test:coverage": "cross-env INTERNAL=true NODE_ENV=test BABEL_ENV=coverage nyc mocha 'packages/**/*.test.js' 'docs/**/*.test.js' --exclude '**/node_modules/**' && nyc report -r lcovonly",
-    "test:coverage:html": "cross-env INTERNAL=true NODE_ENV=test BABEL_ENV=coverage nyc mocha 'packages/**/**/*.test.js' --exclude '**/node_modules/**' && nyc report --reporter=html",
-    "test:karma": "cross-env INTERNAL=true NODE_ENV=test karma start test/karma.conf.js",
+    "test:coverage": "cross-env NODE_ENV=test BABEL_ENV=coverage nyc mocha 'packages/**/*.test.js' 'docs/**/*.test.js' --exclude '**/node_modules/**' && nyc report -r lcovonly",
+    "test:coverage:html": "cross-env NODE_ENV=test BABEL_ENV=coverage nyc mocha 'packages/**/**/*.test.js' --exclude '**/node_modules/**' && nyc report --reporter=html",
+    "test:karma": "cross-env NODE_ENV=test karma start test/karma.conf.js",
     "test:regressions": "yarn test:regressions:build && rimraf test/regressions/screenshots/chrome/* && vrtest run --config test/vrtest.config.js --record",
     "test:regressions:build": "webpack --config test/regressions/webpack.config.js",
     "test:umd": "node packages/material-ui/test/umd/run.js",
-    "test:unit": "cross-env INTERNAL=true NODE_ENV=test mocha 'packages/**/*.test.js' 'docs/**/*.test.js' 'scripts/**/*.test.js' --exclude '**/node_modules/**'",
+    "test:unit": "cross-env NODE_ENV=test mocha 'packages/**/*.test.js' 'docs/**/*.test.js' 'scripts/**/*.test.js' --exclude '**/node_modules/**'",
     "test:watch": "yarn test:unit --watch",
     "typescript": "lerna run --no-bail --parallel typescript"
   },

packages/material-ui/src/RootRef/RootRef.js

@@ -66,7 +66,7 @@ class RootRef extends React.Component {
 
   render() {
     if (process.env.NODE_ENV !== 'production') {
-      if (!warnedOnce && process.env.INTERNAL !== 'true') {
+      if (!warnedOnce) {
         warnedOnce = true;
         console.warn(
           [

packages/material-ui/src/RootRef/RootRef.test.js

@@ -2,6 +2,7 @@ import * as React from 'react';
 import PropTypes from 'prop-types';
 import { expect } from 'chai';
 import { spy } from 'sinon';
+import { consoleWarnMock } from 'test/utils/consoleErrorMock';
 import createMount from 'test/utils/createMount';
 import RootRef from './RootRef';
 
@@ -11,6 +12,15 @@ describe('<RootRef />', () => {
   // StrictModeViolation: uses findDOMNode
   const mount = createMount({ strict: false });
 
+  beforeEach(() => {
+    PropTypes.resetWarningCache();
+    consoleWarnMock.spy();
+  });
+
+  afterEach(() => {
+    consoleWarnMock.reset();
+  });
+
   it('call rootRef function on mount and unmount', () => {
     const rootRef = spy();
     const wrapper = mount(

test/karma.conf.js

@@ -55,7 +55,6 @@ module.exports = function setKarmaConfig(config) {
           'process.env': {
             NODE_ENV: JSON.stringify('test'),
             CI: JSON.stringify(process.env.CI),
-            INTERNAL: JSON.stringify(process.env.INTERNAL),
           },
         }),
       ],