Add system tests for transport API (#216)

Author: offa

test/system/HttpAuthST.cxx

@@ -21,6 +21,7 @@
 // SOFTWARE.
 
 #include "SystemTest.h"
+#include "InfluxDBBuilder.h"
 
 namespace influxdb::test
 {
@@ -43,4 +44,29 @@ namespace influxdb::test
             CHECK_THAT(response, ContainsSubstring(user.name));
         }
     }
+
+    TEST_CASE("Http authentication methods", "[HttpAuthTest]")
+    {
+        using Catch::Matchers::ContainsSubstring;
+
+        const auto user = getUserFromEnv();
+
+        SECTION("Authenticate through factory")
+        {
+            auto db = InfluxDBFactory::Get("http://" + (user.name + ":" + user.pass + "@") + getHostFromEnv() + ":8086?db=ignore");
+
+            const auto response = db->execute("show users");
+            CHECK_THAT(response, ContainsSubstring(user.name));
+        }
+
+        SECTION("Authenticate through builder")
+        {
+            auto db = InfluxDBBuilder::http("http://" + getHostFromEnv() + ":8086?db=ignore")
+                          .setBasicAuthentication(user.name, user.pass)
+                          .connect();
+
+            const auto response = db->execute("show users");
+            CHECK_THAT(response, ContainsSubstring(user.name));
+        }
+    }
 }

test/system/InfluxDBST.cxx

@@ -21,6 +21,7 @@
 // SOFTWARE.
 
 #include "SystemTest.h"
+#include "InfluxDBBuilder.h"
 
 namespace influxdb::test
 {
@@ -38,6 +39,27 @@ namespace influxdb::test
         }
     }
 
+    TEST_CASE("Connection configuration", "[InfluxDBST]")
+    {
+        using namespace Catch::Matchers;
+
+        SECTION("Connect through factory")
+        {
+            auto db = InfluxDBFactory::Get("http://" + getHostFromEnv() + ":8086?db=ignore");
+            const auto response = db->execute("show databases");
+            CHECK(response.empty() == false);
+        }
+
+        SECTION("Connect through builder")
+        {
+            auto db = InfluxDBBuilder::http("http://" + getHostFromEnv() + ":8086?db=ignore")
+                          .setTimeout(std::chrono::seconds{5})
+                          .connect();
+            const auto response = db->execute("show databases");
+            CHECK(response.empty() == false);
+        }
+    }
+
     TEST_CASE("InfluxDB system test", "[InfluxDBST]")
     {
         using namespace Catch::Matchers;

test/system/SystemTest.h

@@ -61,10 +61,14 @@ namespace influxdb::test
         return {*user, *pass};
     }
 
+    inline std::string getHostFromEnv()
+    {
+        return getEnv("INFLUXDBCXX_SYSTEMTEST_HOST").value_or("localhost");
+    }
+
     inline std::unique_ptr<InfluxDB> configure(const std::string& db, std::optional<User> user = {})
     {
-        const auto host = getEnv("INFLUXDBCXX_SYSTEMTEST_HOST").value_or("localhost");
         const std::string authString{user ? (user->name + ":" + user->pass + "@") : ""};
-        return InfluxDBFactory::Get("http://" + authString + host + ":8086?db=" + db);
+        return InfluxDBFactory::Get("http://" + authString + getHostFromEnv() + ":8086?db=" + db);
     }
 }