SizeCredit / size-solidity / 15764294367

93%

Build Type

push

github

Committed by web-flow

Commit Message

Add comprehensive test coverage for v1.8 features and bug fixes (#179)

* WIP

- Add SizeFactory.{addVault,removeVault,isVault}
- Add minimal Vault (needs to change)
- Change Size.initialize
- Change Size.reinitialize
- Change setUserConfiguration
- Change DepositTokenLibrary
- Change deposit/withdraw

* WIP

* Remove liquidityIndexAtRepayment logic

* Add TODO notes

* VaultWrapper WIP

* WIP

* WIP

* Compiling

* First version of NonTransferrableTokenVault

* Update transferFrom function

* Add missing _withdrawFromAave

* Store PPS

* Rollback ppsAtRepayment

* ReinitializeV1_8 tests

* Update coverage script

* Add comments

* Fix deposit/withdraw ERC4626 implementation

* More tests

* Fix slither

* Add more tests

* Remove borrow cap?

* Add slither disable for unused-return

* Remove CapsLibrary

* Fix tests

* Remove caps test

* Remove borrowTokenCap from riskConfig

* Remove deprecated test

* WIP UserVaults tests

* Add test_UserVaults_malicious_vault

* Add more UserVaults tests

* Vault whitelist cannot be disabled

* Finish UserVaults unit tests

* Rename UserVault to Vault

* Update invariant tests to set vault

* Fix slither

* Update prepare_crytic

* Update Properties

* Add new property

* Fix broken compilation

* Fix prepare-crytic

* Add reproducer

* Reset shares to 0 on vault changes

* Update

* svUSDC.totalSupply() reverts

* Add totalSupply O(n)

* Fix totalSupply test

* Fix tests

* Fix tests

* Create ForkVaults test

* Create v1.8 script

* Create fork tests

* transferFrom should always transfer full

* Fix slither

* Add InsufficientTotalAssets expected error

* Update README

* Refactor NonTransferrableRebasingTokenVault with strategies

* Fix CI

* Add ForkVaults_aave test

* Add vaultDust

* Update test_NonTransferrableRebasingTokenVault_admin_removes_vault_after_deposits

* Add README

* v1.8 entrypoint

* Create test for "ideal flow"

* Fix slither

* WIP

* WIP

* WIP

* WIP

* WIP

* Ad ERC721 to CollectionsMa... (continued)

Run Details

334 of 352 branches covered (94.89%)

Branch coverage included in aggregate %.

7 of 9 new or added lines in 2 files covered. (77.78%)

1 existing line in 1 file now uncovered.

1757 of 1818 relevant lines covered (96.64%)

4387.3 hits per line