search

tunnelvisionlabs / antlr4ts

Optimized TypeScript target for ANTLR 4
Other
642 stars 108 forks source link

Broken test discovery VS Test Explorer and mocha-typescript test #40

Open BurtHarris opened 8 years ago

BurtHarris commented 8 years ago

Visual studio gets confused when more than one file with these tests is present with the "Mocha" test framework property. Furthermore, it gets confused and associated the tests with the generated .js file, rather than the .ts source.

I can take another look at this once we get things better merged. For now I suggest we use the command-line tests, via npm test.

BurtHarris commented 8 years ago

This may be related: Microsoft/nodejstools#1087. It still seems like we are pushing the envelope by using the TypeScript outDir option.

Also after digging some, it seems that with the current decorator implementation, we don't can't get the right line number information in the decorator function to fully enable the Visual Studio style test explorer function. I filed a suggestion as Microsoft/Typescript#11466.

BurtHarris commented 7 years ago

It looks like this might be relevant: pana-cc/mocha-typescript#10. I haven't evaluated what this means yet.

BurtHarris commented 4 years ago

Latest PR removes dependency on mocha-typescript to resolve this issue.