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

Hub.exec new structure (paying fees always) + removal of Hub.reap + Better pool shares on GS + Certora specs for Hub #89

Merged
merged 56 commits into from
Oct 3, 2022

Conversation

gbalabasquer
Copy link
Contributor

Recreating the PR to this time use the classic commit and merge as the rebase one did change the commits hashes.

gbalabasquer and others added 26 commits August 9, 2022 14:31
* Add Certora

* Add DISPATCHER(true)

* Add checks

* Fix

* Minor changes

* less restrictive spec

* Use real Vat, DaiJoin and Dai code with solc 0.5.12

* Use D3MTestPool and D3MTestPlan

* More checks and CI setup

* Move certora.yml to correct place

* Fix some asserts

* Add exec_exec rule and some minor changes in exec_normal

* More checks

* multi assert flag via conditional

* Improve checks and add comments

* Use staging and old certora-cli version until production gets fixed

* More checks

* Add exec_ilk_culled

* check dai in vow for normal path

* Add exec_vat_caged

* Add cage and cull

* Add uncull

* Add exit

* Add exit_revert

* Add cage_revert

* Remove unnecessary stuff of cage_revert

* Add cull_revert

* Add uncull_revert + some variables renaming

* Add rely, rely_revert, deny and deny_revert

* Add file_ilk_uint256 and file_ilk_uint256_revert

* Add file_ilk_address and file_ilk_address_revert

* Add ilk_getters

* Add exec_vat_caged_revert + minor changes

* Add exec_ilk_culled_revert and minor chanegs to exec_vat_caged_revert

* Use main staging + split CI runs in several

* Use matrix for running all the rules in CI

* Fix CI config file

* Certora CI only on pushes (for now)

* Add to ignore

* Add timeout flag

* Add exec_normal_revert (without checking if all the revert cases are covered)

* Stick to specific certora-cli-alpha-master version

* Verify inkAfter can't be higher than max_int256

* Add missing end lines
* More precise account for pools shares after shutdown + force unwinding if assets < ink

* No more dapp

* Minor changes

* Rename test

* Add exit tests

* Fix exit and exit_revert specs

* Fix exec_normal_revert expect with the new changes

* Improve exit tests

* Improving exec_normal spec

* Fix comment

* Enforce solc 0.8.14 in make test and use it for CI

* Fix test rounding error

* Remove unnecessary tests

* 1e18 to wad and aave rounding (#88)

* switch compound tests from 1e18 to WAD

* fix aave test rounding issues

* gonza fixing my assertEq dyslexia

Co-authored-by: Gonzalo Balabasquer <[email protected]>

Co-authored-by: Gonzalo Balabasquer <[email protected]>

Co-authored-by: Chris Smith <[email protected]>
@gbalabasquer gbalabasquer changed the title Remove reap and rebalance Hub.exec new structure (paying fees always) + removal of Hub.reap + Better pool shares on GS + Certora specs for Hub Oct 3, 2022
@gbalabasquer gbalabasquer merged commit 64dac3a into v2 Oct 3, 2022
@gbalabasquer gbalabasquer deleted the remove-reap-and-rebalance branch March 7, 2023 14:33
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