test: revert PoV tests to their original expected proofSize value ranges

moondance-labs · Oct 11, 2024 · 58db5bd · 58db5bd
1 parent d962111
commit 58db5bd
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/test/suites/dev-frontier-template/test-pov/test-evm-over-pov.ts b/test/suites/dev-frontier-template/test-pov/test-evm-over-pov.ts
@@ -14,7 +14,7 @@ describeSuite({
         let contracts: HeavyContract[];
         let callData: `0x${string}`;
         const MAX_CONTRACTS = 20;
-        const EXPECTED_POV_ROUGH = 38_000; // bytes
+        const EXPECTED_POV_ROUGH = 16_000; // bytes
 
         beforeAll(async () => {
             const { contractAddress, abi } = await deployCreateCompiledContract(context, "CallForwarder");
@@ -98,7 +98,7 @@ describeSuite({
                 // in the block.
                 // 1M Gas allows ~38k of PoV, so we verify we are within range.
                 expect(block.proofSize).to.be.at.least(15_000);
-                expect(block.proofSize).to.be.at.most(38_000 * 1.2);
+                expect(block.proofSize).to.be.at.most(25_000);
                 expect(result?.successful).to.equal(true);
                 expectEVMResult(result!.events, "Error", "OutOfGas");
             },

diff --git a/test/suites/dev-frontier-template/test-pov/test-evm-over-pov2.ts b/test/suites/dev-frontier-template/test-pov/test-evm-over-pov2.ts
@@ -50,8 +50,8 @@ describeSuite({
                 const { result, block } = await context.createBlock(rawSigned);
 
                 log(`block.proofSize: ${block.proofSize} (successful: ${result?.successful})`);
-                expect(block.proofSize).toBeGreaterThanOrEqual(30_000);
-                expect(block.proofSize).toBeLessThanOrEqual(50_000n + emptyBlockProofSize);
+                expect(block.proofSize).toBeGreaterThanOrEqual(15_000);
+                expect(block.proofSize).toBeLessThanOrEqual(25_000n + emptyBlockProofSize);
                 expect(result?.successful).to.equal(true);
             },
         });

diff --git a/test/suites/dev-frontier-template/test-pov/test-precompile-over-pov.ts b/test/suites/dev-frontier-template/test-pov/test-precompile-over-pov.ts
@@ -12,7 +12,7 @@ describeSuite({
     testCases: ({ context, it }) => {
         let contracts: HeavyContract[];
         const MAX_CONTRACTS = 50;
-        const EXPECTED_POV_ROUGH = 8_000; // bytes
+        const EXPECTED_POV_ROUGH = 20_000; // bytes
         let batchAbi: Abi;
         let proxyAbi: Abi;
         let proxyAddress: `0x${string}`;
@@ -63,8 +63,8 @@ describeSuite({
                 // With 1M gas we are allowed to use ~62kb of POV, so verify the range.
                 // The tx is still included in the block because it contains the failed tx,
                 // so POV is included in the block as well.
-                expect(block.proofSize).to.be.at.least(6_000);
-                expect(block.proofSize).to.be.at.most(10_000);
+                expect(block.proofSize).to.be.at.least(15_000);
+                expect(block.proofSize).to.be.at.most(30_000);
                 expect(result?.successful).to.equal(true);
                 expectEVMResult(result!.events, "Error", "OutOfGas");
             },
@@ -91,7 +91,7 @@ describeSuite({
                 expect(block.proofSize).to.be.at.least(EXPECTED_POV_ROUGH / 1.3);
                 expect(block.proofSize).to.be.at.most(EXPECTED_POV_ROUGH * 1.3);
                 expect(result?.successful).to.equal(true);
-                expectEVMResult(result!.events, "Succeed");
+                expectEVMResult(result!.events, "Succeed", "Reserved");
             },
         });
 
@@ -110,7 +110,7 @@ describeSuite({
                 expect(block.proofSize).to.be.at.least(EXPECTED_POV_ROUGH / 1.3);
                 expect(block.proofSize).to.be.at.most(EXPECTED_POV_ROUGH * 1.3);
                 expect(result?.successful).to.equal(true);
-                expectEVMResult(result!.events, "Succeed");
+                expectEVMResult(result!.events, "Succeed", "Returned");
             },
         });
     },

diff --git a/test/suites/dev-frontier-template/test-pov/test-precompile-over-pov2.ts b/test/suites/dev-frontier-template/test-pov/test-precompile-over-pov2.ts
@@ -60,8 +60,8 @@ describeSuite({
                 });
 
                 const { result, block } = await context.createBlock(rawSigned);
-                expect(block.proofSize).to.be.at.least(Number(6_000));
-                expect(block.proofSize).to.be.at.most(Number(10_000n + emptyBlockProofSize));
+                expect(block.proofSize).to.be.at.least(Number(15_000));
+                expect(block.proofSize).to.be.at.most(Number(30_000n + emptyBlockProofSize));
                 expect(result?.successful).to.equal(true);
             },
         });