Fix Babel issues in tests by applying the right transforms

[gaearon]

This is a followup to #1177 with a less aggressive fix.
For context, read threads #1156 and #1160.

Why this works:

• Babel has a bug where object/spread depends on parameters transform: Object-rest-spread doesn't work in case of destructuring with default object parameter babel/babel#4851. So we have to enable parameters.
• Including regenerator transform in test environment was incorrect. Its async: false option only made sense for development and production configuration where we use babel-preset-latest, and so async functions are already being handled. But for test environment we use babel-preset-env instead of babel-preset-latest, and so including regenerator second time is redundant (and somehow breaks things—@hzoo could you clarify why this happened?)

I am remove destructuring and arrow-functions because they were added as stopgap measure in #1177 but turned out unnecessary per #1156 (comment).

Fixes#1156 and #1160.

[gaearon]

Test file I used in different environments:

importReact,{Component}from'react';importlogofrom'./logo.svg';import'./App.css';exportfunction*foo(){yield42;yield49;}exportasyncfunctionlol(){returnawaitPromise.resolve(5000);}classAppextendsComponent{render(){return(<divclassName="App"><divclassName="App-header"><imgsrc={logo}className="App-logo"alt="logo"/><h2>Welcome to React</h2></div><pclassName="App-intro">
To get started, edit <code>src/App.js</code> and save to reload.
</p></div>);}}varaddIndex=functionaddIndex({ index, ...routes}={},base){};functiondoSometing(){return{};}constgetId=async()=>{const{ id }=awaitdoSometing(42)returnid};constgetId2=async()=>{constp=awaitdoSometing(42)const{ id }=preturnid};functionRecord(){returnfunction(){}}constUser=newRecord({id: NaN})newUser({id: 42})exportconstx=async()=>{vary=foo()varz=y.next()while(!z.done){console.log(z.value);z=y.next();}constwow=awaitlol();console.log(wow)};x();exportdefaultApp;

I tried Chrome, IE11, Node 4, and Node 7.

[hzoo]

Just more explaination for future ref

Babel has a bug where object/spread depends on parameters transform: babel/babel#4851. So we have to enable parameters.

This is simply because I didn't account for a default in object rest spread so currently the object/rest spread transform doesn't do anything to the code and thus the syntax error.

But for test environment we use babel-preset-env instead of babel-preset-latest, and so including regenerator second time is redundant (and somehow breaks things—@hzoo could you clarify why this happened?)

Still need to look into it

[hzoo]

Expected type "Expression" with option {} is coming from babel-types and after getting the stack trace from jest it is coming from transform-regenerator/lib/emit.js

Error: /cra-jest-babel-types/src/merchants.js: ObjectPattern
at Object.t.(anonymous function) [as assertExpression] (/cra-jest-babel-types/node_modules/babel-types/lib/index.js:362:13)
at finish (/cra-jest-babel-types/node_modules/babel-plugin-transform-regenerator/lib/emit.js:623:7)
at Emitter.Ep.explodeExpression (/cra-jest-babel-types/node_modules/babel-plugin-transform-regenerator/lib/emit.js:632:12)

Ep.explodeExpression = function (path, ignoreResult) {
var expr = path.node;
if (expr) {
t.assertExpression(expr);

which is checking an Expression but getting an ObjectPattern via const {result} = doSometing(); (which isn't aliased as an expression since it isn't)

@benjamn just reimplemented the transform in facebook/regenerator#259
and has a PR to update it with some other fixes via babel/babel#4881 so maybe that fixes it, not sure.

[benjamn]

Other transforms are sometimes necessary to satisfy the preconditions of Regenerator. That's why regenerator-preset exists. This PR seems like the right approach, though the errors could always be more helpful.

[gaearon]

Do we need to add everything from regenerator-preset in the mode when we use -env?

[benjamn]

I wouldn't literally use regenerator-preset just yet, since babel/babel#4881 still needs to be merged, but the other plugins are probably worth including whenever Regenerator is used.

[gaearon]

Hmm. Is there ever a case where Node supports classes/arrows/block-scoping/for-of but not generators?

[Jessidhia]

Node 4 / 5 don't support generator return / throw, though they support everything else about them. This also comes into play when break/return/throwing inside a for..of loop, but for..of doesn't implement that either in Node < 6; so you could argue Node 4 / 5 don't actually support for-of. io.js, which is the closest Node 4 predecessor, doesn't support generators but they also don't support arrows / block-scoping.

[gaearon]

@hzoo Does -env take care of this? Should I file an issue?

[Jessidhia]

@gaearon Looks like it does, for both for-of and regenerator; generators were fixed a v8 release before for-of (according to compat-table).

[gaearon]

Sounds great. Thanks!

[gaearon]

Fixed in 0.8.2.
Please verify.

https://github.com/facebookincubator/create-react-app/releases/tag/v0.8.2

[EnoahNetzach]

tests are OK for me