Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SRE-2252: Switch to fine-grained token with single scope #100

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Conversation

timm088
Copy link
Contributor

@timm088 timm088 commented Apr 18, 2024

Required for coverage report, previous token no longer valid

Test failure is expected and is being fixed in another PR

@timm088 timm088 changed the title SRE- Switch to fine-grained token with single scope SRE-2252: Switch to fine-grained token with single scope Apr 18, 2024
@platform-sa
Copy link

📃CI Report

Compiling 156 files with 0.8.19
Solc 0.8.19 finished in 14.21s
Compiler run �[33msuccessful with warnings:�[0m
�[1;33mWarning (9302)�[0m�[1;37m: Return value of low-level calls not used.�[0m
�[34m |�[0m
�[34m361 |�[0m �[33muser.call{value: amount}("")�[0m;
�[34m |�[0m �[1;33m^^^^^^^^^^^^^^^^^^^^^^^^^^^^�[0m

�[1;33mWarning (9302)�[0m�[1;37m: Return value of low-level calls not used.�[0m
�[34m |�[0m
�[34m174 |�[0m �[33muser.call{value: amount}("")�[0m;
�[34m |�[0m �[1;33m^^^^^^^^^^^^^^^^^^^^^^^^^^^^�[0m

Analysing contracts...
Running tests...

File % Lines % Statements % Branches % Funcs
src/child/ChildAxelarBridgeAdaptor.sol 100.00% (54/54) 100.00% (72/72) 100.00% (26/26) 100.00% (9/9)
src/child/ChildERC20.sol 100.00% (15/15) 100.00% (16/16) 100.00% (4/4) 100.00% (8/8)
src/child/ChildERC20Bridge.sol 99.26% (134/135) 99.45% (182/183) 98.53% (67/68) 100.00% (23/23)
src/child/WIMX.sol 100.00% (19/19) 100.00% (22/22) 100.00% (8/8) 100.00% (6/6)
src/common/AdaptorRoles.sol 100.00% (6/6) 100.00% (6/6) 100.00% (0/0) 100.00% (6/6)
src/common/BridgeRoles.sol 100.00% (8/8) 100.00% (8/8) 100.00% (0/0) 100.00% (8/8)
src/lib/EIP712MetaTransaction.sol 8.00% (2/25) 9.68% (3/31) 8.33% (1/12) 14.29% (1/7)
src/lib/EIP712Upgradeable.sol 73.33% (11/15) 60.87% (14/23) 0.00% (0/2) 50.00% (2/4)
src/lib/WETH.sol 89.47% (17/19) 86.36% (19/22) 87.50% (7/8) 66.67% (4/6)
src/root/RootAxelarBridgeAdaptor.sol 100.00% (54/54) 100.00% (72/72) 100.00% (26/26) 100.00% (9/9)
src/root/RootERC20Bridge.sol 74.48% (108/145) 80.29% (167/208) 57.58% (38/66) 73.08% (19/26)
src/root/flowrate/FlowRateDetection.sol 100.00% (30/30) 100.00% (33/33) 100.00% (14/14) 100.00% (4/4)
src/root/flowrate/FlowRateWithdrawalQueue.sol 100.00% (46/46) 100.00% (60/60) 78.57% (11/14) 100.00% (7/7)
src/root/flowrate/RootERC20BridgeFlowRate.sol 0.00% (0/42) 0.00% (0/49) 0.00% (0/10) 0.00% (0/10)

For a full HTML report run: forge coverage --report lcov && genhtml --ignore-errors category --branch-coverage --output-dir coverage lcov.info

@timm088 timm088 requested review from a team and dhf22 April 18, 2024 06:40
@timm088 timm088 merged commit 32d05b6 into main Apr 18, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants