diff --git a/test/caosdb_test_utility.h b/test/caosdb_test_utility.h
index 4cde3dc5059acf60a04b071154b44a59b67e9833..4c97c203a8ab55c9d239d824cd274fe3b99aa60b 100644
--- a/test/caosdb_test_utility.h
+++ b/test/caosdb_test_utility.h
@@ -33,16 +33,16 @@
*/
#define EXPECT_THROW_MESSAGE(statement, exeption_type, message) \
EXPECT_THROW( \
- try { statement; } catch (const exeption_type &e) { \
- EXPECT_EQ(std::string(e.what()), message); \
- throw; \
- }, \
- exeption_type)
+ try { statement; } catch (const exeption_type &e) { \
+ EXPECT_EQ(std::string(e.what()), message); \
+ throw; \
+ }, \
+ exeption_type)
#define ASSERT_THROW_MESSAGE(statement, exeption_type, message) \
ASSERT_THROW( \
- try { statement; } catch (const exeption_type &e) { \
- ASSERT_EQ(std::string(e.what()), message); \
- throw; \
- }, \
- exeption_type)
+ try { statement; } catch (const exeption_type &e) { \
+ ASSERT_EQ(std::string(e.what()), message); \
+ throw; \
+ }, \
+ exeption_type)
#endif
diff --git a/test/test_ccaosdb.cpp b/test/test_ccaosdb.cpp
index 6f8eb10d78561f0eb493ff01f592f6a1fcfed3c6..859f7d392d1a4561a7a74c1273950b154091cd7f 100644
--- a/test/test_ccaosdb.cpp
+++ b/test/test_ccaosdb.cpp
@@ -37,7 +37,7 @@ TEST(test_ccaosdb, connection_ssl_authentication_success) {
auto major = caosdb_constants_COMPATIBLE_SERVER_VERSION_MAJOR();
auto minor = caosdb_constants_COMPATIBLE_SERVER_VERSION_MINOR();
const auto *const pre_release =
- caosdb_constants_COMPATIBLE_SERVER_VERSION_PRE_RELEASE();
+ caosdb_constants_COMPATIBLE_SERVER_VERSION_PRE_RELEASE();
EXPECT_EQ(major, version_info.major);
EXPECT_EQ(minor, version_info.minor);
@@ -48,12 +48,12 @@ TEST(test_ccaosdb, test_count_query) {
caosdb_connection_connection connection;
int return_code(
- caosdb_connection_connection_manager_get_default_connection(&connection));
+ caosdb_connection_connection_manager_get_default_connection(&connection));
EXPECT_EQ(return_code, 0);
caosdb_transaction_transaction transaction;
- return_code = caosdb_connection_connection_create_transaction(&connection,
- &transaction);
+ return_code =
+ caosdb_connection_connection_create_transaction(&connection, &transaction);
EXPECT_EQ(return_code, 0);
// COUNT query with an empty result
@@ -67,7 +67,7 @@ TEST(test_ccaosdb, test_count_query) {
long dummy(-1); // NOLINT
long *count = &dummy; // NOLINT
return_code =
- caosdb_transaction_transaction_get_count_result(&transaction, count);
+ caosdb_transaction_transaction_get_count_result(&transaction, count);
EXPECT_EQ(return_code, 0);
EXPECT_EQ(*count, 0);
@@ -78,12 +78,12 @@ TEST(test_ccaosdb, test_count_query) {
TEST(test_ccaosdb, test_query) {
caosdb_connection_connection connection;
int return_code(
- caosdb_connection_connection_manager_get_default_connection(&connection));
+ caosdb_connection_connection_manager_get_default_connection(&connection));
EXPECT_EQ(return_code, 0);
caosdb_transaction_transaction transaction;
- return_code = caosdb_connection_connection_create_transaction(&connection,
- &transaction);
+ return_code =
+ caosdb_connection_connection_create_transaction(&connection, &transaction);
EXPECT_EQ(return_code, 0);
return_code = caosdb_transaction_transaction_query(&transaction,
@@ -95,7 +95,7 @@ TEST(test_ccaosdb, test_query) {
caosdb_transaction_result_set result_set;
return_code =
- caosdb_transaction_transaction_get_result_set(&transaction, &result_set);
+ caosdb_transaction_transaction_get_result_set(&transaction, &result_set);
EXPECT_EQ(return_code, 0);
int dummy(-1);
@@ -111,16 +111,16 @@ TEST(test_ccaosdb, test_query) {
TEST(test_ccaosdb, test_single_id_retrieve) {
caosdb_connection_connection connection;
int return_code(
- caosdb_connection_connection_manager_get_default_connection(&connection));
+ caosdb_connection_connection_manager_get_default_connection(&connection));
EXPECT_EQ(return_code, 0);
caosdb_transaction_transaction transaction;
- return_code = caosdb_connection_connection_create_transaction(&connection,
- &transaction);
+ return_code =
+ caosdb_connection_connection_create_transaction(&connection, &transaction);
EXPECT_EQ(return_code, 0);
return_code =
- caosdb_transaction_transaction_retrieve_by_id(&transaction, "21");
+ caosdb_transaction_transaction_retrieve_by_id(&transaction, "21");
EXPECT_EQ(return_code, caosdb::StatusCode::GO_ON);
return_code = caosdb_transaction_transaction_execute(&transaction);
@@ -128,7 +128,7 @@ TEST(test_ccaosdb, test_single_id_retrieve) {
caosdb_transaction_result_set result_set;
return_code =
- caosdb_transaction_transaction_get_result_set(&transaction, &result_set);
+ caosdb_transaction_transaction_get_result_set(&transaction, &result_set);
EXPECT_EQ(return_code, 0);
int dummy(-1);
@@ -145,8 +145,8 @@ TEST(test_ccaosdb, test_single_id_retrieve) {
EXPECT_EQ(return_code, 0);
EXPECT_EQ(*count, 0);
- char out[255] = {"255"}; // NOLINT
- return_code = caosdb_entity_entity_get_name(&entity, out);
+ char *out = nullptr; // NOLINT
+ return_code = caosdb_entity_entity_get_name(&entity, &out);
EXPECT_EQ(return_code, 0);
EXPECT_STREQ(out, "unit");
@@ -157,17 +157,17 @@ TEST(test_ccaosdb, test_single_id_retrieve) {
TEST(test_ccaosdb, test_multi_id_retrieve) {
caosdb_connection_connection connection;
int return_code(
- caosdb_connection_connection_manager_get_default_connection(&connection));
+ caosdb_connection_connection_manager_get_default_connection(&connection));
EXPECT_EQ(return_code, 0);
caosdb_transaction_transaction transaction;
- return_code = caosdb_connection_connection_create_transaction(&connection,
- &transaction);
+ return_code =
+ caosdb_connection_connection_create_transaction(&connection, &transaction);
EXPECT_EQ(return_code, 0);
const char *ids[] = {"20", "21", "22"}; // NOLINT
return_code =
- caosdb_transaction_transaction_retrieve_by_ids(&transaction, ids, 3);
+ caosdb_transaction_transaction_retrieve_by_ids(&transaction, ids, 3);
EXPECT_EQ(return_code, caosdb::StatusCode::GO_ON);
return_code = caosdb_transaction_transaction_execute(&transaction);
@@ -176,7 +176,7 @@ TEST(test_ccaosdb, test_multi_id_retrieve) {
caosdb_transaction_result_set result_set;
return_code =
- caosdb_transaction_transaction_get_result_set(&transaction, &result_set);
+ caosdb_transaction_transaction_get_result_set(&transaction, &result_set);
EXPECT_EQ(return_code, 0);
int dummy(-1);
@@ -193,8 +193,8 @@ TEST(test_ccaosdb, test_multi_id_retrieve) {
EXPECT_EQ(return_code, 0);
EXPECT_EQ(*count, 0);
- char out[255] = {"255"}; // NOLINT
- return_code = caosdb_entity_entity_get_name(&entity, out);
+ char *out = nullptr; // NOLINT
+ return_code = caosdb_entity_entity_get_name(&entity, &out);
EXPECT_EQ(return_code, 0);
EXPECT_STREQ(out, "unit");
diff --git a/test/test_connection.cpp b/test/test_connection.cpp
index 6f27f23f40da242a47ac4b818783ad04aeaf72bf..678035b50b4fee53cc246a7335ae5fb74bfb1eca 100644
--- a/test/test_connection.cpp
+++ b/test/test_connection.cpp
@@ -62,10 +62,10 @@ TEST(test_connection, connect_somehost_42347_fails) {
TEST(test_connection, connection_insecure_authentication_error_anonymous) {
const auto *port_str =
- caosdb::utility::get_env_var("CAOSDB_SERVER_GRPC_PORT_HTTP", "8080");
+ caosdb::utility::get_env_var("CAOSDB_SERVER_GRPC_PORT_HTTP", "8080");
auto port = std::stoi(port_str);
const auto *const host =
- caosdb::utility::get_env_var("CAOSDB_SERVER_HOST", "localhost");
+ caosdb::utility::get_env_var("CAOSDB_SERVER_HOST", "localhost");
auto config = InsecureConnectionConfiguration(host, port);
auto connection = Connection(config);
@@ -75,12 +75,12 @@ TEST(test_connection, connection_insecure_authentication_error_anonymous) {
TEST(test_connection, connection_ssl_authentication_error_anonymous) {
const auto *port_str =
- caosdb::utility::get_env_var("CAOSDB_SERVER_GRPC_PORT_HTTPS", "8443");
+ caosdb::utility::get_env_var("CAOSDB_SERVER_GRPC_PORT_HTTPS", "8443");
auto port = std::stoi(port_str);
const auto *const host =
- caosdb::utility::get_env_var("CAOSDB_SERVER_HOST", "localhost");
+ caosdb::utility::get_env_var("CAOSDB_SERVER_HOST", "localhost");
const auto path =
- caosdb::utility::get_env_var("CAOSDB_SERVER_CERT", std::string());
+ caosdb::utility::get_env_var("CAOSDB_SERVER_CERT", std::string());
auto cert = PemFileCertificateProvider(path);
auto config = TlsConnectionConfiguration(host, port, cert);
@@ -94,12 +94,12 @@ TEST(test_connection, connection_ssl_authentication_error_anonymous) {
TEST(test_connection, connection_ssl_authentication_error_wrong_credentials) {
const auto *port_str =
- caosdb::utility::get_env_var("CAOSDB_SERVER_GRPC_PORT_HTTPS", "8443");
+ caosdb::utility::get_env_var("CAOSDB_SERVER_GRPC_PORT_HTTPS", "8443");
auto port = std::stoi(port_str);
const auto *const host =
- caosdb::utility::get_env_var("CAOSDB_SERVER_HOST", "localhost");
+ caosdb::utility::get_env_var("CAOSDB_SERVER_HOST", "localhost");
const auto path =
- caosdb::utility::get_env_var("CAOSDB_SERVER_CERT", std::string());
+ caosdb::utility::get_env_var("CAOSDB_SERVER_CERT", std::string());
const auto *const user = "not-a-user-23461237";
const auto *const password = "very-c-cred";
@@ -109,10 +109,10 @@ TEST(test_connection, connection_ssl_authentication_error_wrong_credentials) {
auto connection = Connection(config);
EXPECT_THROW_MESSAGE(
- connection.RetrieveVersionInfo(), AuthenticationError,
- "The attempt to execute this transaction has not been executed at all "
- "because the authentication did not succeed. Original error: "
- "Authentication failed. Username or password wrong.");
+ connection.RetrieveVersionInfo(), AuthenticationError,
+ "The attempt to execute this transaction has not been executed at all "
+ "because the authentication did not succeed. Original error: "
+ "Authentication failed. Username or password wrong.");
}
TEST(test_connection, connection_ssl_authentication_success) {
@@ -121,7 +121,7 @@ TEST(test_connection, connection_ssl_authentication_success) {
auto major = caosdb::COMPATIBLE_SERVER_VERSION_MAJOR;
auto minor = caosdb::COMPATIBLE_SERVER_VERSION_MINOR;
const auto pre_release =
- std::string(caosdb::COMPATIBLE_SERVER_VERSION_PRE_RELEASE);
+ std::string(caosdb::COMPATIBLE_SERVER_VERSION_PRE_RELEASE);
const auto &v_info = connection->RetrieveVersionInfo();
EXPECT_EQ(major, v_info.GetMajor());
diff --git a/test/test_list_properties.cpp b/test/test_list_properties.cpp
index fdbd3524dd7195285007e992c22504cdb11f02a1..83581bcc8438d513e44a5d46536d3000af923920 100644
--- a/test/test_list_properties.cpp
+++ b/test/test_list_properties.cpp
@@ -40,7 +40,7 @@ protected:
void TearDown() override {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto query_transaction(connection->CreateTransaction());
query_transaction->Query("FIND ENTITY WITH id > 99");
query_transaction->Execute();
@@ -56,7 +56,7 @@ protected:
TEST_F(test_list_properties, insert_list_of_text) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto insertion_prop(connection->CreateTransaction());
@@ -65,7 +65,7 @@ TEST_F(test_list_properties, insert_list_of_text) {
abstract_list_property.SetName("TestProp");
abstract_list_property.SetDataType(DataType::ListOf(AtomicDataType::TEXT));
abstract_list_property.SetValue(
- std::vector<std::string>{"item1", "item2", "item3"});
+ std::vector<std::string>{"item1", "item2", "item3"});
insertion_prop->InsertEntity(&abstract_list_property);
std::cout << "response " << insertion_prop->ResponseToString();
diff --git a/test/test_transaction.cpp b/test/test_transaction.cpp
index 777a1a48c7817f01fa021deb27516873a04881fa..0648c13af45ae098e20808cfece5a2290b9c6f66 100644
--- a/test/test_transaction.cpp
+++ b/test/test_transaction.cpp
@@ -72,7 +72,7 @@ protected:
// delete all entities
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto query_transaction(connection->CreateTransaction());
query_transaction->Query("FIND ENTITY WITH id > 99");
query_transaction->Execute();
@@ -94,7 +94,7 @@ protected:
*/
TEST_F(test_transaction, retrieve_non_existing) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto transaction(connection->CreateTransaction());
@@ -124,7 +124,7 @@ TEST_F(test_transaction, retrieve_non_existing) {
*/
TEST_F(test_transaction, insert_without_delete) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto insert_transaction(connection->CreateTransaction());
@@ -159,7 +159,7 @@ TEST_F(test_transaction, insert_delete) {
// same as in insert_without_delete
// until marked END SAME
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto insert_transaction(connection->CreateTransaction());
@@ -204,7 +204,7 @@ TEST_F(test_transaction, insert_delete_with_parent) {
// same as in insert_without_delete
// until marked END SAME
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto insert_transaction(connection->CreateTransaction());
@@ -299,7 +299,7 @@ TEST_F(test_transaction, insert_delete_with_parent) {
*/
TEST_F(test_transaction, insert_delete_with_property) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
// Create and insert property
Entity prop_ent;
@@ -435,7 +435,7 @@ TEST_F(test_transaction, insert_delete_with_property) {
*/
TEST_F(test_transaction, test_multi_retrieve) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto transaction(connection->CreateTransaction());
@@ -469,7 +469,7 @@ TEST_F(test_transaction, test_multi_retrieve) {
*/
TEST_F(test_transaction, insert_update_delete) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
// INSERT
auto insert_transaction(connection->CreateTransaction());
@@ -537,7 +537,7 @@ TEST_F(test_transaction, insert_update_delete) {
*/
TEST_F(test_transaction, test_query) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
auto insert_transaction(connection->CreateTransaction());
@@ -583,7 +583,7 @@ TEST_F(test_transaction, test_query) {
*/
TEST_F(test_transaction, test_query_with_retrieve) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
// rt1
Entity rt1;
@@ -698,7 +698,7 @@ TEST_F(test_transaction, test_query_with_retrieve) {
*/
TEST_F(test_transaction, test_file_upload) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
Entity file;
file.SetRole(Role::FILE);
@@ -735,7 +735,7 @@ TEST_F(test_transaction, test_file_upload) {
*/
TEST_F(test_transaction, test_file_up_n_download) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
Entity file;
file.SetRole(Role::FILE);
@@ -754,7 +754,7 @@ TEST_F(test_transaction, test_file_up_n_download) {
auto download_transaction(connection->CreateTransaction());
download_transaction->RetrieveAndDownloadFilesById(
- inserted_file.GetId(), test_download_file_1.string());
+ inserted_file.GetId(), test_download_file_1.string());
download_transaction->ExecuteAsynchronously();
ASSERT_EQ(download_transaction->WaitForIt().GetCode(), StatusCode::SUCCESS);
@@ -781,7 +781,7 @@ TEST_F(test_transaction, test_file_up_n_download) {
*/
TEST_F(test_transaction, test_full_workflow) {
const auto &connection =
- caosdb::connection::ConnectionManager::GetDefaultConnection();
+ caosdb::connection::ConnectionManager::GetDefaultConnection();
// ###### Create Data Model ######
Entity dataRT;