Revert "migrate from catch2v2 to catch2v3 and implement VisorTest helper lib (#689)" (#691)

This reverts commit 20403fb.

Author: weyrick

conanfile.txt

@@ -1,5 +1,6 @@
 [requires]
-catch2/3.4.0
+benchmark/1.8.3
+catch2/2.13.10
 corrade/2020.06
 cpp-httplib/0.14.0
 docopt.cpp/0.6.3

libs/CMakeLists.txt

@@ -1,6 +1,5 @@
 message(STATUS "Custom Libraries")
 
-add_subdirectory(visor_test)
 add_subdirectory(visor_transaction)
 add_subdirectory(visor_tcp)
 add_subdirectory(visor_dns)

libs/visor_dns/CMakeLists.txt

@@ -24,16 +24,4 @@ target_link_libraries(VisorLibDns
         ${CONAN_LIBS_FMT}
         )
 
-## TEST SUITE
-add_executable(unit-tests-visor-dns 
-        tests/test_dns.cpp
-        tests/benchmark_dns.cpp)
-
-target_link_libraries(unit-tests-visor-dns 
-        PRIVATE
-        Visor::Lib::Dns
-        ${CONAN_LIBS_CATCH2})
-
-add_test(NAME unit-tests-visor-dns
-        WORKING_DIRECTORY ${PROJECT_SOURCE_DIR}/libs/visor_dns
-        COMMAND unit-tests-visor-dns)
+add_subdirectory(tests)

libs/visor_dns/tests/CMakeLists.txt

@@ -0,0 +1,24 @@
+# Unit
+add_executable(unit-tests-visor-dns
+        main.cpp
+        test_dns.cpp
+        )
+
+target_link_libraries(unit-tests-visor-dns
+        PRIVATE
+        ${CONAN_LIBS_JSON-SCHEMA-VALIDATOR}
+        Visor::Lib::Dns)
+
+add_test(NAME unit-tests-visor-dns
+        WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
+        COMMAND unit-tests-visor-dns
+        )
+
+# Benchmark
+add_executable(benchmark-visor-dns
+        benchmark_dns.cpp
+        )
+
+target_link_libraries(benchmark-visor-dns PRIVATE
+        Visor::Lib::Dns
+        ${CONAN_LIBS_BENCHMARK})

libs/visor_dns/tests/benchmark_dns.cpp

@@ -1,9 +1,8 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
-#include <catch2/benchmark/catch_benchmark.hpp>
-#include <catch2/catch_test_macros.hpp>
 
+#include "benchmark/benchmark.h"
 #include "dns.h"
 #ifdef __GNUC__
 #pragma GCC diagnostic push
@@ -22,65 +21,66 @@
 
 using namespace visor::lib::dns;
 
-void BM_aggregateDomain(const std::string &domain)
+static void BM_aggregateDomain(benchmark::State &state)
 {
     AggDomainResult result;
-    result = aggregateDomain(domain);
+    std::string domain{"biz.foo.bar.com"};
+    for (auto _ : state) {
+        result = aggregateDomain(domain);
+    }
 }
+BENCHMARK(BM_aggregateDomain);
 
-void BM_pcapReadNoParse()
+static void BM_aggregateDomainLong(benchmark::State &state)
 {
-    auto reader = pcpp::IFileReaderDevice::getReader("tests/dns_udp_tcp_random.pcap");
-
-    if (!reader->open()) {
-        throw std::runtime_error("Cannot open pcap/pcapng file");
-    }
-
-    pcpp::RawPacket rawPacket;
-    while (reader->getNextPacket(rawPacket)) {
+    AggDomainResult result;
+    std::string domain{"long1.long2.long3.long4.long5.long6.long7.long8.biz.foo.bar.com"};
+    for (auto _ : state) {
+        result = aggregateDomain(domain);
     }
-
-    reader->close();
-    delete reader;
 }
 
-void BM_pcapReadParse()
+BENCHMARK(BM_aggregateDomainLong);
+
+static void BM_pcapReadNoParse(benchmark::State &state)
 {
 
-    auto reader = pcpp::IFileReaderDevice::getReader("tests/dns_udp_tcp_random.pcap");
+    for (auto _ : state) {
+        auto reader = pcpp::IFileReaderDevice::getReader("fixtures/dns_udp_tcp_random.pcap");
 
-    if (!reader->open()) {
-        throw std::runtime_error("Cannot open pcap/pcapng file");
-    }
+        if (!reader->open()) {
+            throw std::runtime_error("Cannot open pcap/pcapng file");
+        }
 
-    pcpp::RawPacket rawPacket;
-    while (reader->getNextPacket(rawPacket)) {
-        pcpp::Packet packet(&rawPacket, pcpp::OsiModelTransportLayer);
-    }
+        pcpp::RawPacket rawPacket;
+        while (reader->getNextPacket(rawPacket)) {
+        }
 
-    reader->close();
-    delete reader;
+        reader->close();
+        delete reader;
+    }
 }
+BENCHMARK(BM_pcapReadNoParse);
 
-TEST_CASE("DNS benchmark")
+static void BM_pcapReadParse1(benchmark::State &state)
 {
-    BENCHMARK("Aggregate Domain")
-    {
-        return BM_aggregateDomain("biz.foo.bar.com");
-    };
-
-    BENCHMARK("Aggregate Domain Long")
-    {
-        return BM_aggregateDomain("long1.long2.long3.long4.long5.long6.long7.long8.biz.foo.bar.com");
-    };
-
-    BENCHMARK("Pcap Read No Parse")
-    {
-        return BM_pcapReadNoParse();
-    };
-
-    BENCHMARK("Pcap Read No Parse")
-    {
-        return BM_pcapReadParse();
-    };
+
+    for (auto _ : state) {
+        auto reader = pcpp::IFileReaderDevice::getReader("fixtures/dns_udp_tcp_random.pcap");
+
+        if (!reader->open()) {
+            throw std::runtime_error("Cannot open pcap/pcapng file");
+        }
+
+        pcpp::RawPacket rawPacket;
+        while (reader->getNextPacket(rawPacket)) {
+            pcpp::Packet packet(&rawPacket, pcpp::OsiModelTransportLayer);
+        }
+
+        reader->close();
+        delete reader;
+    }
 }
+BENCHMARK(BM_pcapReadParse1);
+
+BENCHMARK_MAIN();

libs/visor_dns/tests/dns_udp_tcp_random.pcap

@@ -0,0 +1,24 @@
+#define CATCH_CONFIG_RUNNER
+#include <catch2/catch.hpp>
+#include <cstdlib>
+#include <spdlog/sinks/stdout_color_sinks.h>
+#include <spdlog/spdlog.h>
+
+int main(int argc, char *argv[])
+{
+    Catch::Session session;
+
+    auto logger = spdlog::get("visor");
+    if (!logger) {
+        spdlog::stderr_color_mt("visor");
+        }
+
+    int result = session.applyCommandLine(argc, argv);
+    if (result != 0) {
+        return result;
+    }
+
+    result = session.run();
+
+    return (result == 0 ? EXIT_SUCCESS : EXIT_FAILURE);
+}

libs/visor_dns/tests/main.cpp

@@ -1,4 +1,4 @@
-#include <catch2/catch_all.hpp>
+#include "catch2/catch.hpp"
 
 #include "dns.h"