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

WIP: port tests from Midas Protocol #17

Closed
wants to merge 34 commits into from

Conversation

zerosnacks
Copy link
Contributor

No description provided.

@zerosnacks zerosnacks changed the base branch from master to feature/add-fuse-plugin-4626 May 17, 2022 17:31
@zerosnacks
Copy link
Contributor Author

Test currently fails due to a difference in FusePoolDirectory's deployPool implementation between Midas and Fuse

@zerosnacks zerosnacks changed the title WIP: port tests from midas protocol WIP: port tests from Midas Protocol May 17, 2022
@zerosnacks
Copy link
Contributor Author

Temporarily blocked until #18 has been investigated

@zerosnacks
Copy link
Contributor Author

Temporarily blocked until #18 has been investigated

In the meantime I will continue to look into a temporary solution as the goal is to bring the codebase first in line 1:1 with what is running in production

@zerosnacks zerosnacks changed the base branch from feature/add-fuse-plugin-4626 to development May 18, 2022 20:07
.gitignore Outdated
@@ -65,6 +65,7 @@ xcuserdata/

# Solidity
/out/
/artifacts/
Copy link
Contributor Author

Choose a reason for hiding this comment

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

evaluate if this markes sense

@zerosnacks
Copy link
Contributor Author

zerosnacks commented May 20, 2022

Aligned with Sri - will look for a workaround but we will keep the issue #18 as it would be desirable to fix this at some point in the future.

No longer blocked as #19 now allows CI to do mainnet forks

@zerosnacks zerosnacks force-pushed the feature/port-tests-from-midas-protocol branch from 96a79e5 to 92d82f6 Compare June 1, 2022 22:28
@zerosnacks
Copy link
Contributor Author

Closing this PR, decided not to port tests 1:1 from Midas but rather take some of their core learnings and implement them separately.

@zerosnacks zerosnacks closed this Jun 7, 2022
@zerosnacks zerosnacks deleted the feature/port-tests-from-midas-protocol branch June 7, 2022 12:30
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.

1 participant