Skip to content

Commit a3256ec

Browse files
authored
Merge pull request #235 from argentlabs/maintenance/fix-tests-under-coverage
Test coverage fixes
2 parents 9481333 + 8c903b8 commit a3256ec

File tree

9 files changed

+102
-18200
lines changed

9 files changed

+102
-18200
lines changed

.solcover.js

+2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
module.exports = {
22
client: require('ganache-cli'),
3+
measureStatementCoverage: false,
4+
measureFunctionCoverage: false,
35
skipFiles: [
46
"../contracts-test",
57
"../lib_0.5",

lib_0.5/compound/CEther.sol

+1-1
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ contract CEther is CToken {
142142

143143
function doTransferOut(address payable to, uint amount) internal returns (Error) {
144144
/* Send the Ether, with minimal gas and revert on failure */
145-
to.transfer(amount);
145+
to.call.value(amount)("");
146146
return Error.NO_ERROR;
147147
}
148148

lib_0.5/maker/WETH9.sol

+1-1
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ contract WETH9 {
3838
function withdraw(uint wad) public {
3939
require(balanceOf[msg.sender] >= wad);
4040
balanceOf[msg.sender] -= wad;
41-
msg.sender.transfer(wad);
41+
msg.sender.call.value(wad)("");
4242
emit Withdrawal(msg.sender, wad);
4343
}
4444

lib_0.7/yearn/yVault.sol

+1-1
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ contract yVault is ERC20 {
156156
}
157157

158158
IWETH(address(token)).withdraw(r);
159-
payable(msg.sender).transfer(r);
159+
payable(msg.sender).call{value: r}("");
160160
}
161161

162162
function withdrawAllETH() external {

0 commit comments

Comments
 (0)