Merge pull request #292 from AxVultis/unit-tests

Add tests
ConcealNetwork · Jul 8, 2022 · 4e00ee1 · 4e00ee1
2 parents 01afa1b + 6dd475e
commit 4e00ee1
Show file tree

Hide file tree

Showing 81 changed files with 1,215 additions and 760 deletions.
diff --git a/.github/workflows/check.yml b/.github/workflows/check.yml
@@ -13,7 +13,7 @@ jobs:
     name: Windows
     runs-on: windows-2019
     env:
-      BOOST_ROOT: C:/tools/boost/x86_64
+      BOOST_ROOT: C:/local/boost_1_72_0
     steps:
       - uses: actions/checkout@master
 
@@ -54,7 +54,7 @@ jobs:
           mv build/src/Release/concealwallet.exe build/conceal
           mv build/src/Release/optimizer.exe build/conceal
           mv build/src/Release/walletd.exe build/conceal
-          mv build/tests/Release/*_tests.exe build/conceal
+          cp build/tests/Release/*_tests.exe build/conceal
 
       - name: Upload To GH Artifacts
         uses: actions/upload-artifact@v2
@@ -82,6 +82,90 @@ jobs:
         run: |
           cd build/conceal
           ./performance_tests.exe
+      
+      - name: unit_tests
+        shell: powershell
+        run: |
+          cd build/tests
+          ctest -R UnitTests --output-on-failure
+
+  build-mingw:
+    name: MinGW
+    runs-on: windows-2019
+    steps:
+      - uses: msys2/setup-msys2@v2
+        with:
+          update: true
+          install: >-
+            git
+            base-devel
+            mingw-w64-x86_64-toolchain
+            mingw-w64-x86_64-boost
+            mingw-w64-x86_64-cmake
+
+      - uses: actions/checkout@master
+
+      - name: Prepare version
+        shell: powershell
+        id: setup
+        run: |
+          $os="mingw"
+          $ccx_version="${{ github.sha }}".SubString(0,7)
+          $release_name="ccx-cli-$os-dev-$ccx_version"
+          echo "::set-output name=release_name::${release_name}"
+
+      - name: Build
+        shell: msys2 {0}
+        id: build
+        run: |
+          ls
+          mkdir build
+          cd build
+          cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTS=ON -DSTATIC=ON
+          cmake --build . --config Release
+
+      - name: Prepare release
+        shell: msys2 {0}
+        run: |
+          mkdir build/conceal
+          mv build/src/conceald.exe build/conceal
+          mv build/src/concealwallet.exe build/conceal
+          mv build/src/optimizer.exe build/conceal
+          mv build/src/walletd.exe build/conceal
+          cp build/tests/*_tests.exe build/conceal
+
+      - name: Upload To GH Artifacts
+        uses: actions/upload-artifact@v2
+        with:
+          name: ${{ steps.setup.outputs.release_name }}
+          path: build/conceal
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: core_tests
+        shell: powershell
+        run: |
+          cd build/conceal
+          ./core_tests.exe --generate_and_play_test_data
+
+      - name: difficulty_tests
+        shell: powershell
+        run: |
+          cp tests/Difficulty/data.txt build/conceal
+          cd build/conceal
+          ./difficulty_tests.exe data.txt
+
+      - name: performance_tests
+        shell: powershell
+        run: |
+          cd build/conceal
+          ./performance_tests.exe
+      
+      - name: unit_tests
+        shell: powershell
+        run: |
+          cd build/tests
+          ctest -R UnitTests --output-on-failure
 
   build-ubuntu18:
     name: Ubuntu 18.04
@@ -117,7 +201,7 @@ jobs:
           mv build/src/concealwallet build/conceal
           mv build/src/optimizer build/conceal
           mv build/src/walletd build/conceal
-          mv build/tests/*_tests build/conceal
+          cp build/tests/*_tests build/conceal
 
       - name: Upload To GH Artifacts
         uses: actions/upload-artifact@v2
@@ -147,6 +231,11 @@ jobs:
         run: |
           cd build/conceal
           ./system_tests
+      
+      - name: unit_tests
+        run: |
+          cd build/tests
+          ctest -R UnitTests --output-on-failure
 
   build-ubuntu20:
     name: Ubuntu 20.04
@@ -182,7 +271,7 @@ jobs:
           mv build/src/concealwallet build/conceal
           mv build/src/optimizer build/conceal
           mv build/src/walletd build/conceal
-          mv build/tests/*_tests build/conceal
+          cp build/tests/*_tests build/conceal
 
       - name: Upload To GH Artifacts
         uses: actions/upload-artifact@v2
@@ -213,6 +302,11 @@ jobs:
           cd build/conceal
           ./system_tests
 
+      - name: unit_tests
+        run: |
+          cd build/tests
+          ctest -R UnitTests --output-on-failure
+
   build-ubuntu20-clang:
     name: Ubuntu 20.04 clang
     runs-on: ubuntu-20.04
@@ -247,7 +341,7 @@ jobs:
           mv build/src/concealwallet build/conceal
           mv build/src/optimizer build/conceal
           mv build/src/walletd build/conceal
-          mv build/tests/*_tests build/conceal
+          cp build/tests/*_tests build/conceal
 
       - name: Upload To GH Artifacts
         uses: actions/upload-artifact@v2
@@ -278,6 +372,11 @@ jobs:
           cd build/conceal
           ./system_tests
 
+      - name: unit_tests
+        run: |
+          cd build/tests
+          ctest -R UnitTests --output-on-failure
+
   build-macos11:
     name: macOS
     runs-on: macos-11
@@ -311,7 +410,7 @@ jobs:
           mv build/src/concealwallet build/conceal
           mv build/src/optimizer build/conceal
           mv build/src/walletd build/conceal
-          mv build/tests/*_tests build/conceal
+          cp build/tests/*_tests build/conceal
 
       - name: Upload To GH Artifacts
         uses: actions/upload-artifact@v2
@@ -336,3 +435,8 @@ jobs:
         run: |
           cd build/conceal
           ./performance_tests
+
+      - name: unit_tests
+        run: |
+          cd build/tests
+          ctest -R UnitTests --output-on-failure
diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml
@@ -10,7 +10,7 @@ jobs:
     name: Windows
     runs-on: windows-2019
     env:
-      BOOST_ROOT: C:/hostedtoolcache/windows/Boost/1.72.0/x86_64
+      BOOST_ROOT: C:/local/boost_1_72_0
     steps:
       - uses: actions/checkout@master
 

diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -28,7 +28,7 @@ if(APPLE OR FREEBSD)
   enable_language(ASM)
 endif()
 
-if(MSVC)
+if(MSVC OR MINGW)
   include_directories(src/Platform/Windows)
 elseif(APPLE)
   include_directories(src/Platform/OSX)
@@ -83,6 +83,10 @@ set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${ARCH_FLAG}")
 
 set(STATIC ${MSVC} CACHE BOOL "Link libraries statically")
 
+if(MINGW AND STATIC)
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -static-libstdc++ -static-libgcc")
+endif()
+
 if(MSVC)
   add_definitions("/bigobj /MP /W3 /GS- /D_CRT_SECURE_NO_WARNINGS /wd4996 /wd4345 /D_WIN32_WINNT=0x0600 /DWIN32_LEAN_AND_MEAN /DGTEST_HAS_TR1_TUPLE=0 /D_VARIADIC_MAX=8 /D__SSE4_1__")
   set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /STACK:10485760")
@@ -121,7 +125,7 @@ else()
   set(C_WARNINGS "-Waggregate-return -Wnested-externs -Wold-style-definition -Wstrict-prototypes")
   set(CXX_WARNINGS "-Wno-reorder -Wno-missing-field-initializers")
   set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -std=c11 ${MINGW_FLAG} ${WARNINGS} ${C_WARNINGS}")
-  if(NOT APPLE)
+  if(NOT APPLE AND NOT MINGW)
     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pthread")
   endif()
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 ${MINGW_FLAG} ${WARNINGS} ${CXX_WARNINGS}")
@@ -152,7 +156,7 @@ find_package(Boost 1.55 REQUIRED COMPONENTS system filesystem thread date_time c
 include_directories(SYSTEM ${Boost_INCLUDE_DIRS})
 
 if(MINGW)
-  set(Boost_LIBRARIES "${Boost_LIBRARIES};ws2_32;mswsock")
+  set(Boost_LIBRARIES "${Boost_LIBRARIES};ws2_32;mswsock;bcrypt")
 elseif(APPLE)
   set(Boost_LIBRARIES "${Boost_LIBRARIES}")
 elseif(NOT MSVC)
@@ -200,5 +204,9 @@ else()
   endif()
 endif()
 
+if(MINGW AND STATIC)
+  set (CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -Wl,-Bstatic,--whole-archive -lwinpthread -Wl,--no-whole-archive")
+endif()
+
 add_subdirectory(external)
 add_subdirectory(src)
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -23,7 +23,7 @@ file(GLOB_RECURSE JsonRpcServer JsonRpcServer/*)
 file(GLOB_RECURSE PaymentGate PaymentGate/*)
 file(GLOB_RECURSE PaymentGateService PaymentGateService/*)
 
-if(MSVC)
+if(MSVC OR MINGW)
   file(GLOB_RECURSE System System/* Platform/Windows/System/*)
 elseif(APPLE)
   file(GLOB_RECURSE System System/* Platform/OSX/System/*)
@@ -59,6 +59,12 @@ if (MSVC)
   target_link_libraries(System ws2_32)
 endif ()
 
+if (MINGW)
+  target_link_libraries(Common dnsapi)
+  target_link_libraries(P2P bcrypt)
+  target_link_libraries(System ws2_32)
+endif ()
+
 target_link_libraries(Daemon CryptoNoteCore P2P Rpc System Http Logging Common crypto libminiupnpc-static BlockchainExplorer ${Boost_LIBRARIES} Serialization)
 target_link_libraries(SimpleWallet Wallet NodeRpcProxy Transfers Rpc Http CryptoNoteCore System Logging Common crypto ${Boost_LIBRARIES} Serialization)
 target_link_libraries(PaymentGateService PaymentGate JsonRpcServer Wallet NodeRpcProxy Transfers CryptoNoteCore crypto P2P Rpc Http System Logging Common InProcessNode libminiupnpc-static BlockchainExplorer ${Boost_LIBRARIES} Serialization)

diff --git a/src/Common/CommandLine.cpp b/src/Common/CommandLine.cpp
@@ -12,4 +12,9 @@ namespace command_line
   const arg_descriptor<bool> arg_help = {"help", "Produce help message"};
   const arg_descriptor<bool> arg_version = {"version", "Output version information"};
   const arg_descriptor<std::string> arg_data_dir = {"data-dir", "Specify data directory"};
+  const command_line::arg_descriptor<bool> arg_testnet_on = {
+    "testnet",
+    "Use testnet instead of mainnet",
+    false
+  };
 }
diff --git a/src/Common/CommandLine.h b/src/Common/CommandLine.h
@@ -176,4 +176,5 @@ namespace command_line
   extern const arg_descriptor<bool> arg_help;
   extern const arg_descriptor<bool> arg_version;
   extern const arg_descriptor<std::string> arg_data_dir;
+  extern const command_line::arg_descriptor<bool> arg_testnet_on;
 }
diff --git a/src/Common/StreamTools.cpp b/src/Common/StreamTools.cpp
@@ -16,7 +16,7 @@ void read(IInputStream& in, void* data, size_t size) {
   while (size > 0) {
     size_t readSize = in.readSome(data, size);
     if (readSize == 0) {
-      throw std::runtime_error("Failed to read from IInputStream");
+      throw std::runtime_error("Reading from IInputStream failed");
     }
 
     data = static_cast<uint8_t*>(data) + readSize;
@@ -85,7 +85,7 @@ void readVarint(IInputStream& in, uint8_t& value) {
     temp |= static_cast<size_t>(piece & 0x7f) << shift;
     if ((piece & 0x80) == 0) {
       if (piece == 0 && shift != 0) {
-        throw std::runtime_error("readVarint, invalid value representation");
+        throw std::runtime_error("readVarint, unrepresentable value");
       }
 
       break;
@@ -107,7 +107,7 @@ void readVarint(IInputStream& in, uint16_t& value) {
     temp |= static_cast<size_t>(piece & 0x7f) << shift;
     if ((piece & 0x80) == 0) {
       if (piece == 0 && shift != 0) {
-        throw std::runtime_error("readVarint, invalid value representation");
+        throw std::runtime_error("readVarint, unrepresentable value");
       }
 
       break;
@@ -129,7 +129,7 @@ void readVarint(IInputStream& in, uint32_t& value) {
     temp |= static_cast<size_t>(piece & 0x7f) << shift;
     if ((piece & 0x80) == 0) {
       if (piece == 0 && shift != 0) {
-        throw std::runtime_error("readVarint, invalid value representation");
+        throw std::runtime_error("readVarint, unrepresentable value");
       }
 
       break;
@@ -151,7 +151,7 @@ void readVarint(IInputStream& in, uint64_t& value) {
     temp |= static_cast<uint64_t>(piece & 0x7f) << shift;
     if ((piece & 0x80) == 0) {
       if (piece == 0 && shift != 0) {
-        throw std::runtime_error("readVarint, invalid value representation");
+        throw std::runtime_error("readVarint, unrepresentable value");
       }
 
       break;
@@ -165,7 +165,7 @@ void write(IOutputStream& out, const void* data, size_t size) {
   while (size > 0) {
     size_t writtenSize = out.writeSome(data, size);
     if (writtenSize == 0) {
-      throw std::runtime_error("Failed to write to IOutputStream");
+      throw std::runtime_error("IOutputStream could not be written to");
     }
 
     data = static_cast<const uint8_t*>(data) + writtenSize;

diff --git a/src/Common/StringTools.cpp b/src/Common/StringTools.cpp
@@ -66,11 +66,11 @@ bool fromHex(char character, uint8_t& value) {
 
 size_t fromHex(const std::string& text, void* data, size_t bufferSize) {
   if ((text.size() & 1) != 0) {
-    throw std::runtime_error("fromHex: invalid string size");
+    throw std::runtime_error("incorrect string size in fromHex");
   }
 
   if (text.size() >> 1 > bufferSize) {
-    throw std::runtime_error("fromHex: invalid buffer size");
+    throw std::runtime_error("incorrect buffer size in fromHex");
   }
 
   for (size_t i = 0; i < text.size() >> 1; ++i) {
@@ -109,7 +109,7 @@ bool fromHex(const std::string& text, void* data, size_t bufferSize, size_t& siz
 
 std::vector<uint8_t> fromHex(const std::string& text) {
   if ((text.size() & 1) != 0) {
-    throw std::runtime_error("fromHex: invalid string size");
+    throw std::runtime_error("incorrect string size in fromHex");
   }
 
   std::vector<uint8_t> data(text.size() >> 1);

diff --git a/src/Common/Util.cpp b/src/Common/Util.cpp
@@ -286,7 +286,7 @@ std::string get_nix_version_display_string()
   }
 #endif
 
-  std::string getDefaultDataDirectory()
+  std::string getDefaultDataDirectory(bool testnet)
   {
     //namespace fs = boost::filesystem;
     // Windows < Vista: C:\Documents and Settings\Username\Application Data\BLOCKCHAIN_DIR
@@ -299,8 +299,8 @@ std::string get_nix_version_display_string()
     config_folder = get_special_folder_path(CSIDL_APPDATA, true) + "/" + cn::BLOCKCHAIN_DIR;
 #else
     std::string pathRet;
-    char* pszHome = getenv("HOME");
-    if (pszHome == NULL || strlen(pszHome) == 0)
+    const char* pszHome = getenv("HOME");
+    if (pszHome == nullptr || strlen(pszHome) == 0)
       pathRet = "/";
     else
       pathRet = pszHome;
@@ -314,7 +314,7 @@ std::string get_nix_version_display_string()
 #endif
 #endif
 
-    return config_folder;
+    return config_folder + (testnet ? "/testnet" : "");
   }
 
   bool create_directories_if_necessary(const std::string& path)

diff --git a/src/Common/Util.h b/src/Common/Util.h
@@ -12,7 +12,7 @@
 
 namespace tools
 {
-  std::string getDefaultDataDirectory();
+  std::string getDefaultDataDirectory(bool testnet = false);
   std::string get_os_version_string();
   bool create_directories_if_necessary(const std::string& path);
   std::error_code replace_file(const std::string& replacement_name, const std::string& replaced_name);