-
Notifications
You must be signed in to change notification settings - Fork 411
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
Wasmvm 1.4 upgrade #1590
Wasmvm 1.4 upgrade #1590
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.
Some first thoughts
Codecov Report
@@ Coverage Diff @@
## main #1590 +/- ##
==========================================
+ Coverage 58.30% 58.48% +0.18%
==========================================
Files 64 64
Lines 8643 8688 +45
==========================================
+ Hits 5039 5081 +42
- Misses 3194 3195 +1
- Partials 410 412 +2
|
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
@@ -42,7 +43,7 @@ import ( | |||
//go:embed testdata/hackatom.wasm | |||
var hackatomWasm []byte | |||
|
|||
const AvailableCapabilities = "iterator,staking,stargate,cosmwasm_1_1" | |||
const AvailableCapabilities = "iterator,staking,stargate,cosmwasm_1_1,cosmwasm_1_2,cosmwasm_1_3,cosmwasm_1_4" |
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.
👍
} | ||
} | ||
|
||
func TestDistributionQuery(t *testing.T) { |
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.
very good tests!
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.
Nice work! LGTM 👍
This testdata based on CosmWasm#1590
Resolves #1586
Includes