[Bf-blender-cvs] [8be3fcab7ec] master: Fix tests after recent commit.

Bastien Montagne noreply at git.blender.org
Mon Feb 6 21:17:12 CET 2023


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);



More information about the Bf-blender-cvs mailing list