🧙 Show error when reverts on expect(...).not.to.be.reverted (#587)

Co-authored-by: Przemyslaw Rzad <roopert7@gmail.com>
TrueFiEng · Mar 25, 2022 · 7992daf · 7992daf
1 parent d1af8bb
commit 7992daf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/waffle-chai/src/matchers/reverted.ts b/waffle-chai/src/matchers/reverted.ts
@@ -9,7 +9,7 @@ export function supportReverted(Assertion: Chai.AssertionStatic) {
       this.assert(
         isReverted || isThrown || isError,
         `Expected transaction to be reverted, but other exception was thrown: ${error}`,
-        'Expected transaction NOT to be reverted',
+        `Expected transaction NOT to be reverted, but it was reverted with "${message}"`,
         'Transaction reverted.',
         error
       );

diff --git a/waffle-chai/test/matchers/reverted.test.ts b/waffle-chai/test/matchers/reverted.test.ts
@@ -41,7 +41,8 @@ describe('INTEGRATION: Matchers: reverted', () => {
   it('Not to revert: fail', async () => {
     await expect(
       expect(matchers.doThrow()).not.to.be.reverted
-    ).to.be.eventually.rejected;
+      // eslint-disable-next-line max-len
+    ).to.be.eventually.rejectedWith('Expected transaction NOT to be reverted, but it was reverted with "VM Exception while processing transaction: invalid opcode"');
   });
 
   it('Revert: fail, random exception', async () => {