sc-forks/solidity-coverage

Tests Fail when using `hardhat-viem` plugin

zdenham opened this issue · 5 comments

zdenham commented

Workaround: (maintainer edit for visibility)

As of 0.8.12, viem should work with coverage if you set an environment variable before the command as below:

SOLIDITY_COVERAGE=true npx hardhat coverage

----- Original Issue -----

Using hardhat viem:

const token = await hre.viem.getContractAt(
    'MyContract',
    contractAddress,
    {
        walletClient,
    }
  );

Overriding the wallet client to test calling a contract from a different EOA doesn't seem to work properly (only when using solidity coverage)

await token.write.mint(); // does not call with the context of the `walletClient`
zdenham commented

Here is minimal reproducible example:

https://github.com/zdenham/hardhat-testy-test

cgewecke commented

@zdenham Yes, sorry this is known issue with the viem plugin that's being tracked at hardhat in PR 4961 comment.

  • 👀1
zdenham commented

Got it--appreciate the response @cgewecke. Will keep up to date there

joelamouche commented

I have this problem too. Please keep me posted 🙏

cgewecke commented

@zdenham, @joelamouche A workaround for this problem has been added in the latest version (0.8.12) via #883

Everything should work as expected if you set a SOLIDITY_COVERAGE env variable to true before running the task, e.g"

SOLIDITY_COVERAGE=true npx hardhat coverage

(This allows the plugin the configure the network correctly before the hardhat-viem plugin hijacks the provider).

Have edited the original comment with this workaround for visibility and am pinning this issue. Please just lmk if this fix does not work for some reason.

  • 👍1