Make watch mode dependency tracking work with custom require hooks

[lo1tuma]

Fixes: #2049

I’ve tested these fixes with one of my projects where sources and test files are written in typescript. I use @babel/register and @babel/typescript instead of tsc.

BTW: I’ve noticed that the sources patterns in ava.config.js must not start with a leading ./ otherwise ava-files doesn’t recognize the files as source files.

[novemberborn]

Thanks for the PR @lo1tuma! Some comments below.

I’ve noticed that the sources patterns in ava.config.js must not start with a leading ./ otherwise ava-files doesn’t recognize the files as source files.

That seems odd. You should still be able to specify patterns within directories that are then treated as source files.

[lo1tuma]

That seems odd. You should still be able to specify patterns within directories that are then treated as source files.

Specifying directories works, but as I said only without leading ./.
Here is a small example to explain the current behavior a bit better:

Directory structure:

├── bar
│  └── b.js
├── foo
│  └── a.js
└── test
│  └── spec.js
├── ava.config.js
└── package.json

When I specify sources as [ './foo/**/*', './bar/**/*'] it doesn’t work, but [ 'foo/**/*', 'bar/**/*'] works.
I’m not really sure if this is an issue or the expected behavior, but it cost me some time to figure it out so I though it is worth mentioning.

[novemberborn]

Hey @lo1tuma, apologies for not reviewing your changes. I've been trying to improve the file globbing itself, see #2103. My focus will be on landing that. Hopefully it also deals with #2085.

[novemberborn]

Hey @lo1tuma, #2103 which fixed #2085. Would you like to update this PR to address #2049?

[lo1tuma]

Hi @novemberborn, great work with #2103. I’ve updated this PR to only address #2049.

[novemberborn]

Thanks @lo1tuma!