-
Notifications
You must be signed in to change notification settings - Fork 25
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
test(integration): setup #6
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please write some comment on testing codes. hard to figure out whats your purpose is. @byeongsu-hong
integration-test/tests/setup/mod.rs
Outdated
#[tokio::test] | ||
async fn test_setup() { | ||
let test_env = setup_env().await.unwrap(); | ||
|
||
println!("{:?}", test_env.cw_deployments.addrs); | ||
println!("{:?}", test_env.cw_deployments.codes); | ||
println!("{:?}", test_env.evm_deployments); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove stdio macros in test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just added this test to check the address of deployments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and please make ci pass. @byeongsu-hong
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good To Me
Codecov Report
@@ Coverage Diff @@
## main #6 +/- ##
=======================================
Coverage ? 56.92%
=======================================
Files ? 39
Lines ? 1393
Branches ? 0
=======================================
Hits ? 793
Misses ? 600
Partials ? 0 |
* Add pausable ism with tests * Fix paused query error case * Run CI against all PRs * Add pausable ISM to README * Build wasm * Fix scripts * Allow threshold == set size and add tests * Upload artifacts * Force * Move into makefile * Install rename * Rename properly * Update test.yaml * Fix renaming * Fix makefile indentation * Force cargo install * simple fee hook (#6) * Implement simple fee hook * Address pr comments * Fix unit tests * Make set fee only owner * Implement remaining unit tests * Fix merkle integration test use --------- Co-authored-by: nambrot <nambrot@googlemail.com> Co-authored-by: ByeongSu Hong <hong@byeongsu.dev> * Add mailbox unit tests for post dispatch (#7) * Add mailbox unit tests for post dispatch * Add test for different denoms --------- Co-authored-by: nambrot <nambrot@googlemail.com> Co-authored-by: ByeongSu Hong <hong@byeongsu.dev>
resolves MITO-18
resolves MITO-110
hpl-interface/types/lib.rs