to.be.reverted to not work as expected in your example

[InfectedIsm]

Hi, I've used your boilerplate for a personal project, but I found out that VM revert are not caught correctly.

Reproduce steps :

hardhat-boilerplate/test/Token.js

Lines 101 to 103 in 13bd712

	await expect(
	hardhatToken.connect(addr1).transfer(owner.address, 1)
	).to.be.revertedWith("Not enough tokens");

Remove to.be.revertedWith :

      await expect(
        hardhatToken.connect(addr1).transfer(owner.address, 1)
      );

Test still passes :
✔ Should fail if sender doesn't have enough tokens

Any idea why ? I found your boilerplate code really useful, too bad I cannot use it

Thanks

[fvictorio]

If you do await expect(...something...), the promise returned inside the expect will be captured by Chai. If you then don't assert anything on that promise, then the result will be discarded and the test will pass.

To check that the transaction indeed fails, remove the expect too:

await hardhatToken.connect(addr1).transfer(owner.address, 1)

[InfectedIsm]

Thanks a lot for taking time to answer ! I had misunderstood its behavior