Commit ca53866
test: add message verification on assert.throws
Add regular expression for error message validation to instances of
assert.throws() in test-assert.js.
PR-URL: #10890
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>1 parent 38b123c commit ca53866
1 file changed
Lines changed: 10 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -90,11 +90,16 @@ assert.doesNotThrow(makeBlock(a.deepEqual, /a/g, /a/g)); | |||
| 90 | 90 | assert.doesNotThrow(makeBlock(a.deepEqual, /a/i, /a/i)); | |
| 91 | 91 | assert.doesNotThrow(makeBlock(a.deepEqual, /a/m, /a/m)); | |
| 92 | 92 | assert.doesNotThrow(makeBlock(a.deepEqual, /a/igm, /a/igm)); | |
| 93 | - assert.throws(makeBlock(a.deepEqual, /ab/, /a/)); | ||
| 94 | - assert.throws(makeBlock(a.deepEqual, /a/g, /a/)); | ||
| 95 | - assert.throws(makeBlock(a.deepEqual, /a/i, /a/)); | ||
| 96 | - assert.throws(makeBlock(a.deepEqual, /a/m, /a/)); | ||
| 97 | - assert.throws(makeBlock(a.deepEqual, /a/igm, /a/im)); | ||
| 93 | + assert.throws(makeBlock(a.deepEqual, /ab/, /a/), | ||
| 94 | + /^AssertionError: \/ab\/ deepEqual \/a\/$/); | ||
| 95 | + assert.throws(makeBlock(a.deepEqual, /a/g, /a/), | ||
| 96 | + /^AssertionError: \/a\/g deepEqual \/a\/$/); | ||
| 97 | + assert.throws(makeBlock(a.deepEqual, /a/i, /a/), | ||
| 98 | + /^AssertionError: \/a\/i deepEqual \/a\/$/); | ||
| 99 | + assert.throws(makeBlock(a.deepEqual, /a/m, /a/), | ||
| 100 | + /^AssertionError: \/a\/m deepEqual \/a\/$/); | ||
| 101 | + assert.throws(makeBlock(a.deepEqual, /a/igm, /a/im), | ||
| 102 | + /^AssertionError: \/a\/gim deepEqual \/a\/im$/); | ||
| 98 | 103 | ||
| 99 | 104 | { | |
| 100 | 105 | const re1 = /a/g; | |
0 commit comments