vm.expectRevert() does not necessarily bind to the primary function

[timefliez1210]

As an example the following lines:

vm.expectRevert();
stable.donate(sUSD.balanceOf(users[1]));

the stable.donate() function is indeed expected to revert, but since sUSD.balanceOf(users[1]) does succeed the expectRevert() will make the test fail, with [FAIL: next call did not revert as expected].

An easy workaround is though to simply rewrite the order:

uint256 susdToDonate = sUSD.balanceOf(users[1]);
vm.expectRevert();
stable.donate(susdToDonate);

I'm unsure if this is expected behavior, but I find it quite unintuitive.