test: fix test case · defi-wonderland/prophet-modules@d9bee52

Triggered via push November 21, 2024 02:41

xorsal

Status Success

Total duration 2m 5s

Artifacts –

ci.yml

on: push

Matrix: Run Linters

12 warnings

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Run Linters (16.x): solidity/test/integration/ReleaseUnutilizedResponse.t.sol#L8

Explicitly mark visibility of state

Run Linters (16.x): solidity/test/integration/ReleaseUnutilizedResponse.t.sol#L9

Explicitly mark visibility of state

Run Linters (16.x): solidity/test/integration/ReleaseUnutilizedResponse.t.sol#L10

Explicitly mark visibility of state

Run Linters (16.x): solidity/test/mocks/MockArbitrator.sol#L4

imported name IAccessController is not used

Run Linters (16.x): solidity/test/unit/modules/resolution/PrivateERC20ResolutionModule.t.sol#L46

The order of data in contract BaseTest should be: Constants, Immutable variables, State Variables

Run Linters (16.x): solidity/contracts/examples/libraries/MerkleLib.sol#L17

'TREE_DEPTH' should start with _

Run Linters (16.x): solidity/contracts/examples/libraries/MerkleLib.sol#L18

'MAX_LEAVES' should start with _

Run Linters (16.x): solidity/contracts/examples/libraries/MerkleLib.sol#L26

'Z_0' should start with _

Run Linters (16.x): solidity/contracts/examples/libraries/MerkleLib.sol#L27

'Z_1' should start with _

Run Linters (16.x): solidity/contracts/examples/libraries/MerkleLib.sol#L28

'Z_2' should start with _

The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/