Skip to content
New issue

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

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

Already on GitHub? # to your account

Add test showing price lookup works even after removing a vault #403

Merged
merged 4 commits into from
Sep 2, 2024

Conversation

hieronx
Copy link
Contributor

@hieronx hieronx commented Aug 15, 2024

No description provided.

@hieronx hieronx enabled auto-merge (squash) September 2, 2024 13:20
@hieronx hieronx merged commit 49e9a7c into main Sep 2, 2024
5 checks passed
Copy link

github-actions bot commented Sep 2, 2024

Coverage after merging price-removing-vault into main will be

99.71%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   Auth.sol100%100%100%100%
   CentrifugeRouter.sol90.50%58.97%100%99.08%102, 117, 128, 128, 145, 189, 200, 294, 302, 307, 309, 323, 51, 58, 58, 58, 85
   ERC7540Vault.sol96.28%78.13%100%100%105, 133, 138, 145, 145, 250, 263
   Escrow.sol100%100%100%100%
   InvestmentManager.sol98.41%93.83%100%100%323, 354, 359, 488, 559
   PoolManager.sol99.01%98.17%96.43%100%216, 437
   Root.sol98.65%94.44%100%100%32
src/admin
   Guardian.sol92.59%100%100%85.71%65–66
src/factories
   ERC7540VaultFactory.sol100%100%100%100%
   TrancheFactory.sol100%100%100%100%
   TransferProxyFactory.sol100%100%100%100%
src/gateway
   GasService.sol100%100%100%100%
   Gateway.sol98.15%94.12%100%100%172, 176, 261–262, 76
src/gateway/adapters/axelar
   Adapter.sol100%100%100%100%
   Forwarder.sol100%100%100%100%
src/token
   ERC20.sol100%100%100%100%
   RestrictionManager.sol100%100%100%100%
   Tranche.sol92.98%70%94.44%100%108, 123, 97

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants