-
Notifications
You must be signed in to change notification settings - Fork 9
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: add integration test #216
test: add integration test #216
Conversation
…//github.com/icon-project/xCall into feat/add-response-message-rust
…/add-response-message-rust
…con-project/xCall into feat/add-response-message-rust
…con-project/xCall into feat/add-response-message-rust
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## development/177-xcall-future-proof-messaging #216 +/- ##
==================================================================================
- Coverage 69.52% 67.04% -2.49%
+ Complexity 64 62 -2
==================================================================================
Files 40 41 +1
Lines 2169 2403 +234
Branches 24 24
==================================================================================
+ Hits 1508 1611 +103
- Misses 651 781 +130
- Partials 10 11 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
…roject/xCall into test/add-integration-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.
Would it be possible to also add a test for the response during execution or is that to complicated?
contracts/cosmwasm-vm/cw-integration/tests/test_xcall_connection.rs
Outdated
Show resolved
Hide resolved
Co-authored-by: AntonAndell <[email protected]>
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
…/add-integration-test
970b7dd
into
development/177-xcall-future-proof-messaging
Description:
Commit Message
see the guidelines for commit messages.
Changelog Entry
Checklist: