Revert reason not supplied on revert for delegated calls.

[sebasgoldberg]

Describe the bug
Revert reason not supplied on revert for delegated calls.

Reproduction steps

The following test fails. The revert happens but no reason it is returned:

const aMock: FakeContract<SomeContract> = await smock.fake('SomeContract');
aMock.aFunction.reverts("A REASON")
...
await expect(otherContract.aFunctionThatExecuteDelegateCall( // delegates to aMock.aFunction
    aMock,
), "Should reverts with the mock programed reason")
    .to.rejectedWith("A REASON")

If I execute the same test without using the smock library, but instead using a mock contract that reverts, compiled from solidity code, the test passes.

Expected behavior
In the above test otherContract.aFunctionThatExecuteDelegateCall should revert using reason "A REASON".

Screenshots
NA

System Specs:

• Ubuntu Server 20.04:
• Package Version (or commit hash): 2.2.0

Additional context
For the contract delegator solidity code I am using functionDelegateCall from Openzeppelin's Address lib:

https://github.com/OpenZeppelin/openzeppelin-contracts/blob/4b16e887474eebb64e3f8266c68ef151a3f1b769/contracts/utils/Address.sol#L170

[sirnicolaz]

Is anyone planning to fix this or provide a workaround? The functionality is pretty much useless if the reason is not forwarded back.

[kalloc]

I see your issue may be connected with my one #152