Skip to content

Refactor registry 2.3 foundry tests (#12499) #57915

Refactor registry 2.3 foundry tests (#12499)

Refactor registry 2.3 foundry tests (#12499) #57915

Triggered via push March 19, 2024 22:57
Status Success
Total duration 5m 19s
Artifacts

solidity.yml

on: push
Detect changes
9s
Detect changes
Prepublish Test
1m 31s
Prepublish Test
Native Compilation
4m 53s
Native Compilation
Solidity Lint
39s
Solidity Lint
Prettier Formatting
50s
Prettier Formatting
Fit to window
Zoom out
Zoom in

Annotations

1 error and 20 warnings
Solidity Lint
Got error Error: No job for job name: "Lint" found during self-lookup, invalid job name given? Available jobs names + ids: Detect changes|22859126540,Prepublish Test|22859133111,Native Compilation|22859133225,Solidity Lint|22859133353,Prettier Formatting|22859133467 , failing action
Solidity Lint: contracts/test/v0.8/Cron.test.ts#L43
'libType' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/KeeperRegistrar.test.ts#L37
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/VRFSubscriptionBalanceMonitor.test.ts#L282
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/AutomationRegistry2_3.test.ts#L4421
'longBytes' is assigned a value but never used. Allowed unused vars must match /^_/u
Solidity Lint: contracts/test/v0.8/automation/ERC20BalanceMonitor.test.ts#L276
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/EthBalanceMonitor.test.ts#L267
'tx' is never reassigned. Use 'const' instead
Solidity Lint: contracts/test/v0.8/automation/KeeperRegistrar2_0.test.ts#L40
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/automation/KeeperRegistrar2_0.test.ts#L42
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/automation/KeeperRegistrar2_0.test.ts#L46
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint: contracts/test/v0.8/automation/KeeperRegistry1_2.test.ts#L77
Use "@ts-expect-error" instead of "@ts-ignore", as "@ts-ignore" will do nothing if the following line is error-free
Solidity Lint
Fallback should be external and payable to accept native currency
Solidity Lint
Immutable variable LINK is not prefixed with 'i_'
Solidity Lint
Explicitly mark visibility of state
Solidity Lint
Contract has 27 states declarations but allowed no more than 15
Solidity Lint
Avoid to use tx.origin
Solidity Lint
Avoid to use low level calls
Solidity Lint
Avoid to use low level calls
Solidity Lint
global import of path ../UpkeepFormat.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
global import of path ../UpkeepFormat.sol is not allowed. Specify names to import individually or bind all exports of the module into a name (import "path" as Name)
Solidity Lint
Fallback should be external and payable to accept native currency