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

feat: update LidoLocator with wstEth contract address #164

Closed
wants to merge 24 commits into from

Conversation

tamtamchik
Copy link
Member

Related issue lidofinance/lido-dao#849

@tamtamchik tamtamchik changed the base branch from develop to repovation August 26, 2024 14:55
# Conflicts:
#	test/0.4.24/contracts/LidoLocator__MutableMock.sol
Copy link

github-actions bot commented Aug 26, 2024

badge

Hardhat Unit Tests Coverage Summary

Filename                                                       Stmts    Miss  Cover    Missing
-----------------------------------------------------------  -------  ------  -------  ---------
contracts/0.4.24/Lido.sol                                        212       0  100.00%
contracts/0.4.24/StETH.sol                                        72       0  100.00%
contracts/0.4.24/StETHPermit.sol                                  15       0  100.00%
contracts/0.4.24/lib/Packed64x4.sol                                5       0  100.00%
contracts/0.4.24/lib/SigningKeys.sol                              36       0  100.00%
contracts/0.4.24/lib/StakeLimitUtils.sol                          37       0  100.00%
contracts/0.4.24/nos/NodeOperatorsRegistry.sol                   485       0  100.00%
contracts/0.4.24/oracle/LegacyOracle.sol                          72       0  100.00%
contracts/0.4.24/utils/Pausable.sol                                9       0  100.00%
contracts/0.4.24/utils/Versioned.sol                               5       0  100.00%
contracts/0.6.12/WstETH.sol                                       17       0  100.00%
contracts/0.8.4/WithdrawalsManagerProxy.sol                       61       0  100.00%
contracts/0.8.9/BeaconChainDepositor.sol                          21       2  90.48%   48, 51
contracts/0.8.9/Burner.sol                                        71       0  100.00%
contracts/0.8.9/DepositSecurityModule.sol                        104       0  100.00%
contracts/0.8.9/EIP712StETH.sol                                   16       0  100.00%
contracts/0.8.9/LidoExecutionLayerRewardsVault.sol                16       0  100.00%
contracts/0.8.9/LidoLocator.sol                                   19       0  100.00%
contracts/0.8.9/OracleDaemonConfig.sol                            28       0  100.00%
contracts/0.8.9/StakingRouter.sol                                303       0  100.00%
contracts/0.8.9/WithdrawalQueue.sol                               88       0  100.00%
contracts/0.8.9/WithdrawalQueueBase.sol                          146       0  100.00%
contracts/0.8.9/WithdrawalQueueERC721.sol                         89       0  100.00%
contracts/0.8.9/WithdrawalVault.sol                               21       0  100.00%
contracts/0.8.9/lib/Math.sol                                       4       0  100.00%
contracts/0.8.9/lib/PositiveTokenRebaseLimiter.sol                22       0  100.00%
contracts/0.8.9/lib/UnstructuredRefStorage.sol                     2       0  100.00%
contracts/0.8.9/oracle/AccountingOracle.sol                      171       1  99.42%   723
contracts/0.8.9/oracle/BaseOracle.sol                             89       1  98.88%   397
contracts/0.8.9/oracle/HashConsensus.sol                         263       1  99.62%   1005
contracts/0.8.9/oracle/ValidatorsExitBusOracle.sol                91      91  0.00%    96-461
contracts/0.8.9/proxy/OssifiableProxy.sol                         17       0  100.00%
contracts/0.8.9/sanity_checks/OracleReportSanityChecker.sol      171       0  100.00%
contracts/0.8.9/utils/DummyEmptyContract.sol                       0       0  100.00%
contracts/0.8.9/utils/PausableUntil.sol                           31       0  100.00%
contracts/0.8.9/utils/Versioned.sol                               11       0  100.00%
contracts/0.8.9/utils/access/AccessControl.sol                    23       0  100.00%
contracts/0.8.9/utils/access/AccessControlEnumerable.sol           9       0  100.00%
TOTAL                                                           2852      96  96.63%

Diff against master

Filename                                                       Stmts    Miss  Cover
-----------------------------------------------------------  -------  ------  --------
contracts/0.8.9/LidoLocator.sol                                   +1       0  +100.00%
contracts/0.8.9/sanity_checks/OracleReportSanityChecker.sol      -54       0  +100.00%
TOTAL                                                            -53       0  -0.03%

Results for commit: d462d8a

Minimum allowed coverage is 0%

♻️ This comment has been updated with latest results

@tamtamchik tamtamchik marked this pull request as ready for review August 26, 2024 15:51
@tamtamchik tamtamchik requested a review from a team as a code owner August 26, 2024 15:51
@tamtamchik tamtamchik changed the base branch from repovation to develop August 27, 2024 16:55
@tamtamchik tamtamchik requested a review from a team as a code owner August 27, 2024 16:55
@tamtamchik tamtamchik marked this pull request as draft August 27, 2024 16:55
@tamtamchik tamtamchik added paused Work on this issue or pull request is temporarily halted. contracts Pull requests that add add or update contracts. labels Aug 30, 2024
@tamtamchik tamtamchik self-assigned this Sep 25, 2024
@tamtamchik
Copy link
Member Author

Closed. Ported to lidofinance/lido-dao#863

@tamtamchik tamtamchik closed this Oct 11, 2024
@tamtamchik tamtamchik deleted the feat/locator-update branch October 11, 2024 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contracts Pull requests that add add or update contracts. paused Work on this issue or pull request is temporarily halted.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant