search

aave / protocol-v2

Aave Protocol V2
https://aave.com
Other
662 stars 754 forks source link

Add fix for test coverage and integrate coverage in the CI #135

Open The-3D opened 3 years ago

The-3D commented 3 years ago

It's currently impossible to run test coverage as the compilation of the protocol contracts currently fails without optimizations enabled. There is a proposed solution here https://github.com/ethereum/solidity/issues/10354. Implement the fix and activate the code coverage for the protocol in the CI

JasoonS commented 3 years ago

BTW, I got it fixed and working here: https://www.npmjs.com/package/@float-capital/solidity-coverage

Simply install the package and change the following in your hardhat.config.js:

-require("solidity-coverage");
+require("@float-capital/solidity-coverage");

Hopefully will be up-streamed soon :+1: