more tests pass

Author: d-led

CMakeLists.txt

@@ -319,9 +319,9 @@ endif()
 enable_testing()
 
 # Register tests with CTest
-add_test(NAME test-influxdb-cpp-rest COMMAND test-influxdb-cpp-rest)
-add_test(NAME test-influx-c-rest COMMAND test-influx-c-rest)
-add_test(NAME test-influxdb-cpp-auth COMMAND test-influxdb-cpp-auth)
+add_test(NAME test-influxdb-cpp-rest COMMAND ${CMAKE_BINARY_DIR}/bin/test-influxdb-cpp-rest)
+add_test(NAME test-influx-c-rest COMMAND ${CMAKE_BINARY_DIR}/bin/test-influx-c-rest)
+add_test(NAME test-influxdb-cpp-auth COMMAND ${CMAKE_BINARY_DIR}/bin/test-influxdb-cpp-auth)
 
 # Set output directories for executables
 set_target_properties(test-influxdb-cpp-rest test-influx-c-rest test-influxdb-cpp-auth

src/auth_test/auth_test.cpp

@@ -20,9 +20,21 @@
 #include <chrono>
 #include <thread>
 #include <iostream>
+#include <random>
 
 const char* url = "http://localhost:8086";
-const char* db_name = "auth_test";
+std::string db_name;
+
+// Generate random database name at startup
+static struct db_name_initializer {
+    db_name_initializer() {
+        std::random_device rd;
+        std::mt19937 gen(rd());
+        std::uniform_int_distribution<> dis(0, 999999);
+        db_name = "auth_test_" + std::to_string(dis(gen));
+    }
+} db_name_init;
+
 const std::string username = "admin";
 const std::string password = "auth";
 
@@ -45,14 +57,14 @@ struct authentication_test {
         db.with_authentication(username, password);
 
         asyncdb = std::shared_ptr<influx_c_rest_async_t>(
-            influx_c_rest_async_new_auth(url, db_name, username.c_str(), password.c_str()),
+            influx_c_rest_async_new_auth(url, db_name.c_str(), username.c_str(), password.c_str()),
             influx_c_rest_async_destroy
         );
 
         REQUIRE(asyncdb.get());
 
         query = std::shared_ptr<influx_c_rest_query_t>(
-            influx_c_rest_query_new_auth(url, db_name, username.c_str(), password.c_str()),
+            influx_c_rest_query_new_auth(url, db_name.c_str(), username.c_str(), password.c_str()),
             influx_c_rest_query_destroy
             );
 

src/test/fixtures.cpp

@@ -12,15 +12,17 @@
 #include <chrono>
 #include <thread>
 
-connected_test::connected_test() 
-{
-    // Generate random database name to ensure tests are idempotent
+std::string connected_test::generate_random_db_name() {
     std::random_device rd;
     std::mt19937 gen(rd());
     std::uniform_int_distribution<> dis(0, 999999);
-    db_name = "testdb_" + std::to_string(dis(gen));
-    
-    raw_db = influxdb::raw::db_utf8("http://localhost:8086", db_name);
+    return "testdb_" + std::to_string(dis(gen));
+}
+
+connected_test::connected_test() 
+    : db_name(generate_random_db_name()),
+      raw_db("http://localhost:8086", db_name)
+{
     raw_db.post(std::string("drop database ") + db_name);
     wait_for_no_db(db_name);
     raw_db.post(std::string("create database ") + db_name);

src/test/fixtures.h

@@ -12,10 +12,14 @@
 #include <random>
 
 struct connected_test {
+    std::string db_name;
     influxdb::raw::db_utf8 raw_db;
     const int milliseconds_waiting_time = 100;
-    std::string db_name;
 
+private:
+    static std::string generate_random_db_name();
+
+public:
     // drop and create test db
     connected_test();
 

src/test/utf8_client_test.cpp

@@ -11,13 +11,13 @@
 #include "fixtures.h"
 
 TEST_CASE_METHOD(connected_test, "creating a database", "[connected]") {
-    CHECK(database_exists("testdb"));
+    CHECK(database_exists(db_name));
 }
 
 
 TEST_CASE_METHOD(connected_test, "posting simple values", "[connected]") {
     auto testdb_test_entries = [this]() {
-        return raw_db.get("select * from testdb..test");
+        return raw_db.get("select * from " + db_name + "..test");
     };
     CHECK(testdb_test_entries().find("hello") == std::string::npos);