Commit: 8be3fcab7ec9fdd1b74909b8065eea613f1e2c5d Author: Bastien Montagne Date: Mon Feb 6 21:14:20 2023 +0100 Branches: master https://developer.blender.org/rB8be3fcab7ec9fdd1b74909b8065eea613f1e2c5d
Fix tests after recent commit. `9c14039a8f4b5f` broke blenlib tests in release builds, due to how `EXPECT_BLI_ASSERT` works (in release builds it just calls the given function, so if that crashes then the test fails). For now remove that check in the test. =================================================================== M source/blender/blenlib/tests/BLI_listbase_test.cc =================================================================== diff --git a/source/blender/blenlib/tests/BLI_listbase_test.cc b/source/blender/blenlib/tests/BLI_listbase_test.cc index c5a2904505c..74d69373c7b 100644 --- a/source/blender/blenlib/tests/BLI_listbase_test.cc +++ b/source/blender/blenlib/tests/BLI_listbase_test.cc @@ -200,7 +200,6 @@ TEST(listbase, SplitAfter) BLI_listbase_split_after(&lb, &split_after_lb, nullptr); EXPECT_EQ(BLI_listbase_is_empty(&split_after_lb), true); - EXPECT_BLI_ASSERT(BLI_listbase_split_after(&lb, &split_after_lb, link1), ""); /* One link */ BLI_listbase_clear(&lb); _______________________________________________ Bf-blender-cvs mailing list Bf-blender-cvs@blender.org List details, subscription details or unsubscribe: https://lists.blender.org/mailman/listinfo/bf-blender-cvs