Skip to content

Commit aa1dabf

Browse files
committed
fix: skip heavy recursion tests in debug builds to fix nightly CI
Adds AvmRecursionConstraintTest, ECCVMRecursiveTests, GoblinRecursiveVerifierTests, GoblinAvmRecursiveVerifierTests, BoomerangGoblinRecursiveVerifierTests, BoomerangGoblinAvmRecursiveVerifierTests, BoomerangTwoLayerAvmRecursiveVerifierTests, and TwoLayer AVM recursion variants to the debug-build skip list. These take 7-10+ min in debug mode, exceeding the 600s CI timeout. Debug assertion audit: none of the newly skipped suites exercise unique debug-only (#ifndef NDEBUG) code paths. The only debug-specific path (native_verification_debug in honk_recursion_constraint.cpp) is already covered by the retained HonkRecursionConstraintTestWithoutPredicate/1 test.
1 parent d092fef commit aa1dabf

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

barretenberg/cpp/bootstrap.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -245,11 +245,12 @@ 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
248+
# 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.
250250
# Keep WithoutPredicate/1.GenerateVKFromConstraints (241s) so that the debug-only
251251
# native_verification_debug path in honk_recursion_constraint.cpp is still exercised.
252-
if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest|AvmRecursionInnerCircuitTests) ]]; then
252+
# None of the other skipped suites exercise unique debug-only (#ifndef NDEBUG) code paths.
253+
if [[ "$native_preset" == *debug* ]] && [[ "$test" =~ ^(HonkRecursionConstraintTest|ChonkRecursionConstraintTest|AvmRecursionInnerCircuitTests|AvmRecursionConstraintTest|AvmRecursiveTests\.TwoLayer|PaddingVariants/AvmRecursiveTestsParameterized\.TwoLayer|BoomerangTwoLayerAvmRecursiveVerifierTests|ECCVMRecursiveTests|GoblinRecursiveVerifierTests|GoblinAvmRecursiveVerifierTests|BoomerangGoblinRecursiveVerifierTests|BoomerangGoblinAvmRecursiveVerifierTests) ]]; then
253254
if [[ "$test" != "HonkRecursionConstraintTestWithoutPredicate/1.GenerateVKFromConstraints" ]]; then
254255
continue
255256
fi

0 commit comments

Comments
 (0)