tests: glTF/FBX/OBJ + point-cloud loader unit tests + coverage flush

test_unit_threedim_loaders: AssetLoader/GltfParser/FbxParser/TinyObj/Ply/Vcg
+ the PrimitiveCloud parsers, compiled straight into the test (plugin is
-fvisibility=hidden). test_unit_primitivecloud: PLY/.splat/.spz parsers +
scene helpers. ObjectGallery: explicit llvm profile flush on _Exit +
SEGV/ABRT so coverage runs of the render-regression harness still emit
.profraw (atop this branch's render-regression mode).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014rZgzE8JjWvHDtaVUhxpLE

Author: jcelerier

tests/integration/ObjectGallery.cpp

@@ -52,10 +52,46 @@
 #include <QImage>
 
 #include <algorithm>
+#include <csignal>
 #include <cstdio>
 
+// Coverage support: this harness intentionally exits via std::_Exit (to bypass a
+// gfx teardown crash) and often SEGVs on shutdown, both of which skip the normal
+// atexit profile writer. When built with -fprofile-instr-generate we flush
+// counters explicitly before _Exit and from a SEGV/ABRT handler so instrumented
+// runs still yield a .profraw. Mach-O has no ELF-style weak undefined externals
+// (the static link still requires the symbol), so resolve dynamically there.
+#if defined(__APPLE__)
+#include <dlfcn.h>
+#else
+extern "C" __attribute__((weak)) int __llvm_profile_write_file(void);
+#endif
+
 namespace
 {
+void flush_coverage()
+{
+#if defined(__APPLE__)
+  if(auto f = reinterpret_cast<int (*)(void)>(
+         dlsym(RTLD_DEFAULT, "__llvm_profile_write_file")))
+    f();
+#else
+  if(&__llvm_profile_write_file != nullptr)
+    __llvm_profile_write_file();
+#endif
+}
+void coverage_signal_handler(int sig)
+{
+  flush_coverage();
+  std::signal(sig, SIG_DFL);
+  std::raise(sig);
+}
+void install_coverage_flush()
+{
+  std::signal(SIGSEGV, coverage_signal_handler);
+  std::signal(SIGABRT, coverage_signal_handler);
+}
+
 bool matches(const QString& name, const QString& filter)
 {
   return filter.isEmpty() || name.contains(filter, Qt::CaseInsensitive);
@@ -253,6 +289,7 @@ void run_render_check(const QString& filter, int seconds)
     // crashes in the offscreen readback release path) via a hard exit. If the
     // object had crashed *during* rendering, we would never reach here — that
     // is exactly the render-path failure this check is meant to catch.
+    flush_coverage();
     std::_Exit(rc);
   });
 }
@@ -353,13 +390,15 @@ void run_shader_check(const QString& path, int seconds)
       }
     }
     std::fflush(stdout);
+    flush_coverage();
     std::_Exit(rc);
   });
 }
 }
 
 int main(int argc, char** argv)
 {
+  install_coverage_flush();
   QLocale::setDefault(QLocale::C);
   std::setlocale(LC_ALL, "C");
 

tests/unit/CMakeLists.txt

@@ -166,3 +166,65 @@ if(TARGET score_plugin_threedim)
     PLUGINS score_plugin_threedim score_plugin_gfx score_plugin_avnd
     LIBS ${QT_PREFIX}::Gui)
 endif()
+# --- threedim asset loaders ------------------------------------------------
+# The plugin is built with -fvisibility=hidden, so the loader entry points
+# (GltfParser/FbxParser/AssetLoader::process, ObjFromString, ...) are not
+# reachable through libscore_plugin_threedim.so. Compile the loader layer
+# straight into the test binary instead; linking the plugins still provides
+# the include dirs + the exported gfx/registry symbols.
+if(TARGET score_plugin_threedim)
+  set(_threedim_src "${SCORE_ROOT_SOURCE_DIR}/src/plugins/score-plugin-threedim/Threedim")
+
+  # C sources go in a separate static lib: mixing C into the test target
+  # breaks the score_lib_pch REUSE_FROM (PCH is C++-only).
+  add_library(test_unit_threedim_3rdparty STATIC
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/miniply/miniply.cpp"
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/mikktspace/mikktspace.c"
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/ufbx/ufbx.c")
+  target_include_directories(test_unit_threedim_3rdparty SYSTEM PUBLIC
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/miniply"
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/mikktspace"
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/ufbx")
+  set_target_properties(test_unit_threedim_3rdparty
+    PROPERTIES FOLDER "Tests" POSITION_INDEPENDENT_CODE ON)
+
+  score_add_test(test_unit_threedim_loaders
+    SOURCES
+      ThreedimLoaderTest.cpp
+      "${_threedim_src}/AssetLoader.cpp"
+      "${_threedim_src}/GltfParser.cpp"
+      "${_threedim_src}/FbxParser.cpp"
+      "${_threedim_src}/TinyObj.cpp"
+      "${_threedim_src}/Ply.cpp"
+      "${_threedim_src}/VcgImporters.cpp"
+      "${_threedim_src}/SceneFromMeshes.cpp"
+      "${_threedim_src}/PrimitiveCloud/PlyParser.cpp"
+      "${_threedim_src}/PrimitiveCloud/SplatBinary.cpp"
+      "${_threedim_src}/PrimitiveCloud/SpzCodec.cpp"
+      "${_threedim_src}/PrimitiveCloud/SceneFromCloud.cpp"
+      "${_threedim_src}/PrimitiveCloud/FormatOverride.cpp"
+    PLUGINS score_plugin_threedim score_plugin_gfx score_plugin_avnd
+    LIBS test_unit_threedim_3rdparty fastgltf spz "${QT_PREFIX}::Gui")
+  target_include_directories(test_unit_threedim_loaders SYSTEM PRIVATE
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/vcglib"
+    "${SCORE_ROOT_SOURCE_DIR}/3rdparty/eigen")
+endif()
+
+# PrimitiveCloud parsers (PLY / .splat / .spz + scene helpers). The parser
+# entry points are not exported from the dynamic score_plugin_threedim
+# (hidden visibility), so compile the parser translation units directly
+# into the test.
+set(_pc_dir "${SCORE_ROOT_SOURCE_DIR}/src/plugins/score-plugin-threedim/Threedim/PrimitiveCloud")
+score_add_test(test_unit_primitivecloud
+  SOURCES
+    PrimitiveCloudTest.cpp
+    "${_pc_dir}/PlyParser.cpp"
+    "${_pc_dir}/SplatBinary.cpp"
+    "${_pc_dir}/SpzCodec.cpp"
+    "${_pc_dir}/SceneFromCloud.cpp"
+    "${_pc_dir}/FormatOverride.cpp"
+    "${3RDPARTY_FOLDER}/miniply/miniply.cpp"
+  LIBS spz)
+target_include_directories(test_unit_primitivecloud SYSTEM PRIVATE
+  "${SCORE_ROOT_SOURCE_DIR}/src/plugins/score-plugin-threedim"
+  "${3RDPARTY_FOLDER}/miniply")