[Bf-blender-cvs] [5f9b3950ebd] experimental-build: Revert test changes to `lib_id_test.cc`.
Bastien Montagne
noreply at git.blender.org
Fri Jun 11 14:12:03 CEST 2021
Commit: 5f9b3950ebd73523cf19afa1b8112589777d8aa7
Author: Bastien Montagne
Date: Fri Jun 11 14:11:28 2021 +0200
Branches: experimental-build
https://developer.blender.org/rB5f9b3950ebd73523cf19afa1b8112589777d8aa7
Revert test changes to `lib_id_test.cc`.
Reset this file to master state.
===================================================================
M source/blender/blenkernel/intern/lib_id_test.cc
===================================================================
diff --git a/source/blender/blenkernel/intern/lib_id_test.cc b/source/blender/blenkernel/intern/lib_id_test.cc
index 8a4839afe44..8e21ae88aa6 100644
--- a/source/blender/blenkernel/intern/lib_id_test.cc
+++ b/source/blender/blenkernel/intern/lib_id_test.cc
@@ -47,7 +47,6 @@ static void test_lib_id_main_sort_free(LibIDMainSortTestContext *ctx)
BKE_main_free(ctx->bmain);
}
-#if 0
static void test_lib_id_main_sort_check_order(std::initializer_list<ID *> list)
{
ID *prev_id = nullptr;
@@ -60,17 +59,6 @@ static void test_lib_id_main_sort_check_order(std::initializer_list<ID *> list)
}
EXPECT_EQ(prev_id->next, nullptr);
}
-#else
-static void test_lib_id_main_sort_check_order(ID *id_a, ID *id_b, ID *id_c)
-{
- EXPECT_EQ(id_a->prev, nullptr);
- EXPECT_EQ(id_a->next, id_b);
- EXPECT_EQ(id_b->prev, id_a);
- EXPECT_EQ(id_b->next, id_c);
- EXPECT_EQ(id_c->prev, id_b);
- EXPECT_EQ(id_c->next, nullptr);
-}
-#endif
TEST(lib_id_main_sort, local_ids_1)
{
@@ -83,7 +71,7 @@ TEST(lib_id_main_sort, local_ids_1)
ID *id_b = static_cast<ID *>(BKE_id_new(ctx.bmain, ID_OB, "OB_B"));
EXPECT_TRUE(ctx.bmain->objects.first == id_a);
EXPECT_TRUE(ctx.bmain->objects.last == id_c);
- test_lib_id_main_sort_check_order(id_a, id_b, id_c);
+ test_lib_id_main_sort_check_order({id_a, id_b, id_c});
test_lib_id_main_sort_free(&ctx);
}
@@ -106,19 +94,19 @@ TEST(lib_id_main_sort, linked_ids_1)
id_sort_by_name(&ctx.bmain->objects, id_b, nullptr);
EXPECT_TRUE(ctx.bmain->objects.first == id_c);
EXPECT_TRUE(ctx.bmain->objects.last == id_b);
- test_lib_id_main_sort_check_order(id_c, id_a, id_b);
+ test_lib_id_main_sort_check_order({id_c, id_a, id_b});
id_a->lib = lib_b;
id_sort_by_name(&ctx.bmain->objects, id_a, nullptr);
EXPECT_TRUE(ctx.bmain->objects.first == id_c);
EXPECT_TRUE(ctx.bmain->objects.last == id_a);
- test_lib_id_main_sort_check_order(id_c, id_b, id_a);
+ test_lib_id_main_sort_check_order({id_c, id_b, id_a});
id_b->lib = lib_b;
id_sort_by_name(&ctx.bmain->objects, id_b, nullptr);
EXPECT_TRUE(ctx.bmain->objects.first == id_c);
EXPECT_TRUE(ctx.bmain->objects.last == id_b);
- test_lib_id_main_sort_check_order(id_c, id_a, id_b);
+ test_lib_id_main_sort_check_order({id_c, id_a, id_b});
test_lib_id_main_sort_free(&ctx);
}
More information about the Bf-blender-cvs
mailing list