diff --git a/packages/contracts-bedrock/.gas-snapshot b/packages/contracts-bedrock/.gas-snapshot index 79a26d75f9a96f225ef03241f9e8550f7b3ce685..2ac226c66758bc84cb157bf3676d0befb77b2da1 100644 --- a/packages/contracts-bedrock/.gas-snapshot +++ b/packages/contracts-bedrock/.gas-snapshot @@ -40,21 +40,21 @@ FaultDisputeGame_ResolvesCorrectly_IncorrectRoot3:test_resolvesCorrectly_succeed FaultDisputeGame_ResolvesCorrectly_IncorrectRoot:test_resolvesCorrectly_succeeds() (gas: 487044) FaultDisputeGame_Test:test_defendRoot_invalidMove_reverts() (gas: 13294) FaultDisputeGame_Test:test_extraData_succeeds() (gas: 17426) -FaultDisputeGame_Test:test_gameData_succeeds() (gas: 17873) +FaultDisputeGame_Test:test_gameData_succeeds() (gas: 17917) FaultDisputeGame_Test:test_gameStart_succeeds() (gas: 10315) -FaultDisputeGame_Test:test_gameType_succeeds() (gas: 8238) -FaultDisputeGame_Test:test_initialRootClaimData_succeeds() (gas: 17713) -FaultDisputeGame_Test:test_move_clockTimeExceeded_reverts() (gas: 26432) -FaultDisputeGame_Test:test_move_duplicateClaim_reverts() (gas: 103275) +FaultDisputeGame_Test:test_gameType_succeeds() (gas: 8282) +FaultDisputeGame_Test:test_initialRootClaimData_succeeds() (gas: 17691) +FaultDisputeGame_Test:test_move_clockTimeExceeded_reverts() (gas: 26387) +FaultDisputeGame_Test:test_move_duplicateClaim_reverts() (gas: 103230) FaultDisputeGame_Test:test_move_gameDepthExceeded_reverts() (gas: 408100) -FaultDisputeGame_Test:test_move_gameNotInProgress_reverts() (gas: 10968) -FaultDisputeGame_Test:test_move_nonExistentParent_reverts() (gas: 24698) -FaultDisputeGame_Test:test_resolve_challengeContested() (gas: 221135) -FaultDisputeGame_Test:test_resolve_notInProgress_reverts() (gas: 9635) -FaultDisputeGame_Test:test_resolve_rootContested() (gas: 106143) -FaultDisputeGame_Test:test_resolve_rootUncontested() (gas: 23625) -FaultDisputeGame_Test:test_resolve_teamDeathmatch() (gas: 391864) -FaultDisputeGame_Test:test_rootClaim_succeeds() (gas: 8159) +FaultDisputeGame_Test:test_move_gameNotInProgress_reverts() (gas: 10923) +FaultDisputeGame_Test:test_move_nonExistentParent_reverts() (gas: 24655) +FaultDisputeGame_Test:test_resolve_challengeContested_succeeds() (gas: 221201) +FaultDisputeGame_Test:test_resolve_notInProgress_reverts() (gas: 9679) +FaultDisputeGame_Test:test_resolve_rootContested_succeeds() (gas: 106068) +FaultDisputeGame_Test:test_resolve_rootUncontested_succeeds() (gas: 23692) +FaultDisputeGame_Test:test_resolve_teamDeathmatch_succeeds() (gas: 391875) +FaultDisputeGame_Test:test_rootClaim_succeeds() (gas: 8181) FaultDisputeGame_Test:test_simpleAttack_succeeds() (gas: 107388) FeeVault_Test:test_constructor_succeeds() (gas: 18185) GasBenchMark_L1CrossDomainMessenger:test_sendMessage_benchmark_0() (gas: 352135) diff --git a/packages/contracts-bedrock/contracts/test/FaultDisputeGame.t.sol b/packages/contracts-bedrock/contracts/test/FaultDisputeGame.t.sol index 4dd915101dfb3d9abb0d888822751bd683604e1f..b1eec84425d69c82772bbe69c9e52864241c48fb 100644 --- a/packages/contracts-bedrock/contracts/test/FaultDisputeGame.t.sol +++ b/packages/contracts-bedrock/contracts/test/FaultDisputeGame.t.sol @@ -236,7 +236,7 @@ contract FaultDisputeGame_Test is FaultDisputeGame_Init { } /// @dev Static unit test for the correctness an uncontested root resolution. - function test_resolve_rootUncontested() public { + function test_resolve_rootUncontested_succeeds() public { GameStatus status = gameProxy.resolve(); assertEq(uint8(status), uint8(GameStatus.DEFENDER_WINS)); assertEq(uint8(gameProxy.status()), uint8(GameStatus.DEFENDER_WINS)); @@ -260,7 +260,7 @@ contract FaultDisputeGame_Test is FaultDisputeGame_Init { } /// @dev Static unit test for the correctness of resolving a single attack game state. - function test_resolve_rootContested() public { + function test_resolve_rootContested_succeeds() public { gameProxy.attack(0, Claim.wrap(bytes32(uint256(5)))); GameStatus status = gameProxy.resolve(); @@ -269,7 +269,7 @@ contract FaultDisputeGame_Test is FaultDisputeGame_Init { } /// @dev Static unit test for the correctness of resolving a game with a contested challenge claim. - function test_resolve_challengeContested() public { + function test_resolve_challengeContested_succeeds() public { gameProxy.attack(0, Claim.wrap(bytes32(uint256(5)))); gameProxy.defend(1, Claim.wrap(bytes32(uint256(6)))); @@ -279,7 +279,7 @@ contract FaultDisputeGame_Test is FaultDisputeGame_Init { } /// @dev Static unit test for the correctness of resolving a game with multiplayer moves. - function test_resolve_teamDeathmatch() public { + function test_resolve_teamDeathmatch_succeeds() public { gameProxy.attack(0, Claim.wrap(bytes32(uint256(5)))); gameProxy.attack(0, Claim.wrap(bytes32(uint256(4)))); gameProxy.defend(1, Claim.wrap(bytes32(uint256(6))));