Skip to content

Conversation

@fab-10
Copy link
Contributor

@fab-10 fab-10 commented Dec 19, 2025

PR description

This PR ports the existing test plugins, used for acceptance tests, to use the Gradle plugin for Besu plugins, this way the acceptance tests are closer to the standard way of using plugins and it will allow for more plugin verification on startup to be implemented in following PR.

To achieve that the test-plugins subproject has to be moved to a new detached project, due to circular dependencies between Besu and the Gradle plugin, while trying to keep the DevUX intact as much as possible.
Basically since the Gradle plugin needs to know the version of Besu to use, and we want to use the current local version, before building anything in the detached project, Besu project must publish its artifacts to mavenLocal, for this helper tasks have been added to Besu's build.gradle to automate that part, and normally a developer should not know about them, because the normal development workflow remains the same.

Fixed Issue(s)

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
  • hive tests: Engine or other RPCs modified?

@fab-10 fab-10 force-pushed the detached-test-plugins2 branch 2 times, most recently from 4da8979 to 9b2621a Compare December 19, 2025 15:41
@fab-10 fab-10 added the plugins label Dec 19, 2025
@fab-10 fab-10 marked this pull request as ready for review December 19, 2025 16:09
Copilot AI review requested due to automatic review settings December 19, 2025 16:09
Copy link

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@fab-10 fab-10 force-pushed the detached-test-plugins2 branch from 9b2621a to 4d4e2a0 Compare January 5, 2026 11:48
…su plugin

Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>
@fab-10 fab-10 force-pushed the detached-test-plugins2 branch from 4d4e2a0 to 6212155 Compare January 7, 2026 11:15
@fab-10 fab-10 enabled auto-merge (squash) January 7, 2026 11:32
@fab-10 fab-10 merged commit 3d27f00 into hyperledger:main Jan 7, 2026
46 checks passed
@fab-10 fab-10 deleted the detached-test-plugins2 branch January 7, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants