Skip to content

chore: adds acceptance tests for debug_traceTransaction #3877

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.

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

konstantinabl
Copy link
Contributor

@konstantinabl konstantinabl commented Jun 25, 2025

Description:

Adding acceptance tests covering the debug_traceTransaction endpoint

Related issue(s):

Fixes #3816

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
@konstantinabl konstantinabl linked an issue Jun 25, 2025 that may be closed by this pull request
Copy link

github-actions bot commented Jun 25, 2025

Test Results

 22 files  + 1  278 suites   - 16   18m 24s ⏱️ -36s
685 tests +18  676 ✅ +15  5 💤 ±0   4 ❌ + 3 
755 runs  +37  738 ✅ +26  5 💤 ±0  12 ❌ +11 

For more details on these failures, see this check.

Results for commit f1a634f. ± Comparison against base commit 8ff3d14.

This pull request removes 1 and adds 19 tests. Note that renamed tests count towards both.
"before each" hook for "reverts" ‑ RPC Server Acceptance Tests Acceptance tests @erc20 Acceptance Tests HTS token should behave like erc20 transfer from when the token owner is not the zero address when the recipient is not the zero address when the spender does not have enough allowance "before each" hook for "reverts"
@release should trace a successful transaction using OpcodeLogger (default when no tracer specified) ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction OpcodeLogger @release should trace a successful transaction using OpcodeLogger (default when no tracer specified)
@release should trace a successful transaction using OpcodeLogger explicitly ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction OpcodeLogger @release should trace a successful transaction using OpcodeLogger explicitly
@release should trace using OpcodeLogger with custom config (disableStack=true) ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction OpcodeLogger @release should trace using OpcodeLogger with custom config (disableStack=true)
@release should trace using OpcodeLogger with custom config (disableStorage=true) ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction OpcodeLogger @release should trace using OpcodeLogger with custom config (disableStorage=true)
@release should trace using OpcodeLogger with custom config (enableMemory=true) ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction OpcodeLogger @release should trace using OpcodeLogger with custom config (enableMemory=true)
@release should trace using OpcodeLogger with custom config (enableMemory=true, disableStorage=true) ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction OpcodeLogger @release should trace using OpcodeLogger with custom config (enableMemory=true, disableStorage=true)
should fail with INVALID_PARAMETER when given an invalid tracer type ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction Edge Cases - Parameter Validation should fail with INVALID_PARAMETER when given an invalid tracer type
should fail with INVALID_PARAMETER when given an invalid transaction hash format ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction Edge Cases - Parameter Validation should fail with INVALID_PARAMETER when given an invalid transaction hash format
should fail with INVALID_PARAMETER when given invalid TracerConfig for CallTracer ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction Edge Cases - Parameter Validation should fail with INVALID_PARAMETER when given invalid TracerConfig for CallTracer
should fail with INVALID_PARAMETER when given invalid TracerConfig for OpcodeLogger ‑ RPC Server Acceptance Tests Acceptance tests @debug API Acceptance Tests debug_traceTransaction Edge Cases - Parameter Validation should fail with INVALID_PARAMETER when given invalid TracerConfig for OpcodeLogger
…

♻️ This comment has been updated with latest results.

Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
@konstantinabl konstantinabl self-assigned this Jun 25, 2025
…undant code;

Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
@konstantinabl konstantinabl changed the title chore: adds acceptance tests for debug_trace chore: adds acceptance tests for debug_traceTransaction Jun 25, 2025
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
@konstantinabl konstantinabl marked this pull request as ready for review June 25, 2025 13:54
@konstantinabl konstantinabl requested review from a team as code owners June 25, 2025 13:54
@konstantinabl konstantinabl requested a review from acuarica June 25, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add acceptance tests for debug_traceTransaction endpoint
1 participant