tape-testing / tape

tap-producing test harness for node and browsers
MIT License
5.77k stars 307 forks source link

[readme] Update the documention for t.doesNotThrow() #554

Open m-r-r opened 3 years ago

m-r-r commented 3 years ago

Hello,

I updated the documentation for t.doesNotThrow() to clarify how it works.

Fixes #553.

ljharb commented 3 years ago

Hmm. Your text is correct, but node's docs say that it does check the type - in terms of what kind of error is thrown. Since tape fails the test instead of throwing an error, it kind of seems like we should be failing the test when it doesn't throw the right kind of error, and instead, throwing the error when it throws the wrong kind of error?