theintern / intern

A next-generation code testing stack for JavaScript.
https://theintern.io/
Other
4.36k stars 311 forks source link

File coverage broken in Node < 10.16.0 #1111

Closed jason0x43 closed 4 years ago

jason0x43 commented 4 years ago

The file coverage functionality in Intern master, at least when running the self tests, will cause a stack overflow error for Node 10.0.0 - 10.15.3. The problem is that for Node < 10.16.0, both the instrumentation require hook and the instrumentation runInThisContext hook will be called for the same file, causing already instrumented code to be sent to the instrumenter.