Skip to content

Commit 2856590

Browse files
committed
Improve customError testing
1 parent 6dc8d10 commit 2856590

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

test/helpers/customError.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,10 @@ const { expect } = require('chai');
22

33
/** Revert handler that supports custom errors. */
44
async function expectRevertCustomError(promise, expectedErrorName, args) {
5+
let promiseSuccesseded = false;
56
try {
67
await promise;
7-
expect.fail("Expected promise to throw but it didn't");
8+
promiseSuccesseded = true;
89
} catch (revert) {
910
if (!Array.isArray(args)) {
1011
expect.fail('Expected 3rd array parameter for error arguments');
@@ -38,6 +39,9 @@ async function expectRevertCustomError(promise, expectedErrorName, args) {
3839
// Assert argument equality
3940
expect(sanitizedActual).to.have.members(sanitizedExpected, `Unexpected ${errorName} arguments`);
4041
}
42+
if (promiseSuccesseded) {
43+
expect.fail("Expected promise to throw but it didn't");
44+
}
4145
}
4246

4347
module.exports = {

0 commit comments

Comments
 (0)