test(internal/ethapi): align api_test coverage with geth ethapi#2232
test(internal/ethapi): align api_test coverage with geth ethapi#2232gzliudan wants to merge 1 commit intoXinFinOrg:dev-upgradefrom
Conversation
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
0f2dbf7 to
f463758
Compare
Broaden internal/ethapi/api_test.go with content-level parity checks against geth where behavior is compatible. - Expand RPC retrieval coverage for block/header/receipt and transaction queries, including hash/number selectors, pending/fullTx modes, and not-found paths. - Add and refine transaction fill/sign/send test paths, including validation and pricing-mode combinations. - Deepen EstimateGas/Call/CreateAccessList/SimulateV1 assertions for block reference forwarding, state overrides, validation errors, transfer logs, precompile movement, storage state buildup, and VM error reporting. - Keep the change test-only (single file) with no runtime behavior modifications.
f463758 to
8ad99af
Compare
Proposed changes
Broaden internal/ethapi/api_test.go with content-level parity checks against geth where behavior is compatible.
Types of changes
What types of changes does your code introduce to XDC network?
Put an
✅in the boxes that applyImpacted Components
Which parts of the codebase does this PR touch?
Put an
✅in the boxes that applyChecklist
Put an
✅in the boxes once you have confirmed below actions (or provide reasons on not doing so) that