Skip to content

Add AGENTS.md with AssertJ/JUnit5 test migration guidelines - #3728

Open
nastra wants to merge 1 commit into
apache:masterfrom
nastra:add-agents-file
Open

Add AGENTS.md with AssertJ/JUnit5 test migration guidelines#3728
nastra wants to merge 1 commit into
apache:masterfrom
nastra:add-agents-file

Conversation

@nastra

@nastra nastra commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Rationale for this change

This adds guidelines that I've been defining/using for Claude when migrating the entire codebase from JUnit4 to JUnit5 + AssertJ

What changes are included in this PR?

An AGENTS.md file that contains migration guidelines

Are these changes tested?

Are there any user-facing changes?

@nastra

nastra commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

/cc @Fokko @wgtmac

Comment thread AGENTS.md
@dossett

dossett commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

I'm glad to see an AGENTS.md file added, this is great. This is a LOT of context about writing unit tests, which is also great. But it might be too much context for the agent to always have, even when doing things not related to unit tests. A pattern I've found very useful is to create some agent-specific documentation in a separate file and add a pointer to that in AGENTS.md. Then the agent always has the context of where to find the information when it needs it.

## Unit tests
When writing, updating, or migrating unit tests please read the reference document [e.g., `./agents/docs/unit-testing.md`]

Something like that

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