From 126c4ed460231b88f8fb776b37138e3192881d41 Mon Sep 17 00:00:00 2001 From: godofecht Date: Mon, 16 Mar 2026 09:15:03 +0000 Subject: [PATCH] fix: disable slow tests and add 120s timeout for CI Phase8PhysicsInformedTest takes 16+ minutes on CI runners, making the pipeline take 20+ minutes total. Disable it for CI (can be run locally). Add --timeout 120 to ctest so no single test can stall the pipeline. Test timing summary (from last green run): - Phase8PhysicsInformedTest: 971s (DISABLED) - Phase8PhysicsComprehensiveTest: (already disabled - flaky) - Phase12ReinforcementTest: (already disabled - segfault) - All remaining tests: <25s each, ~90s total Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/ci.yml | 2 +- CMakeLists.txt | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 6bf815f..9b45bdf 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -36,4 +36,4 @@ jobs: run: cmake --build build - name: Run tests - run: cd build && ctest --output-on-failure + run: cd build && ctest --output-on-failure --timeout 120 diff --git a/CMakeLists.txt b/CMakeLists.txt index 5e4e0fe..4783c66 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -282,6 +282,8 @@ else() endif() endif() add_test(NAME Phase8PhysicsInformedTest COMMAND Phase8PhysicsInformedTest) +# Takes 16+ minutes on CI - too slow for regular CI runs +set_tests_properties(Phase8PhysicsInformedTest PROPERTIES DISABLED true) # Phase 8 Comprehensive Physics-Informed Neural Networks Test executable add_executable(Phase8PhysicsComprehensiveTest tests/test_phase8_physics_comprehensive.cpp)