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.

Sign up for GitHub

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

Jump to bottom

Migration/holograph operator test #338

Open
wants to merge 34 commits into
base: develop
Choose a base branch
from
Open

Migration/holograph operator test #338

wants to merge 34 commits into from

Conversation

djaciel
Copy link
Contributor

@djaciel djaciel commented Jun 17, 2024 •
edited by fgregoribattista
Loading

Describe Changes

  • Migrate 14_holograph_operator_tests to the foundry tool.
    In addition, tests are refactored in common as well as new helpers are generated.

Checklist before requesting a review

  • I have performed a self-review of my code
  • Code styles have been enforced
  • All Foundry tests are passing

@djaciel djaciel marked this pull request as draft June 17, 2024 23:57
djaciel and others added 25 commits June 17, 2024 18:20
@djaciel
add constructor test
81192e5
@djaciel
add init() describe tests
6de67b9
@djaciel
add jobEstimator tests
0353842
@djaciel
add pod related tests
f918b40
@djaciel
add getPodOperators(pod) tests
1d02d44
@djaciel
add getPodOperators and getPodBondAmounts tests
cbe678b
@djaciel
add bondUtilityToken() tests
c6e62aa
@djaciel
topupUtilityToken()
41b4c6b
@djaciel
WIP unbondUtilityToken()
6296bcb
@fgregoribattista
add testSampleERC20
89a22e4
@djaciel
fix external tests
306b630
@djaciel
move getConfigSampleERC20 to CrossChain utils
9043f57
@djaciel
add getBondedAmount() tests
fe05bfb
@djaciel
add getBondedPod() tests
9a297b1
@fgregoribattista
unify helpers
0ea7ae0
@fgregoribattista
refact uses
59a3fb1
@djaciel
add crossChainMessage() tests
ffd7d8c
@djaciel
getJobDetails() wip
ccbc6e5
@fgregoribattista
add get and set. HFM-87
8fef771
@fgregoribattista
add private test functions - HFM-88
9224c39
@djaciel
add getPodOperatorsLength() and bond test operators and fix other tests
1693051
@djaciel
wip executeJob() tests
86da5e8
@djaciel
wip executeJob()
52921af
@fgregoribattista
Merge branch 'develop' of github.com:Think-and-Dev/holograph-protocol…
a8156b4 
… into migration/holographOperatorTest
@fgregoribattista
skip fail test
02b0589
@fgregoribattista fgregoribattista marked this pull request as ready for review July 1, 2024 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@djaciel @fgregoribattista