`assert.throws` doesn't check type of error leading to many false positives in test262
We pass **a lot** of tests in `intl402` and `built-ins/Temporal` even though they aren't implemented at all, because the `assert.throws` implementation in the harness doesn't check type of errors, so we pass any test that only checks for errors with `ReferenceError`s.
Currently builtin types don't extend types they should e.g. `new ReferenceError() instanceof Error == false`, so fixing this might lead to some false negatives, although far less than the amount of false positives right now.
1 条评论