Skip to content
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

gen_runtime_vectors: Sync with ADR14 #1157

Merged
merged 2 commits into from
Oct 13, 2022
Merged

Conversation

matevz
Copy link
Member

@matevz matevz commented Oct 11, 2022

This PR syncs test vectors with the new additions to ADR14 PR:

  • adds Sapphire runtime ID for testnet
  • removes sig_context from Meta
  • adds invalid transaction with empty or too long Meta.runtime_id or Meta.chain_context.
  • adds invalid transaction with unknown call format
  • adds real-world ERC20 transfer call data input for evm.Call and for the encrypted transaction

@matevz matevz marked this pull request as ready for review October 11, 2022 15:04
@codecov
Copy link

codecov bot commented Oct 11, 2022

Codecov Report

Merging #1157 (3100413) into main (bc7cff4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1157   +/-   ##
=======================================
  Coverage   67.86%   67.86%           
=======================================
  Files         131      131           
  Lines       11615    11615           
=======================================
  Hits         7882     7882           
  Misses       3701     3701           
  Partials       32       32           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@matevz matevz force-pushed the matevz/bugfix/gen_runtime_vectors branch from a2bb9da to 3100413 Compare October 13, 2022 16:15
@matevz matevz enabled auto-merge October 13, 2022 16:15
@matevz matevz merged commit 3a537e9 into main Oct 13, 2022
@matevz matevz deleted the matevz/bugfix/gen_runtime_vectors branch October 13, 2022 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants