Skip to content

fix: fixes failing acceptance test #4612

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Nov 18, 2025
+15 −12
Merged

fix: fixes failing acceptance test #4612

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
7964f68
fixes test
konstantinabl Nov 14, 2025
757f3dd
removes release prefix for opcode logger tests
konstantinabl Nov 14, 2025
9a947bc
adds release prefix to tests for prestate and call tracer
konstantinabl Nov 14, 2025
d040c89
adds tracer override for k6 tests to not use opcodeLogger
konstantinabl Nov 14, 2025
7794979
tests fix
konstantinabl Nov 17, 2025
9ba9ef2
Revert "tests fix"
konstantinabl Nov 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion k6/src/scenarios/test/debug_traceTransaction.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
22 changes: 11 additions & 11 deletions packages/server/tests/acceptance/debug.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand All @@ -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,
Expand All @@ -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');
Expand All @@ -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,
Expand All @@ -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,
Expand All @@ -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
Expand All @@ -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,
Expand All @@ -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,
Expand All @@ -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,
Expand All @@ -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,
Expand Down
2 changes: 2 additions & 0 deletions packages/server/tests/acceptance/rpc_batch1.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
Loading