Skip to content

docs: clarify Gradle test task structure in AGENTS.md#19004

Merged
jsvd merged 2 commits into
elastic:mainfrom
andrewvc:agents-md-test-task-docs
Apr 15, 2026
Merged

docs: clarify Gradle test task structure in AGENTS.md#19004
jsvd merged 2 commits into
elastic:mainfrom
andrewvc:agents-md-test-task-docs

Conversation

@andrewvc
Copy link
Copy Markdown
Contributor

@andrewvc andrewvc commented Apr 14, 2026

Summary

  • Document that --tests filtering must be used on javaTests or rubyTests directly, not the top-level test task (which excludes all classes and delegates)
  • Explain that some Java test classes (e.g. OutputDelegatorTest, ConfigCompilerTest) live in rubyTests due to Ruby infrastructure dependencies
  • Add example command for running a single test from rubyTests

Test plan

  • Verify AGENTS.md renders correctly on GitHub

🤖 Generated with Claude Code

Document that --tests filtering must be used on javaTests or rubyTests
directly (not the top-level test task), and that some Java test classes
live in rubyTests due to Ruby infrastructure dependencies.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
  • run exhaustive tests : Run the exhaustive tests Buildkite pipeline.

@andrewvc andrewvc marked this pull request as draft April 14, 2026 21:42
@andrewvc andrewvc self-assigned this Apr 14, 2026
@andrewvc andrewvc marked this pull request as ready for review April 14, 2026 21:49
@andrewvc andrewvc added backport-skip Skip automated backport with mergify and removed backport-skip Skip automated backport with mergify labels Apr 14, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 14, 2026

This pull request does not have a backport label. Could you fix it @andrewvc? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit.
  • If no backport is necessary, please add the backport-skip label

1 similar comment
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 14, 2026

This pull request does not have a backport label. Could you fix it @andrewvc? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit.
  • If no backport is necessary, please add the backport-skip label

@jsvd jsvd merged commit f7a42a5 into elastic:main Apr 15, 2026
12 checks passed
@andrewvc andrewvc deleted the agents-md-test-task-docs branch April 15, 2026 18:31
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