Skip to content

Commit f590cd1

Browse files
AztecBotcritesjosh
authored andcommitted
fix: skip heavy recursion tests in debug builds (#22446)
## Summary Instead of bumping timeouts for slow tests in debug mode, skip them entirely. `HonkRecursionConstraintTest` and `ChonkRecursionConstraintTest` take 400-600s in debug builds, hitting the 600s default timeout. These are heavy recursion constraint tests that build full recursive circuits — the same code paths are already exercised (with debug assertions) by faster tests in the suite. This replaces the timeout-bump approach from PR #22347 with Adam's suggestion to be selective about what runs in debug mode. Supersedes #22427 ClaudeBox log: https://claudebox.work/s/c3fd56d2f55c879c?run=2
1 parent 83fc568 commit f590cd1

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

barretenberg/cpp/bootstrap.sh

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -245,16 +245,17 @@ function test_cmds_native {
245245
awk '/^[a-zA-Z]/ {suite=$1} /^[ ]/ {print suite$1}' | \
246246
grep -v 'DISABLED_' | \
247247
while read -r test; do
248+
# Skip heavy recursion tests in debug builds — they take 400-600s and the same
249+
# code paths are already exercised (with assertions) by faster tests in the suite.
250+
if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then
251+
continue
252+
fi
248253
local prefix=$hash
249254
# A little extra resource for these tests.
250255
# IPARecursiveTests fails with 2 threads.
251256
if [[ "$test" =~ ^(AcirAvmRecursionConstraint|ChonkKernelCapacity|AvmRecursiveTests|IPARecursiveTests|HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then
252257
prefix="$prefix:CPUS=4:MEM=8g"
253258
fi
254-
# These tests routinely take 400-600s in debug builds; bump from the 600s default.
255-
if [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest) ]]; then
256-
prefix="$prefix:TIMEOUT=900s"
257-
fi
258259
echo -e "$prefix barretenberg/cpp/scripts/run_test.sh $bin_name $test"
259260
done || (echo "Failed to list tests in $bin" && exit 1)
260261
done

0 commit comments

Comments
 (0)