From 5527afc5218d71893f86755720840ac17bf34f05 Mon Sep 17 00:00:00 2001 From: AztecBot Date: Fri, 10 Apr 2026 09:41:39 +0000 Subject: [PATCH] fix: skip heavy recursion tests in debug builds, keep one for assertion coverage MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Skip AvmRecursionInnerCircuitTests in debug builds (timed out at 600s in nightly). Same pattern as HonkRecursionConstraintTest and ChonkRecursionConstraintTest. Reinstate HonkRecursionConstraintTestWithoutPredicate/1.GenerateVK FromConstraints (241s, well within timeout) so the debug-only native_verification_debug code path in honk_recursion_constraint.cpp is still exercised. This is the only debug assertion unique to these skipped suites — all others either fire on the happy path (covered by other tests) or are disabled via BB_DISABLE_ASSERTS() in the test. --- barretenberg/cpp/bootstrap.sh | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/barretenberg/cpp/bootstrap.sh b/barretenberg/cpp/bootstrap.sh index feaf939f5dd7..060f41086dba 100755 --- a/barretenberg/cpp/bootstrap.sh +++ b/barretenberg/cpp/bootstrap.sh @@ -247,8 +247,12 @@ function test_cmds_native { while read -r test; do # Skip heavy recursion tests in debug builds — they take 400-600s and the same # code paths are already exercised (with assertions) by faster tests in the suite. - if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then - continue + # Keep WithoutPredicate/1.GenerateVKFromConstraints (241s) so that the debug-only + # native_verification_debug path in honk_recursion_constraint.cpp is still exercised. + if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest|AvmRecursionInnerCircuitTests) ]]; then + if [[ "$test" != "HonkRecursionConstraintTestWithoutPredicate/1.GenerateVKFromConstraints" ]]; then + continue + fi fi local prefix=$hash # A little extra resource for these tests.