Minimal c interface

Summary

See https://gitlab.indiscale.com/caosdb/customers/lfpb/management/-/issues/404

• Minimal C Interface
• Configuration via json files
• Renaming of many classes and functions (In order to comply with best-practices as formulated by https://google.github.io/styleguide/cppguide.html)

Integration tests are in caosdb-cppinttest!1 (merged)

Focus

• Please take a look at the c-interface functions:
  • I the naming consistent?
  • I see one problem regarding the destructors: When a client uses caosdb_connection_connection_manager_get_default_connection to get the default connection, the destructor caosdb_connection_delete_connection should not be called, because the connection object is actually owned by the ConnectionManager. Maybe, we should prevent the deletion of the wrapped connection somehow, when the connection is owned by the ConnectionManager?
• ConfigurationManager and ConnectionManager:
  • These two singletons are tightly coupled. When the ConfigurationManager is being reset, also the ConnectionManager resets. Is this a good behavior?
• There is currently no error/exception handling in the c-interface. I will add this in https://gitlab.indiscale.com/caosdb/customers/lfpb/management/-/issues/390

Test Environment

Please try and setup a client (e.g. the ccaosdbcli) with a json configuration file.

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

• All automated tests pass
• Reference related Issues
• Up-to-date CHANGELOG.md
• Annotations in code (Gitlab comments)
  • Intent of new code
  • Problems with old code
  • Why this implementation?

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md
• The test environment setup works and the intended behavior is reproducible in the test environment
• In-code documentation and comments are up-to-date.
• Check: Are there spezifications? Are they satisfied?

For further good practices have a look at our review guidelines.

test/test_connection.cpp

@@ -23,30 +23,59 @@
 #include <gtest/gtest-message.h>   // for Message
 #include <gtest/gtest-test-part.h> // for TestPartResult, SuiteApiResolver
 #include <memory>                  // for allocator, operator!=, shared_ptr
-#include "caosdb/connection.h"     // for PemCACertProvider, InsecureCaosDB...
-#include "gtest/gtest_pred_impl.h" // for Test, AssertionResult, EXPECT_EQ
+#include <string>                  // for operator+, string
+#include "caosdb/configuration.h"  // for ConfigurationManager
+#include "caosdb/connection.h"     // for ConnectionManager, InsecureConnec...
+#include "caosdb/exceptions.h"     // for UnknownConnectionError
+#include "caosdb_test_utility.h"   // for EXPECT_THROW_MESSAGE, TEST_DATA_DIR
+#include "gtest/gtest_pred_impl.h" // for Test, AssertionResult, TestInfo
 
 namespace caosdb::connection {
+using caosdb::configuration::ConfigurationManager;
 
-TEST(test_connection, configure_insecure_localhost_8080) {
-  InsecureCaosDBConnectionConfig config("localhost", 8000);
+class test_connection : public ::testing::Test {
+protected:
+  void SetUp() override {
+    ConfigurationManager::Clear();
+    ConfigurationManager::LoadSingleJSONConfiguration(
+      TEST_DATA_DIR + "/test_caosdb_client.json");
+  };
+  void TearDown() override { ConfigurationManager::Clear(); };
+};
 
-  EXPECT_EQ("localhost", config.getHost());
-  EXPECT_EQ(8000, config.getPort());
-  std::shared_ptr<grpc::ChannelCredentials> icc =
-    config.getChannelCredentials();
+TEST_F(test_connection, configure_insecure_localhost_8080) {
+  InsecureConnectionConfiguration configuration("localhost", 8000);
+
+  EXPECT_EQ("localhost", configuration.GetHost());
+  EXPECT_EQ(8000, configuration.GetPort());
+  auto icc = configuration.GetChannelCredentials();
   EXPECT_TRUE(icc != nullptr);
 }
 
-TEST(test_connection, configure_ssl_localhost_8080) {
-  auto cacert = std::make_shared<PemCACertProvider>("ca chain");
-  SslCaosDBConnectionConfig config("localhost", 44300, cacert);
+TEST_F(test_connection, configure_ssl_localhost_8080) {
+  auto cacert = PemCertificateProvider("ca chain");
+  TlsConnectionConfiguration configuration("localhost", 44300, cacert);
 
-  EXPECT_EQ("localhost", config.getHost());
-  EXPECT_EQ(44300, config.getPort());
-  std::shared_ptr<grpc::ChannelCredentials> sslcc =
-    config.getChannelCredentials();
+  EXPECT_EQ("localhost", configuration.GetHost());
+  EXPECT_EQ(44300, configuration.GetPort());
+  auto sslcc = configuration.GetChannelCredentials();
   EXPECT_TRUE(sslcc != nullptr);
 }
 
+TEST_F(test_connection, connection_manager_unknown_connection) {
+  EXPECT_THROW_MESSAGE(ConnectionManager::GetConnection("test"),
+                       caosdb::exceptions::UnknownConnectionError,
+                       "No connection named 'test' present.");
+}
+
+TEST_F(test_connection, connection_manager_get_default_connection) {
+  auto connection = ConnectionManager::GetDefaultConnection();
+  EXPECT_EQ(connection, ConnectionManager::GetConnection("local-caosdb"));
+}
+
+TEST_F(test_connection, connection_manager_get_connection) {
+
+  EXPECT_TRUE(ConnectionManager::GetConnection("local-caosdb-admin"));
+}
+
 } // namespace caosdb::connection