Skip to content

Commit 5527afc

Browse files
committed
fix: skip heavy recursion tests in debug builds, keep one for assertion coverage
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.
1 parent 5e23bc0 commit 5527afc

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

barretenberg/cpp/bootstrap.sh

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -247,8 +247,12 @@ function test_cmds_native {
247247
while read -r test; do
248248
# Skip heavy recursion tests in debug builds — they take 400-600s and the same
249249
# code paths are already exercised (with assertions) by faster tests in the suite.
250-
if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then
251-
continue
250+
# Keep WithoutPredicate/1.GenerateVKFromConstraints (241s) so that the debug-only
251+
# native_verification_debug path in honk_recursion_constraint.cpp is still exercised.
252+
if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest|AvmRecursionInnerCircuitTests) ]]; then
253+
if [[ "$test" != "HonkRecursionConstraintTestWithoutPredicate/1.GenerateVKFromConstraints" ]]; then
254+
continue
255+
fi
252256
fi
253257
local prefix=$hash
254258
# A little extra resource for these tests.

0 commit comments

Comments
 (0)