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

Merge branch 'dev' into f-insert

parents 113840e5 e017463f
No related branches found
No related tags found
1 merge request!3TST: Add Tests for insert and delete
Pipeline #10907 failed
This commit is part of merge request !3. Comments created here will be created in the context of that merge request.
...@@ -36,8 +36,7 @@ using caosdb::entity::MessageCode; ...@@ -36,8 +36,7 @@ using caosdb::entity::MessageCode;
class test_transaction : public ::testing::Test { class test_transaction : public ::testing::Test {
protected: protected:
void SetUp() override { void SetUp() override {}
}
void TearDown() override { void TearDown() override {
// TODO(tf): delete all created entities // TODO(tf): delete all created entities
...@@ -59,8 +58,7 @@ TEST(test_transaction, DISABLED_retrieve_manufacturer_by_id) { ...@@ -59,8 +58,7 @@ TEST(test_transaction, DISABLED_retrieve_manufacturer_by_id) {
transaction->Execute(); transaction->Execute();
const auto &result_set = const auto &result_set =
dynamic_cast<const UniqueResult &>( dynamic_cast<const UniqueResult &>(transaction->GetResultSet());
transaction->GetResultSet());
const auto &entity = result_set.GetEntity(); const auto &entity = result_set.GetEntity();
EXPECT_EQ(id, entity.GetId()); EXPECT_EQ(id, entity.GetId());
...@@ -85,8 +83,7 @@ TEST(test_transaction, retrieve_non_existing) { ...@@ -85,8 +83,7 @@ TEST(test_transaction, retrieve_non_existing) {
ASSERT_EQ(status.GetCode(), StatusCode::SUCCESS); ASSERT_EQ(status.GetCode(), StatusCode::SUCCESS);
const auto &result_set = const auto &result_set =
dynamic_cast<const UniqueResult &>( dynamic_cast<const UniqueResult &>(transaction->GetResultSet());
transaction->GetResultSet());
const auto &entity = result_set.GetEntity(); const auto &entity = result_set.GetEntity();
EXPECT_EQ(id, entity.GetId()); EXPECT_EQ(id, entity.GetId());
...@@ -114,8 +111,7 @@ TEST(test_transaction, insert_delete) { ...@@ -114,8 +111,7 @@ TEST(test_transaction, insert_delete) {
ASSERT_FALSE(insert_status.IsError()); ASSERT_FALSE(insert_status.IsError());
const auto &insert_result_set = const auto &insert_result_set =
dynamic_cast<const UniqueResult &>( dynamic_cast<const UniqueResult &>(insert_transaction->GetResultSet());
insert_transaction->GetResultSet());
const auto &new_entity = insert_result_set.GetEntity(); const auto &new_entity = insert_result_set.GetEntity();
EXPECT_FALSE(new_entity.GetId().empty()); EXPECT_FALSE(new_entity.GetId().empty());
...@@ -132,13 +128,11 @@ TEST(test_transaction, insert_delete) { ...@@ -132,13 +128,11 @@ TEST(test_transaction, insert_delete) {
ASSERT_FALSE(delete_status.IsError()); ASSERT_FALSE(delete_status.IsError());
const auto &delete_result_set = const auto &delete_result_set =
dynamic_cast<const UniqueResult &>( dynamic_cast<const UniqueResult &>(delete_transaction->GetResultSet());
delete_transaction->GetResultSet());
const auto &deleted_entity = delete_result_set.GetEntity(); const auto &deleted_entity = delete_result_set.GetEntity();
EXPECT_EQ(deleted_entity.GetId(), new_entity.GetId()); EXPECT_EQ(deleted_entity.GetId(), new_entity.GetId());
EXPECT_FALSE(deleted_entity.HasErrors()); EXPECT_FALSE(deleted_entity.HasErrors());
} }
} // namespace caosdb::transaction } // namespace caosdb::transaction
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment