Skip to content

Commit 76989f9

Browse files
committed
chore: update more tests
1 parent 21e39fc commit 76989f9

2 files changed

Lines changed: 6 additions & 6 deletions

File tree

barretenberg/cpp/src/barretenberg/vm2/constraining/relations/ecc.test.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1183,7 +1183,7 @@ TEST(EccAddMemoryConstrainingTest, EccAddMemoryInvalidDstRange)
11831183

11841184
StrictMock<MockExecutionIdManager> execution_id_manager;
11851185
EXPECT_CALL(execution_id_manager, get_execution_id)
1186-
.WillRepeatedly(Return(0)); // Use a fixed execution IDfor the test
1186+
.WillRepeatedly(Return(0)); // Use a fixed execution ID for the test
11871187
PureGreaterThan gt;
11881188
PureToRadix to_radix_simulator = PureToRadix();
11891189

@@ -1194,7 +1194,7 @@ TEST(EccAddMemoryConstrainingTest, EccAddMemoryInvalidDstRange)
11941194
scalar_mul_event_emitter,
11951195
ecc_add_memory_event_emitter);
11961196

1197-
uint32_t dst_address = AVM_HIGHEST_MEM_ADDRESS - 1; // Invalid address, will result in out of range error
1197+
uint32_t dst_address = AVM_HIGHEST_MEM_ADDRESS; // Invalid address, will result in out of range error
11981198
// Set the execution and gt traces
11991199
TestTraceContainer trace = TestTraceContainer({
12001200
// Row 0

barretenberg/cpp/src/barretenberg/vm2/simulation/gadgets/ecc.test.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ TEST(AvmSimulationEccTest, AddWithMemory)
148148
MemoryStore memory;
149149

150150
EXPECT_CALL(execution_id_manager, get_execution_id()).WillOnce(Return(0));
151-
EXPECT_CALL(gt, gt(0x1000 + 2, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
151+
EXPECT_CALL(gt, gt(0x1000 + 1, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
152152

153153
Ecc ecc(
154154
execution_id_manager, gt, to_radix, ecc_event_emitter, scalar_mul_event_emitter, ecc_add_memory_event_emitter);
@@ -184,7 +184,7 @@ TEST(AvmSimulationEccTest, AddNotOnCurve)
184184
MemoryStore memory;
185185

186186
EXPECT_CALL(execution_id_manager, get_execution_id()).WillOnce(Return(0));
187-
EXPECT_CALL(gt, gt(0x1000 + 2, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
187+
EXPECT_CALL(gt, gt(0x1000 + 1, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
188188

189189
Ecc ecc(
190190
execution_id_manager, gt, to_radix, ecc_event_emitter, scalar_mul_event_emitter, ecc_add_memory_event_emitter);
@@ -215,7 +215,7 @@ TEST(AvmSimulationEccTest, InfinityOnCurve)
215215
MemoryStore memory;
216216

217217
EXPECT_CALL(execution_id_manager, get_execution_id()).WillOnce(Return(0));
218-
EXPECT_CALL(gt, gt(0x1000 + 2, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
218+
EXPECT_CALL(gt, gt(0x1000 + 1, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
219219

220220
Ecc ecc(
221221
execution_id_manager, gt, to_radix, ecc_event_emitter, scalar_mul_event_emitter, ecc_add_memory_event_emitter);
@@ -248,7 +248,7 @@ TEST(AvmSimulationEccTest, AddsUpToInfinity)
248248
MemoryStore memory;
249249

250250
EXPECT_CALL(execution_id_manager, get_execution_id()).WillOnce(Return(0));
251-
EXPECT_CALL(gt, gt(0x1000 + 2, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
251+
EXPECT_CALL(gt, gt(0x1000 + 1, AVM_HIGHEST_MEM_ADDRESS)).WillOnce(Return(false));
252252

253253
Ecc ecc(
254254
execution_id_manager, gt, to_radix, ecc_event_emitter, scalar_mul_event_emitter, ecc_add_memory_event_emitter);

0 commit comments

Comments
 (0)