Skip to content

Commit a8c15f7

Browse files
committed
fix: disable debug field assertions in TranslatorRelationFailureTests
1 parent 848af53 commit a8c15f7

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

barretenberg/cpp/src/barretenberg/translator_vm/relation_failure.test.cpp

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
* InRangeValueInMaskingFlowsToOrderedTail — trace FF(42) from wire masking through
3838
* concatenation into ordered poly tail
3939
*/
40+
#include "barretenberg/common/assert.hpp"
4041
#include "barretenberg/honk/library/grand_product_library.hpp"
4142
#include "barretenberg/honk/relation_checker.hpp"
4243
#include "barretenberg/translator_vm/translator_circuit_builder.hpp"
@@ -188,6 +189,15 @@ ValidTranslatorState build_valid_accumulator_transfer_state()
188189
class TranslatorRelationFailureTests : public ::testing::Test {
189190
protected:
190191
static void SetUpTestSuite() { bb::srs::init_file_crs_factory(bb::srs::bb_crs_path()); }
192+
193+
// Corruption tests intentionally place witness values outside the ranges the relations assume
194+
// (e.g. negative deltas, oversized sorted values). Evaluating the relation polynomials on such
195+
// values can produce intermediate field elements that violate the debug-only
196+
// `assert_coarse_form()` invariant (values must be in `[0, 2p)`), aborting the test before
197+
// `RelationChecker::check` can report the expected failure. Downgrade debug field assertions
198+
// to warnings — the test's `EXPECT_FALSE(failures.empty())` still validates the detection
199+
// path. Same pattern as #21881 for `shplemini.test.cpp` HighDegreeAttack tests.
200+
bb::AssertGuard assert_guard{ bb::AssertMode::WARN };
191201
};
192202

193203
/**

0 commit comments

Comments
 (0)