-
Notifications
You must be signed in to change notification settings - Fork 402
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
Better wasmvm 1.1 integration test #988
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.
LGTM. If the TestReflectTotalSupplyQuery
test starts failing when you comment out the cosmwasm_1_1
capability, then everything's great!
Codecov Report
@@ Coverage Diff @@
## main #988 +/- ##
==========================================
+ Coverage 59.25% 59.34% +0.09%
==========================================
Files 51 51
Lines 6246 6246
==========================================
+ Hits 3701 3707 +6
+ Misses 2279 2272 -7
- Partials 266 267 +1
|
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.
LGTM 👍
* Add cosmwasm_1_1 cability, bump reflect contract, supply query integration test * Review feedback
* Add cosmwasm_1_1 cability, bump reflect contract, supply query integration test * Review feedback
Relates to #903 and #980
Add an integration test to call the new supply query via reflect contract
Includes:
cosmwasm_1_1
capabiltiy