gossamer icon indicating copy to clipboard operation
gossamer copied to clipboard

fix(core): handle latest extrinsic version and fix handleTxnPool logic

Open jimjbrettj opened this issue 3 years ago • 1 comments

Changes

  • Change how we construct extrinsics to support latest runtime version
  • Add gssmrLatestTxnQueue runtime to run integration tests with the latest runtime for maintaining the transaction pool (to be removed at a later date)
  • Upgrade the centrifuge library to v4
  • Build decoding logic to decode and handle data returned from validate transaction runtime call

Tests

go test -tags integration github.com/ChainSafe/gossamer
go test ./dot/core -tags integration 
go test ./dot/core
go test ./dot/rpc/modules

Issues

#2462

Primary Reviewer

@timwu20

jimjbrettj avatar May 17 '22 19:05 jimjbrettj

Codecov Report

Merging #2562 (9d09dac) into development (75fcd75) will decrease coverage by 0.23%. The diff coverage is 53.81%.

:exclamation: Current head 9d09dac differs from pull request most recent head 956b97c. Consider uploading reports for the commit 956b97c to get more accurate results

Additional details and impacted files
@@               Coverage Diff               @@
##           development    #2562      +/-   ##
===============================================
- Coverage        62.92%   62.69%   -0.24%     
===============================================
  Files              212      216       +4     
  Lines            27019    27200     +181     
===============================================
+ Hits             17003    17052      +49     
- Misses            8470     8588     +118     
- Partials          1546     1560      +14     

codecov[bot] avatar Jun 06 '22 19:06 codecov[bot]

Closed and replaced with #2809

jimjbrettj avatar Sep 27 '22 17:09 jimjbrettj