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

TST: add test for string id in a transaction

parent f9c94085
Branches
Tags
1 merge request!17F mixed write transactions
Pipeline #14222 passed
......@@ -1271,4 +1271,26 @@ TEST_F(test_transaction, test_full_workflow) {
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
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment