From 7964f6865a03bc78bce4509ef6871a0ca9c3fc6e Mon Sep 17 00:00:00 2001 From: Konstantina Blazhukova Date: Fri, 14 Nov 2025 12:25:47 +0200 Subject: [PATCH 1/6] fixes test Signed-off-by: Konstantina Blazhukova --- packages/server/tests/acceptance/rpc_batch1.spec.ts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/packages/server/tests/acceptance/rpc_batch1.spec.ts b/packages/server/tests/acceptance/rpc_batch1.spec.ts index 0688b6bce0..b3f18bc55c 100644 --- a/packages/server/tests/acceptance/rpc_batch1.spec.ts +++ b/packages/server/tests/acceptance/rpc_batch1.spec.ts @@ -2499,6 +2499,8 @@ describe('@api-batch-1 RPC Server Acceptance Tests', function () { const signedTx = await accounts[2].wallet.signTransaction(transaction); const transactionHash = await relay.sendRawTransaction(signedTx); + // wait for tx receipt + await relay.pollForValidTransactionReceipt(transactionHash); const res = await relay.call(RelayCalls.ETH_ENDPOINTS.ETH_GET_TRANSACTION_BY_HASH, [transactionHash]); expect(res.to).to.be.null; From 757f3ddcb1b46b2a2280453fe38ef1a96e1657f1 Mon Sep 17 00:00:00 2001 From: Konstantina Blazhukova Date: Fri, 14 Nov 2025 17:23:54 +0200 Subject: [PATCH 2/6] removes release prefix for opcode logger tests Signed-off-by: Konstantina Blazhukova --- packages/server/tests/acceptance/debug.spec.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/server/tests/acceptance/debug.spec.ts b/packages/server/tests/acceptance/debug.spec.ts index ba620c54eb..4a64f4c886 100644 --- a/packages/server/tests/acceptance/debug.spec.ts +++ b/packages/server/tests/acceptance/debug.spec.ts @@ -511,7 +511,7 @@ describe('@debug API Acceptance Tests', function () { }); describe('OpcodeLogger', () => { - it('@release should trace a successful transaction using OpcodeLogger (default when no tracer specified)', async function () { + it('should trace a successful transaction using OpcodeLogger (default when no tracer specified)', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [createChildTx.hash]); // Validate response structure for OpcodeLogger @@ -528,13 +528,13 @@ describe('@debug API Acceptance Tests', function () { } }); - it('@release should trace a successful transaction using OpcodeLogger explicitly', async function () { + it('should trace a successful transaction using OpcodeLogger explicitly', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [createChildTx.hash, TRACER_CONFIGS.OPCODE_LOGGER]); Assertions.validateOpcodeLoggerResult(result); }); - it('@release should trace using OpcodeLogger with custom config (enableMemory=true)', async function () { + it('should trace using OpcodeLogger with custom config (enableMemory=true)', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, TRACER_CONFIGS.OPCODE_WITH_MEMORY, @@ -550,7 +550,7 @@ describe('@debug API Acceptance Tests', function () { } }); - it('@release should trace using OpcodeLogger with custom config (disableStack=true)', async function () { + it('should trace using OpcodeLogger with custom config (disableStack=true)', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, TRACER_CONFIGS.OPCODE_WITH_STACK, @@ -566,7 +566,7 @@ describe('@debug API Acceptance Tests', function () { } }); - it('@release should trace using OpcodeLogger with custom config (disableStorage=true)', async function () { + it('should trace using OpcodeLogger with custom config (disableStorage=true)', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, TRACER_CONFIGS.OPCODE_WITH_STORAGE, @@ -582,7 +582,7 @@ describe('@debug API Acceptance Tests', function () { } }); - it('@release should trace using OpcodeLogger with custom config (enableMemory=true, disableStorage=true)', async function () { + it('should trace using OpcodeLogger with custom config (enableMemory=true, disableStorage=true)', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, TRACER_CONFIGS.OPCODE_WITH_MEMORY_AND_STORAGE, From 9a947bc912185a17831f92e8e9e6427126511715 Mon Sep 17 00:00:00 2001 From: Konstantina Blazhukova Date: Fri, 14 Nov 2025 17:44:35 +0200 Subject: [PATCH 3/6] adds release prefix to tests for prestate and call tracer Signed-off-by: Konstantina Blazhukova --- packages/server/tests/acceptance/debug.spec.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/server/tests/acceptance/debug.spec.ts b/packages/server/tests/acceptance/debug.spec.ts index 4a64f4c886..9885b5f541 100644 --- a/packages/server/tests/acceptance/debug.spec.ts +++ b/packages/server/tests/acceptance/debug.spec.ts @@ -417,7 +417,7 @@ describe('@debug API Acceptance Tests', function () { }); describe('Call Tracer', () => { - it('should trace a transaction using CallTracer with onlyTopCall=false', async function () { + it('@release should trace a transaction using CallTracer with onlyTopCall=false', async function () { // Call debug_traceTransaction with CallTracer (default config) const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, @@ -433,7 +433,7 @@ describe('@debug API Acceptance Tests', function () { expect(result).to.have.property('calls'); }); - it('should trace a transaction using CallTracer with onlyTopCall=true', async function () { + it('@release should trace a transaction using CallTracer with onlyTopCall=true', async function () { // Call debug_traceTransaction with CallTracer (default config) const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, @@ -452,7 +452,7 @@ describe('@debug API Acceptance Tests', function () { }); describe('PrestateTracer', () => { - it('should trace a transaction using PrestateTracer', async function () { + it('@release should trace a transaction using PrestateTracer', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [createChildTx.hash, TRACER_CONFIGS.PRESTATE_TRACER]); expect(result).to.be.an('object'); @@ -469,7 +469,7 @@ describe('@debug API Acceptance Tests', function () { } }); - it('should trace a transaction using PrestateTracer with onlyTopCall=true', async function () { + it('@release should trace a transaction using PrestateTracer with onlyTopCall=true', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, TRACER_CONFIGS.PRESTATE_TRACER_TOP_ONLY, @@ -489,7 +489,7 @@ describe('@debug API Acceptance Tests', function () { } }); - it('should trace a transaction using PrestateTracer with onlyTopCall=false', async function () { + it('@release should trace a transaction using PrestateTracer with onlyTopCall=false', async function () { const result = await relay.call(DEBUG_TRACE_TRANSACTION, [ createChildTx.hash, TRACER_CONFIGS.PRESTATE_TRACER_TOP_ONLY_FALSE, From d040c895ceea3f48b64dec268cab6aa24bc355a1 Mon Sep 17 00:00:00 2001 From: Konstantina Blazhukova Date: Fri, 14 Nov 2025 19:00:27 +0200 Subject: [PATCH 4/6] adds tracer override for k6 tests to not use opcodeLogger Signed-off-by: Konstantina Blazhukova --- k6/src/scenarios/test/debug_traceTransaction.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/k6/src/scenarios/test/debug_traceTransaction.js b/k6/src/scenarios/test/debug_traceTransaction.js index 328b9447c8..732c24cb69 100644 --- a/k6/src/scenarios/test/debug_traceTransaction.js +++ b/k6/src/scenarios/test/debug_traceTransaction.js @@ -12,7 +12,8 @@ const { options, run } = new TestScenarioBuilder() .request((testParameters) => { // Use existing transaction hash from test parameters const transactionHash = testParameters.DEFAULT_TRANSACTION_HASH; - return http.post(testParameters.RELAY_BASE_URL, getPayLoad(methodName, [transactionHash]), httpParams); + const tracerConfig = {"tracer": "callTracer"}; + return http.post(testParameters.RELAY_BASE_URL, getPayLoad(methodName, [transactionHash, tracerConfig]), httpParams); }) .check(methodName, (r) => isNonErrorResponse(r)) .maxDuration(5000) // Extended timeout for potentially slow debug responses From 7794979b7ab8297556801331d0ac5a781d66a59d Mon Sep 17 00:00:00 2001 From: Konstantina Blazhukova Date: Mon, 17 Nov 2025 16:25:32 +0200 Subject: [PATCH 5/6] tests fix Signed-off-by: Konstantina Blazhukova --- packages/server/tests/acceptance/rpc_batch1.spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/server/tests/acceptance/rpc_batch1.spec.ts b/packages/server/tests/acceptance/rpc_batch1.spec.ts index b3f18bc55c..c6f0c6d34b 100644 --- a/packages/server/tests/acceptance/rpc_batch1.spec.ts +++ b/packages/server/tests/acceptance/rpc_batch1.spec.ts @@ -2500,7 +2500,7 @@ describe('@api-batch-1 RPC Server Acceptance Tests', function () { const transactionHash = await relay.sendRawTransaction(signedTx); // wait for tx receipt - await relay.pollForValidTransactionReceipt(transactionHash); + //await relay.pollForValidTransactionReceipt(transactionHash); const res = await relay.call(RelayCalls.ETH_ENDPOINTS.ETH_GET_TRANSACTION_BY_HASH, [transactionHash]); expect(res.to).to.be.null; From 9ba9ef28cd4178df92251ee2b12ecfb8b10ee88a Mon Sep 17 00:00:00 2001 From: Konstantina Blazhukova Date: Mon, 17 Nov 2025 17:06:01 +0200 Subject: [PATCH 6/6] Revert "tests fix" This reverts commit 3c17da6f3ec7c3e39e824be4f3d79645186cb0c7. Signed-off-by: Konstantina Blazhukova --- packages/server/tests/acceptance/rpc_batch1.spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/server/tests/acceptance/rpc_batch1.spec.ts b/packages/server/tests/acceptance/rpc_batch1.spec.ts index c6f0c6d34b..b3f18bc55c 100644 --- a/packages/server/tests/acceptance/rpc_batch1.spec.ts +++ b/packages/server/tests/acceptance/rpc_batch1.spec.ts @@ -2500,7 +2500,7 @@ describe('@api-batch-1 RPC Server Acceptance Tests', function () { const transactionHash = await relay.sendRawTransaction(signedTx); // wait for tx receipt - //await relay.pollForValidTransactionReceipt(transactionHash); + await relay.pollForValidTransactionReceipt(transactionHash); const res = await relay.call(RelayCalls.ETH_ENDPOINTS.ETH_GET_TRANSACTION_BY_HASH, [transactionHash]); expect(res.to).to.be.null;