Skip to content

[Gradle] Rework internal test plugins to support test suites #126341

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 2 commits into
base: main
Choose a base branch
from

Conversation

breskeby
Copy link
Contributor

@breskeby breskeby commented Apr 4, 2025

Long term we want to have an easier way to declare custom sets of tests. Therefore, we want to support
gradle test suites syntax for all our test types to make this easier.

@breskeby breskeby added >non-issue :Delivery/Build Build or test infrastructure labels Apr 4, 2025
@breskeby breskeby requested a review from a team as a code owner April 4, 2025 19:36
@breskeby breskeby added Team:Delivery Meta label for Delivery team auto-backport Automatically create backport pull requests when merged v8.18.1 v8.19.0 v9.0.1 v9.1.0 v8.17.5 v8.16.7 labels Apr 4, 2025
@breskeby breskeby self-assigned this Apr 4, 2025
@breskeby breskeby force-pushed the internal-cluster-test-using-testsuites branch from 5cf9ac4 to fb16829 Compare April 4, 2025 19:36
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@breskeby breskeby changed the title Rework internalClusterTest to support test suites [Gradle] Rework internal test plugins to support test suites Apr 4, 2025
@breskeby breskeby force-pushed the internal-cluster-test-using-testsuites branch 4 times, most recently from c16bb93 to 1b6c4ff Compare April 9, 2025 21:52
@breskeby breskeby force-pushed the internal-cluster-test-using-testsuites branch from fecfd7c to f4231eb Compare April 10, 2025 12:35
@@ -396,19 +396,6 @@ allprojects {
tasks.matching { it.name.equals('integTest') }.configureEach { integTestTask ->
integTestTask.mustRunAfter tasks.matching { it.name.equals("test") }
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

just removing this unused stuff while being here.

breskeby added 2 commits May 6, 2025 13:06
Make InternalYamlRestTestPlugin test suites compatible

Make InternalYamlRestTest and JavaRestTestPlugin test suites compatible

Remove duplicate task dependency declaration of test tasks and check

Fix transport-netty4 internalClusterTest

Fix JavaREstTest wiring

Rework dependencies declaration for repository-s3

WIP

Organize repository-s3 tests in one project as test suites

[CI] Auto commit changes from spotless

Rely on auto detection of test framework in test suites

Default test task is handled differently in Gradle
@breskeby breskeby force-pushed the internal-cluster-test-using-testsuites branch from bcd83ea to d8e1897 Compare May 6, 2025 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v8.16.7 v8.17.7 v8.18.2 v8.19.0 v9.0.2 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants