Strange revert error message

[ricktobacco]

The following error message happens with tests related to closeLoan, as well as mint and burn :

Transaction reverted without a reason and without a valid sourcemap provided by the compiler. Some line numbers may be off. We strongly recommend upgrading solc and always using revert reasons.

[RickGriff]

I am now getting strange revert error messages for all CDPManager tests in AccessControlTest.js after updating my editor compiler settings. Some revert without a reason, others with this:

Transaction reverted: function was called with incorrect parameters

The txs succeed when the access control is disabled, i.e. if the _require(...) is commented out. Otherwise, the txs revert (as they should), but with the wrong revert message.

It does seem like a compiler-related issue.

As a temporary fix I commented out the revert message assertions in bd7677c, so the CDPManager access control tests just check the tx receipt status.

[ricktobacco]

Now I'm strangely getting it on the protect_lusd_transfer branch @bingen

[bingen]

I’ve hit this one too. Very weird, with test “registerFrontEnd(): reverts if the kickback rate >1”.

• I thought it may be because buidler uses solc: 0.6.8, so I tried upgrading to hardhat, but no luck.
• I also tried downgrade contracts to 0.6.8, no luck.
• I tried with Node 14 instead of 12, no luck.

Just reporting here so we can discard possible origins.

[bingen]

I tried with waffle too, it didn’t work either. 😢

[bingen]

Reference: NomicFoundation/hardhat#1046