Skip to content
Snippets Groups Projects
Verified Commit ea46e0e7 authored by Timm Fitschen's avatar Timm Fitschen
Browse files

TST: update tests for string ids

parent c6b02f51
No related branches found
No related tags found
1 merge request!31Draft: Update tests for string ids
Pipeline #41736 failed
...@@ -363,9 +363,7 @@ TEST_F(test_select, test_select_property_unit) { ...@@ -363,9 +363,7 @@ TEST_F(test_select, test_select_property_unit) {
EXPECT_EQ(query_transaction->GetResultTable().size(), 1); EXPECT_EQ(query_transaction->GetResultTable().size(), 1);
for (const auto &row : query_transaction->GetResultTable().GetRows()) { for (const auto &row : query_transaction->GetResultTable().GetRows()) {
// Fails due to https://gitlab.indiscale.com/caosdb/src/caosdb-server/-/issues/264 EXPECT_EQ(row.GetValue("TestPropDouble.unit").GetAsString(), "m");
EXPECT_NONFATAL_FAILURE({ EXPECT_EQ(row.GetValue("TestPropDouble.unit").GetAsString(), "m"); },
"TestPropDouble.unit");
} }
} }
......
...@@ -1328,25 +1328,4 @@ TEST_F(test_transaction, test_full_workflow) { ...@@ -1328,25 +1328,4 @@ TEST_F(test_transaction, test_full_workflow) {
EXPECT_EQ(retr_transaction->GetResultSet().size(), 1); EXPECT_EQ(retr_transaction->GetResultSet().size(), 1);
} }
TEST_F(test_transaction, test_delete_string_id) {
const auto &connection = caosdb::connection::ConnectionManager::GetDefaultConnection();
auto transaction(connection->CreateTransaction());
transaction->DeleteById("20");
transaction->DeleteById("string");
transaction->DeleteById("21");
transaction->ExecuteAsynchronously();
transaction->WaitForIt();
EXPECT_TRUE(transaction->GetStatus().IsTerminated());
EXPECT_TRUE(transaction->GetStatus().IsError());
const auto &results = transaction->GetResultSet();
EXPECT_FALSE(results.at(0).HasErrors());
EXPECT_TRUE(results.at(1).HasErrors());
EXPECT_EQ(results.at(1).GetErrors().size(), 1);
EXPECT_EQ(results.at(1).GetErrors().at(0).GetDescription(), "Entity does not exist.");
EXPECT_FALSE(results.at(2).HasErrors());
}
} // namespace caosdb::transaction } // namespace caosdb::transaction
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment