Skip to content

Conversation

@lupin012
Copy link
Contributor

@lupin012 lupin012 commented Dec 9, 2025

Disable temporary latest test (debug_traceBlockByNumber/test_43.json) because obtain sometimes different response from Geth

@mriccobene mriccobene changed the title QA latest test: disable flaky test debug_tarceBlockByNumber/test_43 temporary qa-tests: on RPC latest test suite, temporary disable flaky debug_tarceBlockByNumber/test_43 test Dec 9, 2025
@mriccobene mriccobene self-requested a review December 9, 2025 11:29
@mriccobene mriccobene marked this pull request as ready for review December 9, 2025 11:29
Copilot AI review requested due to automatic review settings December 9, 2025 11:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR temporarily disables a flaky test (debug_traceBlockByNumber/test_43.json) in the Ethereum RPC test suite for the latest version. The change prevents this unreliable test from causing intermittent CI failures while the underlying flakiness is investigated or resolved.

Key Changes:

  • Added debug_traceBlockByNumber/test_43.json to the disabled test list with a comment indicating it's a flaky test

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lupin012 lupin012 changed the title qa-tests: on RPC latest test suite, temporary disable flaky debug_tarceBlockByNumber/test_43 test qa-tests: on RPC latest test suite, temporary disable flaky debug_traceBlockByNumber/test_43 test Dec 9, 2025
@lupin012 lupin012 enabled auto-merge (squash) December 9, 2025 11:53
@lupin012 lupin012 merged commit 2802fa6 into main Dec 9, 2025
31 of 37 checks passed
@lupin012 lupin012 deleted the lupin012/disable_fluky_test_43 branch December 9, 2025 12:32
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.

3 participants