WIP TEST: Wrote test which did not fail before.

Probably there should be a similar test for ccaosdb.

include/caosdb/utility.h

@@ -91,12 +91,13 @@ auto getEnumValueFromName<caosdb::entity::Role>(const std::string &name) -> caos
  */
 inline auto load_string_file(const path &path) -> std::string {
   std::string result;
-  try {
+  // TODO(htw)
+  // try {
     boost::filesystem::load_string_file(path, result);
-  } catch (const std::exception &exc) {
-    throw caosdb::exceptions::Exception(StatusCode::FILE_CANNOT_BE_READ ,
-                                          "Cannot read the following file: "+ path.string());
-  }
+  // } catch (const std::exception &exc) {
+  //   throw caosdb::exceptions::Exception(StatusCode::FILE_CANNOT_BE_READ ,
+  //                                         "Cannot read the following file: "+ path.string());
+  // }
   return result;
 }
 

src/caosdb/connection.cpp

@@ -109,8 +109,8 @@ auto ConnectionManager::mGetConnection(const std::string &name) const
       auto connection = ConfigurationManager::GetConnectionConfiguration(name);
       connections[name] = std::make_shared<Connection>(*connection.release());
     } catch (const caosdb::exceptions::ConfigurationError &exc) {
-      throw caosdb::exceptions::UnknownConnectionError("No connection named '" + name +
-                                                       "' present.");
+      throw caosdb::exceptions::UnknownConnectionError("Error with the connection named '" + name +
+                                                       "': " + exc.what());
     }
   }
   return this->connections.at(name);

src/ccaosdb.cpp

@@ -185,6 +185,7 @@ ERROR_RETURN_CODE(GENERIC_ERROR,
                     if (out->_deletable) {
                       return caosdb::StatusCode::EXTERN_C_ASSIGNMENT_ERROR;
                     }
+                    // TODO(htw) eitehr check for file existence or call CreateCertificateProvider?
                     out->wrapped_certificate_provider =
                       new caosdb::configuration::PemFileCertificateProvider(std::string(path));
                     out->_deletable = true;

test/test_connection.cpp

@@ -65,9 +65,16 @@ TEST_F(test_connection, configure_ssl_localhost_8080) {
 }
 
 TEST_F(test_connection, connection_manager_unknown_connection) {
-  EXPECT_THROW_MESSAGE(ConnectionManager::GetConnection("test"),
-                       caosdb::exceptions::UnknownConnectionError,
-                       "No connection named 'test' present.");
+  EXPECT_THROW_MESSAGE(
+      ConnectionManager::GetConnection("test"), caosdb::exceptions::UnknownConnectionError,
+      "Error with the connection named 'test': The connection 'test' has not been defined.");
+}
+
+TEST_F(test_connection, connection_missing_certificate) {
+  EXPECT_THROW_MESSAGE(
+      ConnectionManager::GetConnection("missing"), caosdb::exceptions::UnknownConnectionError,
+      std::string("Error with the connection named 'missing': ")
+      + "File does not exist (server_certificate_path): /missing");
 }
 
 TEST_F(test_connection, connection_manager_get_default_connection) {

test/test_data/test_caosdb_client.json

@@ -18,6 +18,16 @@
         "username": "me",
           "password": "secret!"
       }
+    },
+    "missing": {
+      "host": "localhost",
+      "port": 8443,
+      "server_certificate_path": "/missing",
+      "authentication": {
+        "type": "plain",
+        "username": "me",
+        "password": "secret!"
+      }
     }
   },
   "logging": {