NoBey/openzeppelin-contracts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #506 from TalAter/fix/deprecated-throw-in-tests

Browse Source 
Replace deprecated throw with assert in Solidity tests
This commit is contained in:
Facundo Spagnuolo
2017-11-21 14:41:31 -03:00
committed by GitHub
parent 99f3e26f83 5fc53b6f2f
commit db5a12cfdd
2 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
test/helpers/ReentrancyAttack.sol
View File

@ -3,9 +3,7 @@ pragma solidity ^0.4.11;
contract ReentrancyAttack { contract ReentrancyAttack {
function callSender(bytes4 data) { function callSender(bytes4 data) {
if(!msg.sender.call(data)) { require(msg.sender.call(data));
throw;
}
} }
} }

4
test/helpers/ReentrancyMock.sol
View File

@ -27,9 +27,7 @@ contract ReentrancyMock is ReentrancyGuard {
if(n > 0) { if(n > 0) {
count(); count();
bool result = this.call(func, n - 1); bool result = this.call(func, n - 1);
if(result != true) { require(result == true);
throw;
}
} }
} }