-
Notifications
You must be signed in to change notification settings - Fork 334
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: Fix polkadot-decoding smoke test #2972
Conversation
WASM runtime size check:Compared to target branchMoonbase runtime: 2188 KB (no changes) ✅ Moonbeam runtime: 2148 KB (no changes) ✅ Moonriver runtime: 2148 KB (no changes) ✅ Compared to latest release (runtime-3200)Moonbase runtime: 2188 KB (+228 KB compared to latest release) Moonbeam runtime: 2148 KB (+224 KB compared to latest release) Moonriver runtime: 2148 KB (+224 KB compared to latest release) |
Coverage Report@@ Coverage Diff @@
## master pablo/test/smoke/improve-decoding +/- ##
====================================================================
Coverage 79.23% 79.23% 0.00%
Files 299 299
Lines 87602 87602
====================================================================
Hits 69405 69405
Misses 18197 18197
|
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.
The changes are not correct
What does it do?
This PR adds a retry for query responses which do not contain entries. From some experimentation, it seems that for the
accounts::assets
data, random storage keys with high starting values such asf
seem to be out of bounds for the existing records and fail when trying to read the first entry of the response.There are also changes to improve reproducing failing cases, mainly to avoid running the whole test and just try to query the module using the provided key override.
What important points reviewers should know?
Is there something left for follow-up PRs?
What alternative implementations were considered?
Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?
What value does it bring to the blockchain users?