From ca63592985c489d791bb56212fc2850410c50715 Mon Sep 17 00:00:00 2001 From: 0xsuryansh Date: Mon, 11 Nov 2024 14:35:45 +0530 Subject: [PATCH] split PingPong.plumbing.t.sol --- contracts/gas-snapshots/ccip.gas-snapshot | 4 +- .../PingPong/PingPong.plumbing.t.sol | 51 ------------------- .../PingPong/PingPong.setCounterpart.t.sol | 15 ++++++ .../PingPong.setCounterpartAddress.t.sol | 14 +++++ ...PingPong.setCounterpartChainSelector.t.sol | 14 +++++ .../PingPong.setOutOfOrderExecution.t.sol | 18 +++++++ .../PingPong/PingPong.setPaused.t.sol | 14 +++++ 7 files changed, 77 insertions(+), 53 deletions(-) delete mode 100644 contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.plumbing.t.sol create mode 100644 contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpart.t.sol create mode 100644 contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartAddress.t.sol create mode 100644 contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartChainSelector.t.sol create mode 100644 contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setOutOfOrderExecution.t.sol create mode 100644 contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setPaused.t.sol diff --git a/contracts/gas-snapshots/ccip.gas-snapshot b/contracts/gas-snapshots/ccip.gas-snapshot index 14fd849b650..18ea50101d4 100644 --- a/contracts/gas-snapshots/ccip.gas-snapshot +++ b/contracts/gas-snapshots/ccip.gas-snapshot @@ -554,8 +554,8 @@ OnRamp_setDynamicConfig:test_setDynamicConfig_InvalidConfigReentrancyGuardEntere OnRamp_setDynamicConfig:test_setDynamicConfig_Success() (gas: 56440) OnRamp_withdrawFeeTokens:test_WithdrawFeeTokens_Success() (gas: 125867) PingPong_ccipReceive:test_CcipReceive_Success() (gas: 172841) -PingPong_plumbing:test_OutOfOrderExecution_Success() (gas: 20283) -PingPong_plumbing:test_Pausing_Success() (gas: 17738) +PingPong_setOutOfOrderExecution:test_OutOfOrderExecution_Success() (gas: 20283) +PingPong_setPaused:test_Pausing_Success() (gas: 17738) PingPong_startPingPong:test_StartPingPong_With_OOO_Success() (gas: 151954) PingPong_startPingPong:test_StartPingPong_With_Sequenced_Ordered_Success() (gas: 177569) RMNHome_getConfigDigests:test_getConfigDigests_success() (gas: 1079685) diff --git a/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.plumbing.t.sol b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.plumbing.t.sol deleted file mode 100644 index e1a227aeb57..00000000000 --- a/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.plumbing.t.sol +++ /dev/null @@ -1,51 +0,0 @@ -// SPDX-License-Identifier: BUSL-1.1 -pragma solidity 0.8.24; - -import {PingPongDemo} from "../../../applications/PingPongDemo.sol"; -import {PingPongDappSetup} from "./PingPongDappSetup.t.sol"; - -contract PingPong_plumbing is PingPongDappSetup { - function test_Fuzz_CounterPartChainSelector_Success( - uint64 chainSelector - ) public { - s_pingPong.setCounterpartChainSelector(chainSelector); - - assertEq(s_pingPong.getCounterpartChainSelector(), chainSelector); - } - - function test_Fuzz_CounterPartAddress_Success( - address counterpartAddress - ) public { - s_pingPong.setCounterpartAddress(counterpartAddress); - - assertEq(s_pingPong.getCounterpartAddress(), counterpartAddress); - } - - function test_Fuzz_CounterPartAddress_Success(uint64 chainSelector, address counterpartAddress) public { - s_pingPong.setCounterpartChainSelector(chainSelector); - - s_pingPong.setCounterpart(chainSelector, counterpartAddress); - - assertEq(s_pingPong.getCounterpartAddress(), counterpartAddress); - assertEq(s_pingPong.getCounterpartChainSelector(), chainSelector); - } - - function test_Pausing_Success() public { - assertFalse(s_pingPong.isPaused()); - - s_pingPong.setPaused(true); - - assertTrue(s_pingPong.isPaused()); - } - - function test_OutOfOrderExecution_Success() public { - assertFalse(s_pingPong.getOutOfOrderExecution()); - - vm.expectEmit(); - emit PingPongDemo.OutOfOrderExecutionChange(true); - - s_pingPong.setOutOfOrderExecution(true); - - assertTrue(s_pingPong.getOutOfOrderExecution()); - } -} diff --git a/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpart.t.sol b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpart.t.sol new file mode 100644 index 00000000000..365ccf33e71 --- /dev/null +++ b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpart.t.sol @@ -0,0 +1,15 @@ +// SPDX-License-Identifier: BUSL-1.1 +pragma solidity 0.8.24; + +import {PingPongDappSetup} from "./PingPongDappSetup.t.sol"; + +contract PingPong_setCounterpart is PingPongDappSetup { + function test_Fuzz_CounterPartAddress_Success(uint64 chainSelector, address counterpartAddress) public { + s_pingPong.setCounterpartChainSelector(chainSelector); + + s_pingPong.setCounterpart(chainSelector, counterpartAddress); + + assertEq(s_pingPong.getCounterpartAddress(), counterpartAddress); + assertEq(s_pingPong.getCounterpartChainSelector(), chainSelector); + } +} diff --git a/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartAddress.t.sol b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartAddress.t.sol new file mode 100644 index 00000000000..6762d8aab85 --- /dev/null +++ b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartAddress.t.sol @@ -0,0 +1,14 @@ +// SPDX-License-Identifier: BUSL-1.1 +pragma solidity 0.8.24; + +import {PingPongDappSetup} from "./PingPongDappSetup.t.sol"; + +contract PingPong_setCounterpartAddress is PingPongDappSetup { + function test_Fuzz_CounterPartAddress_Success( + address counterpartAddress + ) public { + s_pingPong.setCounterpartAddress(counterpartAddress); + + assertEq(s_pingPong.getCounterpartAddress(), counterpartAddress); + } +} diff --git a/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartChainSelector.t.sol b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartChainSelector.t.sol new file mode 100644 index 00000000000..a6fea957753 --- /dev/null +++ b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setCounterpartChainSelector.t.sol @@ -0,0 +1,14 @@ +// SPDX-License-Identifier: BUSL-1.1 +pragma solidity 0.8.24; + +import {PingPongDappSetup} from "./PingPongDappSetup.t.sol"; + +contract PingPong_setCounterpartChainSelector is PingPongDappSetup { + function test_Fuzz_CounterPartChainSelector_Success( + uint64 chainSelector + ) public { + s_pingPong.setCounterpartChainSelector(chainSelector); + + assertEq(s_pingPong.getCounterpartChainSelector(), chainSelector); + } +} diff --git a/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setOutOfOrderExecution.t.sol b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setOutOfOrderExecution.t.sol new file mode 100644 index 00000000000..0e09e67f7cb --- /dev/null +++ b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setOutOfOrderExecution.t.sol @@ -0,0 +1,18 @@ +// SPDX-License-Identifier: BUSL-1.1 +pragma solidity 0.8.24; + +import {PingPongDemo} from "../../../applications/PingPongDemo.sol"; +import {PingPongDappSetup} from "./PingPongDappSetup.t.sol"; + +contract PingPong_setOutOfOrderExecution is PingPongDappSetup { + function test_OutOfOrderExecution_Success() public { + assertFalse(s_pingPong.getOutOfOrderExecution()); + + vm.expectEmit(); + emit PingPongDemo.OutOfOrderExecutionChange(true); + + s_pingPong.setOutOfOrderExecution(true); + + assertTrue(s_pingPong.getOutOfOrderExecution()); + } +} diff --git a/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setPaused.t.sol b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setPaused.t.sol new file mode 100644 index 00000000000..82cd22199ec --- /dev/null +++ b/contracts/src/v0.8/ccip/test/applications/PingPong/PingPong.setPaused.t.sol @@ -0,0 +1,14 @@ +// SPDX-License-Identifier: BUSL-1.1 +pragma solidity 0.8.24; + +import {PingPongDappSetup} from "./PingPongDappSetup.t.sol"; + +contract PingPong_setPaused is PingPongDappSetup { + function test_Pausing_Success() public { + assertFalse(s_pingPong.isPaused()); + + s_pingPong.setPaused(true); + + assertTrue(s_pingPong.isPaused()); + } +}