Skip to content

disable flaky test classes#8239

Merged
macfarla merged 1 commit intobesu-eth:mainfrom
macfarla:disable-flakes
Feb 4, 2025
Merged

disable flaky test classes#8239
macfarla merged 1 commit intobesu-eth:mainfrom
macfarla:disable-flakes

Conversation

@macfarla
Copy link
Copy Markdown
Contributor

@macfarla macfarla commented Feb 4, 2025

PR description

Disabling these tests for now since they are blocking the PR pipeline

Fixed Issue(s)

Refs #7973 and #8238

Thanks for sending a pull request! Have you done the following?

  • Checked out our contribution guidelines?
  • Considered documentation and added the doc-change-required label to this PR if updates are required.
  • Considered the changelog and included an update if required.
  • For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • spotless: ./gradlew spotlessApply
  • unit tests: ./gradlew build
  • acceptance tests: ./gradlew acceptanceTest
  • integration tests: ./gradlew integrationTest
  • reference tests: ./gradlew ethereum:referenceTests:referenceTests

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
@macfarla macfarla added testing flake 60% of the time it works 100% of the time. labels Feb 4, 2025
@macfarla macfarla merged commit a2ef71b into besu-eth:main Feb 4, 2025
pullurib pushed a commit to pullurib/besu that referenced this pull request Feb 6, 2025
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Bhanu Pulluri <bhanu.pulluri@kaleido.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flake 60% of the time it works 100% of the time. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants