Merge pull request #123 from ethereum/develop

version 0.1.1 development

Author: winsvega

CMakeLists.txt

@@ -14,8 +14,8 @@ HunterGate(
 	LOCAL
 )
 
-project(retesteth VERSION 0.1.0)
-set(VERSION_SUFFIX "accesslist")
+project(retesteth VERSION 0.1.1)
+set(VERSION_SUFFIX "eip1559")
 
 set(Boost_USE_STATIC_LIBS ON)
 set(Boost_USE_MULTITHREADED ON)

Dockerfile

@@ -33,7 +33,7 @@ RUN mkdir /build && cd /build \
 #    && cmake /solidity -DCMAKE_BUILD_TYPE=Release && make solc \
 #    && cp /build/solc/solc /bin/solc \
 #    && rm -rf /build /solidity /var/cache/* /root/.hunter/*
-RUN apt-get install wget && wget https://github.com/ethereum/solidity/releases/download/v0.7.0/solc-static-linux \
+RUN apt-get install wget && wget https://github.com/ethereum/solidity/releases/download/v0.8.5/solc-static-linux \
    && cp solc-static-linux /bin/solc \
    && chmod +x /bin/solc
 

circle.yml

@@ -149,24 +149,23 @@ jobs:
       - image: ethereum/cpp-build-env:7
     steps: *linux-steps
 
-  macOS-XCode10:
+  macOS-XCode11:
     environment:
       - CXX: clang++
       - GCOV: gcov
       - GENERATOR: Ninja
       - BUILD_PARALLEL_JOBS: 8
       - TEST_PARALLEL_JOBS: 8
     macos:
-      xcode: "10.0"
+      xcode: "11.3.1"
     steps:
       - checkout
       - *environment-info
       - run:
           name: "Install macOS dependencies"
           # Python3 and requests are needed for Hunter upload.
           command: |
-            HOMEBREW_NO_AUTO_UPDATE=1 brew install -q cmake ninja python3
-            pip3 install -q requests gitpython
+            HOMEBREW_NO_AUTO_UPDATE=1 brew install -q cmake ninja
       - *configureMac
 #      - *upload-hunter-cache
 #      - *install-solidity-mac
@@ -182,6 +181,6 @@ workflows:
   version: 2
   cpp-ethereum:
     jobs:
-      - macOS-XCode10
+      - macOS-XCode11
       - Linux-Clang6
       - Linux-GCC6-Debug

cmake/Hunter/config.cmake

@@ -17,7 +17,10 @@ hunter_config(
     HTTP_CLIENT=NO
 )
 
-hunter_config(Boost VERSION 1.65.1)
+hunter_config(Boost VERSION 1.65.2
+    URL http://retesteth.ethdevops.io/dependencies/boost_1_65_1.tar.bz2
+    SHA1 4a5b0c3c1b1b9a4d6cb6a6cc395e903e76f76720
+)
 
 hunter_config(ethash VERSION 0.4.0
     URL https://github.com/chfast/ethash/archive/v0.4.0.tar.gz

retesteth/Options.cpp

@@ -224,6 +224,24 @@ Options::Options(int argc, const char** argv)
         else if (arg == "--fillchain")
         {
             fillchain = true;
+
+            bool noFilltests = !filltests;
+            if (noFilltests)
+            {
+                // Look ahead if this option ever provided
+                for (auto i = 0; i < argc; ++i)
+                {
+                    auto arg = std::string{argv[i]};
+                    if (arg == "--filltests")
+                    {
+                        noFilltests = false;
+                        break;
+                    }
+                }
+            }
+
+            if (noFilltests)
+                ETH_STDOUT_MESSAGEC("WARNING: `--fillchain` option provided without `--filltests`, activating `--filltests` (did you mean `--filltests`?)", cYellow);
             filltests = true;
         }
         else if (arg == "--showhash")

retesteth/Options.h

@@ -83,6 +83,7 @@ class Options
 	/// The first time used, options are parsed with argc, argv
 	static Options const& get(int argc = 0, const char** argv = 0);
     static DynamicOptions& getDynamicOptions() { return m_dynamicOptions; }
+    static ClientConfig const& getCurrentConfig() { return m_dynamicOptions.getCurrentConfig(); }
     string getGStateTransactionFilter() const;
 
 private:

retesteth/TestOutputHelper.cpp

@@ -108,7 +108,7 @@ bool TestOutputHelper::markError(std::string const& _message)
     m_errors.push_back(_message + testDebugInfo);
     if (testDebugInfo.empty())
         ETH_WARNING(TestOutputHelper::get().testName() + ", Message: " + _message +
-                    ", has empty debugInfo! Missing debug Tesinfo for test step.");
+                    ", has empty debugInfo! Missing debug Testinfo for test step.");
     std::lock_guard<std::mutex> lock(g_failedTestsMap);
     if (!s_failedTestsMap.count(TestOutputHelper::get().testName()))
         s_failedTestsMap[TestOutputHelper::get().testName()] = testDebugInfo;

retesteth/TestSuite.cpp

@@ -213,51 +213,78 @@ string const c_copierPostf = "Copier";
 
 void TestSuite::runTestWithoutFiller(boost::filesystem::path const& _file) const
 {
-    for (auto const& config : Options::getDynamicOptions().getClientConfigs())
+    try
     {
-        Options::getDynamicOptions().setCurrentConfig(config);
-
-        std::cout << "Running tests for config '" << config.cfgFile().name() << "' "
-                  << config.getId().id() << std::endl;
-        ETH_LOG("Running " + _file.filename().string() + ": ", 3);
-
-        // Allow to execute a custom test .json file on any test suite
-        auto& testOutput = test::TestOutputHelper::get();
-        testOutput.initTest(1);
-
-        if (Options::get().filltests)
+        for (auto const& config : Options::getDynamicOptions().getClientConfigs())
         {
-            TestFileData testData = readTestFile(_file);
-            removeComments(testData.data);
+            Options::getDynamicOptions().setCurrentConfig(config);
 
-            string fileName = _file.stem().c_str();
-            if (fileName.find("Filler") == string::npos)
-                ETH_ERROR_MESSAGE("Trying to fill `" + string(_file.c_str()) + "`, but file does not have Filler suffix!");
+            std::cout << "Running tests for config '" << config.cfgFile().name() << "' " << config.getId().id() << std::endl;
+            ETH_LOG("Running " + _file.filename().string() + ": ", 3);
 
-            // output filename. substract Filler suffix
-            fileName = fileName.substr(0, fileName.length() - 6) + ".json";
+            // Allow to execute a custom test .json file on any test suite
+            auto& testOutput = test::TestOutputHelper::get();
+            testOutput.initTest(1);
 
-            fs::path outPath;
-            if (Options::get().singleTestOutFile.is_initialized())
-                outPath = fs::path(Options::get().singleTestOutFile.get());
-            else
-                outPath = _file.parent_path() / fileName;
-
-            TestSuiteOptions opt;
-            opt.doFilling = true;
-            opt.allowInvalidBlocks = true;
-            DataObject output = doTests(testData.data, opt);
-            addClientInfo(output, _file, testData.hash);
-            writeFile(outPath, asBytes(output.asJson()));
-        }
-        else
-            executeFile(_file);
-
-        testOutput.finishTest();
+            try
+            {
+                if (Options::get().filltests)
+                {
+                    TestFileData testData = readTestFile(_file);
+                    removeComments(testData.data);
+
+                    string fileName = _file.stem().c_str();
+                    if (fileName.find("Filler") == string::npos)
+                        ETH_ERROR_MESSAGE(
+                            "Trying to fill `" + string(_file.c_str()) + "`, but file does not have Filler suffix!");
+
+                    // output filename. substract Filler suffix
+                    fileName = fileName.substr(0, fileName.length() - 6) + ".json";
+
+                    fs::path outPath;
+                    if (Options::get().singleTestOutFile.is_initialized())
+                        outPath = fs::path(Options::get().singleTestOutFile.get());
+                    else
+                        outPath = _file.parent_path() / fileName;
+
+                    TestSuiteOptions opt;
+                    opt.doFilling = true;
+                    opt.allowInvalidBlocks = true;
+                    DataObject output = doTests(testData.data, opt);
+                    addClientInfo(output, _file, testData.hash);
+                    writeFile(outPath, asBytes(output.asJson()));
+                }
+                else
+                    executeFile(_file);
+            }
+            catch (test::EthError const& _ex)
+            {
+                // Something went wrong inside the test. skip the test.
+                // (error message is stored at TestOutputHelper. EthError is via ETH_ERROR_())
+            }
+            catch (test::UpwardsException const& _ex)
+            {
+                // UpwardsException is thrown upwards in tests for debug info
+                // And it should be catched on upper level for report till this point
+                ETH_ERROR_MESSAGE(string("Unhandled UpwardsException: ") + _ex.what());
+            }
+            catch (std::exception const& _ex)
+            {
+                if (!ExitHandler::receivedExitSignal())
+                    ETH_ERROR_MESSAGE("ERROR OCCURED TESTFILE RUN: " + string(_ex.what()));
+                RPCSession::sessionEnd(TestOutputHelper::getThreadID(), RPCSession::SessionStatus::HasFinished);
+            }
 
-        // Disconnect threads from the client
-        if (Options::getDynamicOptions().getClientConfigs().size() > 1)
-            RPCSession::clear();
+            testOutput.finishTest();
+            // Disconnect threads from the client
+            if (Options::getDynamicOptions().getClientConfigs().size() > 1)
+                RPCSession::clear();
+        }
+    }
+    catch (std::exception const&)
+    {
+        test::TestOutputHelper::get().finishTest();
+        test::TestOutputHelper::printTestExecStats();
     }
 }
 
@@ -479,6 +506,8 @@ void TestSuite::runFunctionForAllClients(std::function<void()> _func)
         Options::getDynamicOptions().setCurrentConfig(config);
         std::cout << "Running tests for config '" << config.cfgFile().name() << "' "
                   << config.getId().id() << std::endl;
+
+        // Run tests
         _func();
 
         // Disconnect threads from the client

retesteth/compiler/Compiler.cpp

@@ -72,6 +72,7 @@ string replaceCode(string const& _code, solContracts const& _preSolidity)
     string const c_rawPrefix = ":raw";
     string const c_abiPrefix = ":abi";
     string const c_solidityPrefix = ":solidity";
+    string const c_yulPrefix = ":yul";
 
     if (_code.find("pragma solidity") != string::npos)
     {
@@ -99,6 +100,13 @@ string replaceCode(string const& _code, solContracts const& _preSolidity)
         compiledCode = utiles::encodeAbi(abiCode);
         utiles::checkHexHasEvenLength(compiledCode);
     }
+    else if (_code.find(c_yulPrefix) != string::npos)
+    {
+        size_t const pos = _code.find(c_yulPrefix);
+        string const yulCode = _code.substr(pos + c_yulPrefix.length() + 1);
+        compiledCode = compileYul(yulCode);
+        utiles::checkHexHasEvenLength(compiledCode);
+    }
     else if (_code.find('{') != string::npos || _code.find("(asm") != string::npos )
         compiledCode = compileLLL(_code);
     else

retesteth/compiler/Compiler.h

@@ -36,5 +36,8 @@ solContracts compileSolidity(string const& _code);
 /// compile LLL / wasm or other src code into bytecode
 std::string replaceCode(std::string const& _code, solContracts const& _preSolidity = solContracts());
 
+/// compile clean YUL
+std::string compileYul(string const& _code);
+
 }  // namespace compiler
 }  // namespace test