Hi guys,
I check your code on plugin.util.js, function assembleFiles(), you just filter out .sol files, however, there are other kinds of contracts, like .vy, which truffle compile handles correctly. I think it will be good to become more compatible.
+1
even though we are not instrumenting vyper but in cases our solidity files have a dependency on vyper contracts, it would be nice if they can be handled correctly.
failing scenario using truffle v5: have A.sol and B.vy under contracts, adding B.vy to skipped files but it's part of our testing dependency, then truffle run coverage won't compile the vyper file thus failing the test
@ninjaahhh, @FreshmanQ Thanks for opening.
Agree this is a bug. Will fix.
thanks for the quick response.
here's a related issue on truffle side which may affect the fix: https://github.com/trufflesuite/truffle/issues/3455
hope it will help
This should be fixed in 0.7.12.
Vyper contracts will be automatically skipped for coverage, but they should now be included in Truffle's compilation set correctly.
Most helpful comment
@ninjaahhh, @FreshmanQ Thanks for opening.
Agree this is a bug. Will fix.