Remove console.warn of babel-plugin-import when the tests are running

Environment(required)

• antd version: 2.8 & 2.9
• OS and its version: Ubuntu 16.04
• Browser and its version: Chrome 57.0.2987.98 (64-bit)

What did you do? Please provide steps to re-produce your problem.

Run npm test in:

• Project antd.
• Another project created with Create-React-App

What do you expected?

No warnings.

What happen?

Both projects always show this message:

PASS  components/locale-provider/__tests__/index.test.js
  Console
    console.warn components/index.tsx:60
      You are using a whole package of antd,
      please use https://www.npmjs.com/package/babel-plugin-import to reduce app bundle size.

I think it will be important remove it when you’re executing the tests.

I think, too, the solution is change this kind of code:

if (process.env.NODE_ENV !== 'production') {
    if (typeof console !== 'undefined' && console.warn && typeof window !== 'undefined') {
        console.warn(`You are using a whole package of antd,
please use https://www.npmjs.com/package/babel-plugin-import to reduce app bundle size.`);
    }
}

Providing another conditional for remove warnings if you are lauching the tests. What do you think about it?

Re-producible online demo

Just download the antd project and run npm test.

[andriijas]

#10363

[trinitroglycerin]

Not only that, but this assumes the user is using Babel. I use Typescript with Webpack. I know that, because of Webpack, dead code is going to be eliminated. This warning is useless in this scenario and there’s no way to turn it off.

I don’t need this warning cluttering up my terminal. You shouldn’t expect the user to change NODE_ENV to production to disable this. In tests, NODE_ENV should be test. You also really shouldn’t encourage the user to know about the directory structure of your library. You have exposed a context module, as is the usual pattern; let users use it without being told off.

http://www.linfo.org/rule_of_silence.html