Rewire only in a test environment

[BenjaminVanRyseghem]

Rewire conflict with babel-plugin-wildcard.

As a workaround (which could benefits others), I make rewire run only in a test environment.

[speedskater]

@BenjaminVanRyseghem sorry for the late reply. I have just had a look at your PR.Thank you for your work in providing this feature to babel-plugin-rewire. If I haven't missunderstood your intention, I think the same behaviour can be achieved by specifying babel-plugin-rewire only in the desired babel environment in the babelrc file. I think this leaves the decision when to apply the plugin to the user. Is this okay for you?

[BenjaminVanRyseghem]

Thanks for your answer.

I'm kinda new to babel, and didn't find how to specify environment in babelrc. Do you have some documentation for me?

[speedskater]

@BenjaminVanRyseghem this is a sample of one of the projects I am involved. Please keep in mind that this is not the complete configuration and it is not checked against babel 7. For further information I would recommend the babel documentation itself: https://babeljs.io/docs/usage/babelrc/

In this example we have general plugins and presets which are applicable to all environments. Further we have a specific configuration for our karma tests which adds istanbul for code coverage and the babel-plugin-rewire.

{
	"plugins": [
		"transform-runtime",
		"syntax-flow",
		"transform-flow-strip-types"
	],
	"presets": [
		"es2015",
		"react"
	],
	"babelrc": false,
	"env": {
		"karma": {
			"plugins": [
				["istanbul", {
					"exclude": [
						"tests/**/*"
					]
				}], "rewire"
			]
		}
	}
}

Hope this is able to clarify your issue?

[BenjaminVanRyseghem]

Thanks! I'll have a look :smile:

I found this stating it's deprecated, but I'll give it a try asap :smile:

[BenjaminVanRyseghem]

sorry for the delay :smile:

It works, thank you. But as it seems deprecated, it looks like this will be needed anytime soon. What do you think?

[tfrommen]

As Babel 7 does/will not use env any longer, this PR is actually still interesting. I'm with @BenjaminVanRyseghem here, for what it's worth. 🙂

[BenjaminVanRyseghem]

ping?

[josh08h]

I'd be interested in getting this PR merged. We are on babel 7 and would like to only have this plugin run against spec files.

[theKashey]

Disabling ‘rewire’ in not-test environments could be a bad thing, as long using rewire in not-test(And not production) environment could be a good.

[Sinewyk]

I think this PR is still not needed, ok env will probably be deprecated. But nothing is stopping you from using a .babelrc.js instead of a .babelrc, and once you have a js file you can do whatever you want.

[KosnIre]

It looks like the babel team eventually decided they would continue to support env-specific configs in babelrc, so I think this issue can be closed.

https://github.com/babel/babel/issues/5276#issuecomment-348598859

https://babeljs.io/docs/en/configuration#how-babel-merges-config-items

I'm on babel 7 and this config is working for me:

{
  "env": {
    "test": {
      "plugins": [
        "rewire"
      ]
    }
  }
}