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

Add E2E Tests #9

Merged
merged 14 commits into from
May 23, 2024
Merged

Add E2E Tests #9

merged 14 commits into from
May 23, 2024

Conversation

daopunk
Copy link
Member

@daopunk daopunk commented May 17, 2024

closes #5

E2E Tests

  • fuzz test saviour saveSafe
  • fuzz access control tests
  • liquidation tests
  • math tests

ODSaviour

  • changes to calculations in saveSafe

@daopunk daopunk requested a review from pi0neerpat May 20, 2024 17:11
@daopunk daopunk marked this pull request as ready for review May 20, 2024 17:18
Copy link
Member

@pi0neerpat pi0neerpat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix failing tests

  • test_algorithm
  • test_liquidateUnprotectedSafe

@daopunk daopunk requested a review from pi0neerpat May 22, 2024 00:07
src/contracts/ODSaviour.sol Outdated Show resolved Hide resolved
@pi0neerpat
Copy link
Member

Add logs to show math output during algorithm test

@daopunk
Copy link
Member Author

daopunk commented May 22, 2024

Add logs to show math output during algorithm test

logs added, but for more in-depth look, run
forge t --match-test test_liquidateProtectedSafe -vvvv

@daopunk daopunk merged commit 07e5917 into main May 23, 2024
2 checks passed
@daopunk daopunk deleted the fuzz-and-e2e-tests branch May 23, 2024 17:35
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.

Add e2e tests
2 participants