CCIP-4116 restructure tests #15165
CCIP-4116 restructure tests #15165
Conversation
contracts/src/v0.8/ccip/test/rmn/armProxy/ArmPorxy.setARM.t.sol
Outdated
Show resolved
Hide resolved
contracts/src/v0.8/ccip/test/rmn/armProxy/ARMProxyStandaloneTestSetup.t.sol
Outdated
Show resolved
Hide resolved
Solidity Review Jira issueHey! We have taken the liberty to link this PR to a Jira issue for Solidity Review. This is a new feature, that's currently in the pilot phase, so please make sure that the linkage is correct. In a contrary case, please update it manually in JIRA and replace Solidity Review issue key in the changeset file with the correct one. Any changes to the Solidity Review Jira issue should be reflected in the changeset file. If you need to update the issue key, please do so manually in the following changeset file: This PR has been linked to Solidity Review Jira issue: CCIP-3966 |
| ) public { | ||
| s_pingPong.setCounterpartAddress(counterpartAddress); | ||
|
|
||
| assertEq(s_pingPong.getCounterpartAddress(), counterpartAddress); |
There was a problem hiding this comment.
should be a separate file for each of these functions getCounterPartAddress(), getCounterpartChainSelector, and isPaused
contracts/src/v0.8/ccip/test/capability/CCIPHome/CCIPHome.constructor.t.sol
Outdated
Show resolved
Hide resolved
contracts/src/v0.8/ccip/test/capability/CCIPHome/CCIPHome.constructor.t.sol
Outdated
Show resolved
Hide resolved
…o CCIP-4116_Restructure-tests
| @@ -1,8 +1,8 @@ | |||
| // SPDX-License-Identifier: BUSL-1.1 | |||
| pragma solidity 0.8.24; | |||
|
|
|||
| import {FeeQuoter} from "../../FeeQuoter.sol"; | |||
There was a problem hiding this comment.
FeeQuoter is a top level contract so you don't need a feeQuoter/FeeQuoter folder, just a FeeQuoter one
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
Test restructuring