Document that `vm.expectRevert` should not be used for low-level calls

Open PaulRBerg opened this issue 3 years ago • 1 comments

As discussed in https://github.com/foundry-rs/foundry/issues/1124, and https://github.com/foundry-rs/foundry/issues/3901#issuecomment-1356501651.

Feb 17 '23 12:02 PaulRBerg

As @mds1 said in this comment:

If so, then maybe the best path forward here is leaving as-is and updating the foundry book to mention you must assert on the return status for expectRevert work as expected for low-level calls

Mar 17 '23 13:03 PaulRBerg