"require" option doesn't load precompiled helpers

[novemberborn]

From #1187 (comment), when test helpers are specified in the "require" option, AVA does not load the precompiled file.

The problem seems to be that we install our loader too late:

ava/lib/test-worker.js

Line 33 in 9c9dfef

adapter.installPrecompilerHook();

I suspect that it might be OK if we move that line up before:

ava/lib/test-worker.js

Line 30 in 9c9dfef

(opts.require || []).forEach(require);

We need to test how this interacts with babel-register, and add an integration test to verify we require the precompiled helpers.

[darthgera123]

is this issue still open or is it closed?

[novemberborn]

@darthgera123 it's still open.

[JavierPons]

Hi there,

I'm interesting in help in this issue. It will be my first intervention. (Im a beginner).
What concretely do you need? Where I can help in this? Or if you need help in an other issue as well Im ready to participate.

[novemberborn]

@JavierPons that's great! Beyond what's in the issue description at the top, what information are you looking for?

[JavierPons]

@novemberborn Im looking for some issue with code that I can make better for example or if it is a bug that I can resolve. (Maybe its very high level for me :p ) But I would like to try.

[novemberborn]

@JavierPons by all means, go ahead.

As the issue describes, you probably need to move up the adapter.installPrecompilerHook(); line. Test it on a local installation first (use npm link so you can test against your local AVA checkout). Helper files are files inside a test folder that start with an underscore. If you use import to require a dependency in that helper file it'll be a good way of checking whether the precompiled version was loaded.

If that's all looking good you'll have to make an integration test. There should be bunch of examples in test/cli.js.

Please open a PR with your initial changes so we can iterate on them more effectively. Thanks!