Skip to content

Conversation

@TepNik
Copy link
Contributor

@TepNik TepNik commented Mar 4, 2025

No description provided.

@TepNik TepNik requested a review from sakulstra March 4, 2025 13:09
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@github-actions
Copy link

github-actions bot commented Mar 4, 2025

🌈 Test Results
Compiling 107 files with Solc 0.8.28
Solc 0.8.28 finished in 1.77s
Compiler run successful!

Ran 7 tests for test/Lido_GhoDirectMinter.t.sol:Lido_GHODirectMinter_Test
[PASS] test_mintAndSupply_council(uint256) (runs: 256, μ: 238017, ~: 237710)
[PASS] test_mintAndSupply_owner(uint256) (runs: 256, μ: 235700, ~: 235409)
[PASS] test_mintAndSupply_rando() (gas: 18197)
[PASS] test_transferExcessToTreasury() (gas: 850986)
Logs:
  Bound result 1000000000000000000000

[PASS] test_withdrawAndBurn_council(uint256,uint256) (runs: 256, μ: 308031, ~: 308753)
[PASS] test_withdrawAndBurn_owner(uint256,uint256) (runs: 256, μ: 304151, ~: 304498)
[PASS] test_withdrawAndBurn_rando() (gas: 18097)
Suite result: ok. 7 passed; 0 failed; 0 skipped; finished in 1.05s (2.14s CPU time)

Ran 1 test suite in 1.05s (1.05s CPU time): 7 tests passed, 0 failed, 0 skipped (7 total tests)

@github-actions
Copy link

github-actions bot commented Mar 4, 2025

🔮 Coverage report
File Line Coverage Function Coverage Branch Coverage
src/GhoDirectMinter.sol ${\color{green}100\%}$
$26 / 26$
${\color{green}100\%}$
$5 / 5$
${\color{red}50\%}$
$1 / 2$
src/proposals/LidoGHOListing.sol ${\color{green}100\%}$
$9 / 9$
${\color{green}100\%}$
$2 / 2$
${\color{green}100\%}$
$0 / 0$

@github-actions
Copy link

github-actions bot commented Mar 4, 2025

♻️ Forge Gas Snapshots

Seems like you are not measuring gas of any operations yet. 🤔
Consider adding some snapshot tests to measure regressions & improvements.

@sakulstra sakulstra merged commit d96c7a5 into main Mar 4, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants