--watch not working with typescript recipe

[midgethoen]

Im new to ava so I followed the generic setup and then the typescript recipe.

running ava finds the test.ts file nicely. but --watch doesn't work then.. whut?!

when renaming test.ts to test.js it does work;
when running ava --watch test.ts it also does work.

[novemberborn]

Oh nice find! I don't think the watcher provides the extensions to the file globber:

ava/lib/watcher.js

Line 83 in 3ac2a8f

this.avaFiles = new AvaFiles({

We also need to provide them to the watcher itself:

ava/lib/cli.js

Line 237 in 3ac2a8f

const watcher = new Watcher(reporter, api, files, arrify(conf.sources));

@midgethoen would you be interested in working on this?

[midgethoen]

To be honest I was trying out several testing frameworks to see which appeals most to me. So I’m not even sure yet if this will be ava. However I have been wanting to contribute to open source for some time and the issue seems pretty straight forward.

I’ll give it a go. Although I’m a bit occupied atm. So it might be in a litte while.

[novemberborn]

That sounds great @midgethoen. Though this is a bit of an embarrassing bug, we should tackle it before we do the 1.0 release.

[midgethoen]

I have some time tonight. Let's see how far I get :)

[novemberborn]

@midgethoen oh no, I stepped on your toes!

I got as far as https://github.com/avajs/ava/tree/watcher-custom-extensions. Perhaps you could test that with your setup if you still have it lying around? npm install avajs/ava#watcher-custom-extensions should do the trick.

If that works we just need some tests in test/integration/watcher. That's not particularly glamorous work though, so feel free to punt that back to me 😉

[midgethoen]

@midgethoen oh no, I stepped on your toes!

Ah, That's ok 😁

The fix works like a charm for me!

[novemberborn]

Glad to hear, thanks @midgethoen.