diff options
author | Alex Beregszaszi <alex@rtfs.hu> | 2018-07-12 07:43:19 +0800 |
---|---|---|
committer | chriseth <chris@ethereum.org> | 2018-07-16 21:33:20 +0800 |
commit | aa08460d94f0e3ac8e067f89786175fb5ebba73b (patch) | |
tree | 9a0f9b91f71886ab8648f2d82e04a7449c325fda /test/libsolidity | |
parent | 21e97da2949a421987b0c6ad75bd401ce1dad0ab (diff) | |
download | dexon-solidity-aa08460d94f0e3ac8e067f89786175fb5ebba73b.tar.gz dexon-solidity-aa08460d94f0e3ac8e067f89786175fb5ebba73b.tar.zst dexon-solidity-aa08460d94f0e3ac8e067f89786175fb5ebba73b.zip |
Replace throw with revert() in end-to-end tests
Diffstat (limited to 'test/libsolidity')
-rw-r--r-- | test/libsolidity/SolidityEndToEndTest.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/test/libsolidity/SolidityEndToEndTest.cpp b/test/libsolidity/SolidityEndToEndTest.cpp index be74c5ff..9317eedd 100644 --- a/test/libsolidity/SolidityEndToEndTest.cpp +++ b/test/libsolidity/SolidityEndToEndTest.cpp @@ -2095,7 +2095,7 @@ BOOST_AUTO_TEST_CASE(transfer_ether) contract C { function () external payable { - throw; + revert(); } } )"; @@ -7741,7 +7741,7 @@ BOOST_AUTO_TEST_CASE(simple_throw) if (x > 10) return x + 10; else - throw; + revert(); return 2; } } @@ -9603,7 +9603,7 @@ BOOST_AUTO_TEST_CASE(mutex) contract mutexed { bool locked; modifier protected { - if (locked) throw; + if (locked) revert(); locked = true; _; locked = false; @@ -9615,16 +9615,16 @@ BOOST_AUTO_TEST_CASE(mutex) function withdraw(uint amount) protected returns (uint) { // NOTE: It is very bad practice to write this function this way. // Please refer to the documentation of how to do this properly. - if (amount > shares) throw; - if (!msg.sender.call.value(amount)("")) throw; + if (amount > shares) revert(); + if (!msg.sender.call.value(amount)("")) revert(); shares -= amount; return shares; } function withdrawUnprotected(uint amount) public returns (uint) { // NOTE: It is very bad practice to write this function this way. // Please refer to the documentation of how to do this properly. - if (amount > shares) throw; - if (!msg.sender.call.value(amount)("")) throw; + if (amount > shares) revert(); + if (!msg.sender.call.value(amount)("")) revert(); shares -= amount; return shares; } @@ -11159,7 +11159,7 @@ BOOST_AUTO_TEST_CASE(inline_assembly_in_modifiers) a := 2 } if (a != 2) - throw; + revert(); _; } function f() m public returns (bool) { |