Skip to content

Add gas reporting job parallelization on CI#1305

Merged
eternauta1337 merged 5 commits intodevelopfrom
add/gas-reporting-parallelization
Jun 17, 2021
Merged

Add gas reporting job parallelization on CI#1305
eternauta1337 merged 5 commits intodevelopfrom
add/gas-reporting-parallelization

Conversation

@mjlescano
Copy link
Copy Markdown
Contributor

@mjlescano mjlescano commented May 31, 2021

The objective of this PR is to be able to generate gas reports on the parallelized unit tests job. To achieve that we have to be able to generate multiple gas reports and merge them later on a separate job.

Right now, gas reporting is done by 2 steps:

  1. Run npx hardhat test --gas, which will generate the file gasReporterOutput.json if running on CI
  2. Then npx codechecks codechecks.unit.yml, which will generate a test-gas-used.log and upload it to Codechecks

And, for implementing parallelization, this PR generates different gasReporterOutput-N.json files, and then merges them into a single file to be able to upload it to codechecks using the npx hardhat test:merge-gas-reports gasReporterOutput-*.json command.

Pendings

  • The merge of the gasReporterOutput.json files has different results compared when we generate it in a single run. This has to be fixed before merging this PR

Loading
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