Test arrow function inside generator.

[yavorsky]

Q	A
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Deprecations?
Spec Compliancy?
Tests Added/Pass?	y
Fixed Tickets
License	MIT
Doc PR
Dependency Changes

[codecov]

Codecov Report

Merging #5605 into 7.0 will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              7.0    #5605      +/-   ##
==========================================
+ Coverage   84.39%   84.45%   +0.05%     
==========================================
  Files         285      285              
  Lines        9647     9647              
  Branches     2710     2710              
==========================================
+ Hits         8142     8147       +5     
+ Misses       1003      998       -5     
  Partials      502      502

Impacted Files	Coverage Δ
...bel-plugin-transform-es2015-classes/src/vanilla.js	90.17% <0%> (+0.42%)	⬆️
packages/babel-traverse/src/path/modification.js	74.75% <0%> (+0.97%)	⬆️
packages/babel-traverse/src/path/context.js	86.2% <0%> (+1.72%)	⬆️
packages/babel-helper-call-delegate/src/index.js	68% <0%> (+4%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b917c0...7e6e8a2. Read the comment docs.

[loganfsmyth]

I think our tests are all describe and it.

I think this tests would be best as an exec.js test like https://github.com/babel/babel/blob/7.0/packages/babel-plugin-transform-regenerator/test/fixtures/function-sent/example/exec.js

[yavorsky]

@loganfsmyth thanks! Initially, I created this for 6.0 as an addition for #5567 and needed to prevent running code (error with arrow functions with node < 4).

I think our tests are all describe and it.

Yeah, accidentally looked into https://github.com/babel/babel/blob/7.0/packages/babel-plugin-transform-es2015-modules-commonjs/test/esmodule-flag.js

I could change base branch to 6.0 or just update it to run with exec.js.

[loganfsmyth]

Focusing on 7.x seems best at this point. If you just want to test the input/output without exec.js you can go the expect.js and actual.js route too, not strong feelings really.