dillongoostudios/goo-engine
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Tests: only disable big endian checks on macOS/x64"

Browse Source 
This reverts commit 9786d7603a.

While the test passed when I checked, it's unreliable enough to keep
disabled.
This commit is contained in:
Campbell Barton
2024-09-05 20:37:02 +10:00
parent 093192a9d0
commit 352c69f437
1 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/python/bl_blendfile_versioning.py
+11 -8
View File
@@ -49,14 +49,17 @@ class TestBlendFileOpenAllTestFiles(TestHelper):
self.excluded_dirs = ()
if platform.system() == "Darwin":
if platform.machine() == "x86_64":
self.excluded_dirs = (
*self.excluded_dirs,
# The assert in `BKE_libblock_alloc_in_lib` often fails:
# `BLI_assert(bmain->is_locked_for_linking == false || ELEM(type, ID_WS, ID_GR, ID_NT))`.
# This needs to be investigated.
"io_tests/blend_big_endian/",
)
# NOTE(@ideasman42):
# - `x86_64` fails often.
# - `arm64` can succeed but is unreliable enough to keep disabled.
# Keep both disabled.
self.excluded_dirs = (
*self.excluded_dirs,
# The assert in `BKE_libblock_alloc_in_lib` often fails:
# `BLI_assert(bmain->is_locked_for_linking == false || ELEM(type, ID_WS, ID_GR, ID_NT))`.
# This needs to be investigated.
"io_tests/blend_big_endian/",
)
assert all(p.endswith("/") for p in self.excluded_dirs)
self.excluded_dirs = tuple(p.replace("/", os.sep) for p in self.excluded_dirs)