Skip to content

Commit

Permalink
♻️ Update revertTo and snapshot Cheat Codes
Browse files Browse the repository at this point in the history 
Signed-off-by: Pascal Marco Caversaccio <pascal.caversaccio@hotmail.ch>
  • Loading branch information
@pcaversaccio
pcaversaccio committed Oct 1, 2024
1 parent 1ff03b4 commit 7aa2906
Show file tree
Hide file tree
Showing 6 changed files with 27 additions and 27 deletions.
2 changes: 1 addition & 1 deletion lib/create-util
Submodule create-util updated 2 files
+2 −2 package.json
+79 −79 pnpm-lock.yaml
2 changes: 1 addition & 1 deletion lib/openzeppelin-contracts
Submodule openzeppelin-contracts updated 6 files
+8 −0 .solcover.js
+70 −24 contracts/utils/cryptography/P256.sol
+6 −6 docs/modules/ROOT/pages/utilities.adoc
+1 −1 lib/forge-std
+2 −2 test/helpers/iterate.js
+13 −104 test/utils/cryptography/P256.t.sol
2 changes: 1 addition & 1 deletion lib/solady
Submodule solady updated 3 files
+1 −1 package.json
+15 −0 src/utils/LibSort.sol
+3 −0 test/LibSort.t.sol
2 changes: 1 addition & 1 deletion package.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
"@openzeppelin/merkle-tree": "^1.0.7",
"eslint": "^9.11.1",
"eslint-config-prettier": "^9.1.0",
"ethers": "^6.13.2",
"ethers": "^6.13.3",
"keccak256": "^1.0.6",
"merkletreejs": "^0.4.0",
"prettier": "^3.3.3",
Expand Down
10 changes: 5 additions & 5 deletions pnpm-lock.yaml
View file

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

36 changes: 18 additions & 18 deletions test/tokens/ERC721.t.sol
View file
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ contract ERC721Test is Test {
bool withData,
bytes memory data
) internal {
uint256 snapshot = vm.snapshot();
uint256 snapshot = vm.snapshotState();
vm.startPrank(owner);
vm.expectEmit(true, true, true, false);
emit IERC721.Transfer(owner, receiver, tokenId);
Expand Down Expand Up @@ -249,9 +249,9 @@ contract ERC721Test is Test {
}
_transferSuccess(owner, tokenId, receiver);
vm.stopPrank();
vm.revertTo(snapshot);
vm.revertToState(snapshot);

snapshot = vm.snapshot();
snapshot = vm.snapshotState();
vm.startPrank(approved);
vm.expectEmit(true, true, true, false);
emit IERC721.Transfer(owner, receiver, tokenId);
Expand Down Expand Up @@ -279,9 +279,9 @@ contract ERC721Test is Test {
}
_transferSuccess(owner, tokenId, receiver);
vm.stopPrank();
vm.revertTo(snapshot);
vm.revertToState(snapshot);

snapshot = vm.snapshot();
snapshot = vm.snapshotState();
vm.startPrank(operator);
vm.expectEmit(true, true, true, false);
emit IERC721.Transfer(owner, receiver, tokenId);
Expand Down Expand Up @@ -309,9 +309,9 @@ contract ERC721Test is Test {
}
_transferSuccess(owner, tokenId, receiver);
vm.stopPrank();
vm.revertTo(snapshot);
vm.revertToState(snapshot);

snapshot = vm.snapshot();
snapshot = vm.snapshotState();
vm.startPrank(owner);
ERC721Extended.approve(zeroAddress, tokenId);
vm.stopPrank();
Expand Down Expand Up @@ -342,9 +342,9 @@ contract ERC721Test is Test {
}
_transferSuccess(owner, tokenId, receiver);
vm.stopPrank();
vm.revertTo(snapshot);
vm.revertToState(snapshot);

snapshot = vm.snapshot();
snapshot = vm.snapshotState();
vm.startPrank(owner);
vm.expectEmit(true, true, true, false);
emit IERC721.Transfer(owner, owner, tokenId);
Expand All @@ -371,7 +371,7 @@ contract ERC721Test is Test {
assertEq(ERC721Extended.tokenOfOwnerByIndex(owner, 0), tokenId);
assertTrue(ERC721Extended.tokenOfOwnerByIndex(owner, 1) == tokenId + 1);
vm.stopPrank();
vm.revertTo(snapshot);
vm.revertToState(snapshot);

/**
* @dev Validates all possible reverts.
Expand Down Expand Up @@ -408,7 +408,7 @@ contract ERC721Test is Test {
address receiver,
bytes memory data
) internal {
uint256 snapshot = vm.snapshot();
uint256 snapshot = vm.snapshotState();
_shouldTransferTokensByUsers(
transferFunction,
owner,
Expand All @@ -419,9 +419,9 @@ contract ERC721Test is Test {
true,
data
);
vm.revertTo(snapshot);
vm.revertToState(snapshot);

snapshot = vm.snapshot();
snapshot = vm.snapshotState();
_shouldTransferTokensByUsers(
transferFunction,
owner,
Expand All @@ -432,9 +432,9 @@ contract ERC721Test is Test {
true,
data
);
vm.revertTo(snapshot);
vm.revertToState(snapshot);

snapshot = vm.snapshot();
snapshot = vm.snapshotState();
vm.startPrank(owner);
vm.expectEmit(true, true, true, true, receiver);
emit ERC721ReceiverMock.Received(owner, owner, tokenId, data);
Expand All @@ -450,9 +450,9 @@ contract ERC721Test is Test {
);
_transferSuccess(owner, tokenId, receiver);
vm.stopPrank();
vm.revertTo(snapshot);
vm.revertToState(snapshot);

snapshot = vm.snapshot();
snapshot = vm.snapshotState();
vm.startPrank(approved);
vm.expectEmit(true, true, true, true, receiver);
emit ERC721ReceiverMock.Received(approved, owner, tokenId, data);
Expand All @@ -468,7 +468,7 @@ contract ERC721Test is Test {
);
_transferSuccess(owner, tokenId, receiver);
vm.stopPrank();
vm.revertTo(snapshot);
vm.revertToState(snapshot);

vm.startPrank(owner);
vm.expectRevert(bytes("erc721: invalid token ID"));
Expand Down

0 comments on commit 7aa2906

Please sign in to comment.