testdouble / teenytest

A very simple, zero-config test runner for Node.js
MIT License
96 stars 14 forks source link

show loading errors #59

Closed giltayar closed 4 years ago

giltayar commented 4 years ago

If a test module fails to load (syntax error or any other exception), teenytest is currently silent about it. Not sure whether this is a regression due to the ESM change, or was always like this, but this PR fixes it (with a test).

searls commented 4 years ago

Either way, this seems like a good idea. BTW, I think assert.match is a Node 12+ -ism

giltayar commented 4 years ago

Oops. Ouch. I'll fix that. In my usual projects, I have an eslint plugin that warns me of these problems, so I've stopped looking.

searls commented 4 years ago

Landed in 6.0.2