Skip to content

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

@The-3D

Description

@The-3D
Collaborator

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 ethereum/solidity#10354.
Implement the fix and activate the code coverage for the protocol in the CI

Activity

changed the title [-]Add fix for test coverage[/-] [+]Add fix for test coverage and integrate coverage in the CI[/+] on May 31, 2021
linked a pull request that will close this issuefix: Add solidity-coverage for tests #136on May 31, 2021
JasoonS

JasoonS commented on Jul 3, 2021

@JasoonS

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 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

    Development

    Participants

    @JasoonS@kartojal@LHerskind@The-3D

    Issue actions

      Add fix for test coverage and integrate coverage in the CI · Issue #135 · aave/protocol-v2