update test descriptions

runmyself · Jun 20, 2023 · c10e866 · c10e866
1 parent f4303d5
commit c10e866
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 23 deletions.
diff --git a/packages/contracts-bedrock/.gas-snapshot b/packages/contracts-bedrock/.gas-snapshot
@@ -32,25 +32,25 @@ DisputeGameFactory_SetImplementation_Test:test_setImplementation_notOwner_revert
 DisputeGameFactory_SetImplementation_Test:test_setImplementation_succeeds() (gas: 44243)
 DisputeGameFactory_TransferOwnership_Test:test_transferOwnership_notOwner_reverts() (gas: 15950)
 DisputeGameFactory_TransferOwnership_Test:test_transferOwnership_succeeds() (gas: 18642)
-FaultDisputeGame_Test:test_clockTimeExceeded_reverts() (gas: 26474)
-FaultDisputeGame_Test:test_defendRoot_reverts() (gas: 13280)
-FaultDisputeGame_Test:test_duplicateClaim_reverts() (gas: 103381)
+FaultDisputeGame_Test:test_clockTimeExceeded_reverts() (gas: 26496)
+FaultDisputeGame_Test:test_defendRoot_reverts() (gas: 13236)
+FaultDisputeGame_Test:test_duplicateClaim_reverts() (gas: 103425)
 FaultDisputeGame_Test:test_extraData_succeeds() (gas: 17478)
 FaultDisputeGame_Test:test_gameData_succeeds() (gas: 17859)
-FaultDisputeGame_Test:test_gameDepthExceeded_reverts() (gas: 5907275)
+FaultDisputeGame_Test:test_gameDepthExceeded_reverts() (gas: 5907231)
 FaultDisputeGame_Test:test_gameStart_succeeds() (gas: 10337)
-FaultDisputeGame_Test:test_gameType_succeeds() (gas: 8194)
-FaultDisputeGame_Test:test_initialRootClaimData_succeeds() (gas: 17580)
-FaultDisputeGame_Test:test_moveAgainstNonexistentParent_reverts() (gas: 24609)
+FaultDisputeGame_Test:test_gameType_succeeds() (gas: 8259)
+FaultDisputeGame_Test:test_initialRootClaimData_succeeds() (gas: 17624)
+FaultDisputeGame_Test:test_moveAgainstNonexistentParent_reverts() (gas: 24632)
 FaultDisputeGame_Test:test_move_gameNotInProgress_reverts() (gas: 10945)
-FaultDisputeGame_Test:test_resolve_challenge_contested() (gas: 222404)
-FaultDisputeGame_Test:test_resolve_reverts() (gas: 27143)
-FaultDisputeGame_Test:test_resolve_root_contested() (gas: 107149)
-FaultDisputeGame_Test:test_resolve_root_uncontested() (gas: 24504)
-FaultDisputeGame_Test:test_resolve_team_deathmatch() (gas: 393619)
-FaultDisputeGame_Test:test_rootClaim_succeeds() (gas: 8191)
+FaultDisputeGame_Test:test_resolve_challengeContested() (gas: 222383)
+FaultDisputeGame_Test:test_resolve_reverts() (gas: 27121)
+FaultDisputeGame_Test:test_resolve_rootContested() (gas: 107225)
+FaultDisputeGame_Test:test_resolve_rootUncontested() (gas: 24459)
+FaultDisputeGame_Test:test_resolve_teamDeathmatch() (gas: 393609)
+FaultDisputeGame_Test:test_rootClaim_succeeds() (gas: 8169)
 FaultDisputeGame_Test:test_simpleAttack_succeeds() (gas: 107389)
-FaultDisputeGame_Test:test_version_succeeds() (gas: 9802)
+FaultDisputeGame_Test:test_version_succeeds() (gas: 9780)
 FeeVault_Test:test_constructor_succeeds() (gas: 18185)
 GasBenchMark_L1CrossDomainMessenger:test_sendMessage_benchmark_0() (gas: 352113)
 GasBenchMark_L1CrossDomainMessenger:test_sendMessage_benchmark_1() (gas: 2950320)

diff --git a/packages/contracts-bedrock/contracts/test/FaultDisputeGame.t.sol b/packages/contracts-bedrock/contracts/test/FaultDisputeGame.t.sol
@@ -270,16 +270,16 @@ contract FaultDisputeGame_Test is DisputeGameFactory_Init {
     }
 
     /**
-     * @dev Static unit test for the correctness of resolution.
+     * @dev Static unit test for the correctness an uncontested root resolution.
      */
-    function test_resolve_root_uncontested() public {
+    function test_resolve_rootUncontested() public {
         GameStatus status = gameProxy.resolve();
         assertEq(uint8(status), uint8(GameStatus.DEFENDER_WINS));
         assertEq(uint8(gameProxy.status()), uint8(GameStatus.DEFENDER_WINS));
     }
 
     /**
-     * @dev Static unit test for the correctness of resolution.
+     * @dev Static unit test asserting that resolve reverts when the game is not in progress.
      */
     function test_resolve_reverts() public {
         gameProxy.resolve();
@@ -288,9 +288,9 @@ contract FaultDisputeGame_Test is DisputeGameFactory_Init {
     }
 
     /**
-     * @dev Static unit test for the correctness of resolution.
+     * @dev Static unit test for the correctness of resolving a single attack game state.
      */
-    function test_resolve_root_contested() public {
+    function test_resolve_rootContested() public {
         gameProxy.attack(0, Claim.wrap(bytes32(uint256(5))));
 
         GameStatus status = gameProxy.resolve();
@@ -299,9 +299,9 @@ contract FaultDisputeGame_Test is DisputeGameFactory_Init {
     }
 
     /**
-     * @dev Static unit test for the correctness of resolution.
+     * @dev Static unit test for the correctness of resolving a game with a contested challenge claim.
      */
-    function test_resolve_challenge_contested() public {
+    function test_resolve_challengeContested() public {
         gameProxy.attack(0, Claim.wrap(bytes32(uint256(5))));
         gameProxy.defend(1, Claim.wrap(bytes32(uint256(6))));
 
@@ -311,9 +311,9 @@ contract FaultDisputeGame_Test is DisputeGameFactory_Init {
     }
 
     /**
-     * @dev Static unit test for the correctness of resolution.
+     * @dev Static unit test for the correctness of resolving a game with multiplayer moves.
      */
-    function test_resolve_team_deathmatch() public {
+    function test_resolve_teamDeathmatch() 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))));