Do not include babel-register in test helper

[danez]

Seems this is not needed for the internal tests? Not sure if this might be used outside of babel and break stuff there?

I discovered that while debugging the pirates integration and was curious why we do this, although not needed. Anybody knows?

[hzoo]

Yeah I was getting this issue before, but I thought this was transpiling the required tests? Not sure @kittens

[codecov-io]

Current coverage is 88.47% (diff: 100%)

Merging #3669 into master will increase coverage by 0.12%

@@             master      #3669   diff @@
==========================================
  Files           194        192     -2   
  Lines         13624      13491   -133   
  Methods        1427       1412    -15   
  Messages          0          0          
  Branches       3151       3126    -25   
==========================================
- Hits          12037      11936   -101   
+ Misses         1587       1555    -32   
  Partials          0          0          

Powered by Codecov. Last update d7533e8...5e8c6f5

[danez]

By looking more at it it seems to ignore anyway everything (node_modules and ./packages/)

{
  ignore: [
    path.resolve(__dirname + "/../.."),
    "node_modules",
  ]
}

And we don't have code or tests outside of that.

I'm going to merge these and will keep an eye on this after the release, I tested with some external projects that use this module, but couldn't find any usecase.

[hzoo]

This means we can remove the babel-register dep?

[danez]

Oh ups, forgot that