diff options
author | chriseth <chris@ethereum.org> | 2018-09-14 05:38:46 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-14 05:38:46 +0800 |
commit | 5cf570e100543415f425de7abb34893a0038ca34 (patch) | |
tree | 14121e9b87c608cbe648d2e2d842d6c5c7afb393 | |
parent | dc62e084cc0070f87c0252c53ea4fbbe35e1eccc (diff) | |
parent | 3e55aa3fa2ce0f7fa0d28fb6de1ef361d500181d (diff) | |
download | dexon-solidity-5cf570e100543415f425de7abb34893a0038ca34.tar dexon-solidity-5cf570e100543415f425de7abb34893a0038ca34.tar.gz dexon-solidity-5cf570e100543415f425de7abb34893a0038ca34.tar.bz2 dexon-solidity-5cf570e100543415f425de7abb34893a0038ca34.tar.lz dexon-solidity-5cf570e100543415f425de7abb34893a0038ca34.tar.xz dexon-solidity-5cf570e100543415f425de7abb34893a0038ca34.tar.zst dexon-solidity-5cf570e100543415f425de7abb34893a0038ca34.zip |
Merge pull request #4968 from ethereum/alethUpdateCompatible
Fix tests for newer versions of aleth.
-rw-r--r-- | test/ExecutionFramework.cpp | 4 | ||||
-rw-r--r-- | test/libsolidity/SolidityEndToEndTest.cpp | 49 |
2 files changed, 22 insertions, 31 deletions
diff --git a/test/ExecutionFramework.cpp b/test/ExecutionFramework.cpp index ea624735..3922d5e9 100644 --- a/test/ExecutionFramework.cpp +++ b/test/ExecutionFramework.cpp @@ -122,9 +122,9 @@ void ExecutionFramework::sendMessage(bytes const& _data, bool _isCreation, u256 if (!_isCreation) { d.to = dev::toString(m_contractAddress); - BOOST_REQUIRE(m_rpc.eth_getCode(d.to, "latest").size() > 2); + BOOST_REQUIRE(m_rpc.eth_getCode(d.to, "pending").size() > 2); // Use eth_call to get the output - m_output = fromHex(m_rpc.eth_call(d, "latest"), WhenError::Throw); + m_output = fromHex(m_rpc.eth_call(d, "pending"), WhenError::Throw); } string txHash = m_rpc.eth_sendTransaction(d); diff --git a/test/libsolidity/SolidityEndToEndTest.cpp b/test/libsolidity/SolidityEndToEndTest.cpp index 642c9929..f65c8b27 100644 --- a/test/libsolidity/SolidityEndToEndTest.cpp +++ b/test/libsolidity/SolidityEndToEndTest.cpp @@ -3039,26 +3039,26 @@ BOOST_AUTO_TEST_CASE(gasprice) BOOST_AUTO_TEST_CASE(blockhash) { - char const* sourceCode = R"( - contract C { - uint256 counter; - function g() public returns (bool) { counter++; return true; } - function f() public returns (bytes32[] memory r) { - r = new bytes32[](259); - for (uint i = 0; i < 259; i++) - r[i] = blockhash(block.number - 257 + i); - } - } - )"; - compileAndRun(sourceCode); - // generate a sufficient amount of blocks - while (blockNumber() < u256(255)) - ABI_CHECK(callContractFunction("g()"), encodeArgs(true)); - - vector<u256> hashes; - // currently the test only works for pre-constantinople + // depending on the aleth version, this test only works for pre-constantinople if (Options::get().evmVersion() < EVMVersion::constantinople()) { + char const* sourceCode = R"( + contract C { + uint256 counter; + function g() public returns (bool) { counter++; return true; } + function f() public returns (bytes32[] memory r) { + r = new bytes32[](259); + for (uint i = 0; i < 259; i++) + r[i] = blockhash(block.number - 257 + i); + } + } + )"; + compileAndRun(sourceCode); + // generate a sufficient amount of blocks + while (blockNumber() < u256(255)) + ABI_CHECK(callContractFunction("g()"), encodeArgs(true)); + + vector<u256> hashes; // ``blockhash()`` is only valid for the last 256 blocks, otherwise zero hashes.emplace_back(0); for (u256 i = blockNumber() - u256(255); i <= blockNumber(); i++) @@ -3067,18 +3067,9 @@ BOOST_AUTO_TEST_CASE(blockhash) hashes.emplace_back(0); // future block hashes are zero hashes.emplace_back(0); + + ABI_CHECK(callContractFunction("f()"), encodeDyn(hashes)); } - else - // TODO: Starting from constantinople blockhash always seems to return zero. - // The blockhash contract introduced in EIP96 seems to break in our setup of - // aleth (setting the constantinople fork block to zero and resetting the chain - // to block zero before each test run). Pre-deploying the blockchain contract - // during genesis seems to help, but currently causes problems with other tests. - // Set the expectation to zero for now, so that this test tracks changes in this - // behavior. - hashes.assign(259, 0); - - ABI_CHECK(callContractFunction("f()"), encodeDyn(hashes)); } BOOST_AUTO_TEST_CASE(value_complex) |