Skip to content

Describe the unit and integration tests in AGENTS.md - #60

Merged
slominskir merged 1 commit into
mainfrom
docs/agents-integration-tests
Oct 28, 2026
Merged

slominskir merged 1 commit into
mainfrom
docs/agents-integration-tests

Conversation

@slominskir-coding-agent

Copy link
Copy Markdown
Contributor

What changed

AGENTS.md's Development section said "There are no tests yet". Since then:

  • ./gradlew build runs the unit tests in src/test/java, and so does CI.
  • src/integration/java has integration tests (PagesIT, ReadinessIT, StatusReportsIT, MaskingRequestIT) that ./gradlew integrationTest runs against the app in containers. Neither build nor CI runs them.

The section now says:

  • which tests build runs;
  • that agents run the integration tests themselves for a change to behavior or pages;
  • that they add a case for what they change, and for a bug fix show it failing without the fix;
  • that the masking request tests write components and read MailHog;
  • where the README covers SRM_URL, KEYCLOAK_URL and MAILHOG_URL.

The pre-PR step now runs the integration tests after rebuilding the app.

The shared rules block isn't touched. Documentation only: no upgrade steps.

Checks

  • ./gradlew build: passed, in 13 s from clean and 1 s warm. The "about 20 s cold" figure still fits, so I left it.
  • I checked the commands, paths, variable names and the README's "Develop" heading against the repository.

Decisions

Prepared-by: Anthropic Claude Code:claude-opus-5-5

🤖 Generated with Claude Code

AGENTS.md still said the project had no tests, though `./gradlew build`
now runs the unit tests in src/test/java, and src/integration/java holds
integration tests that `./gradlew integrationTest` runs against the app
in containers. Neither build nor CI runs those, so agents skipped them
or treated the first test as theirs to set up.

Say what each kind covers, that agents run the integration tests
themselves for changes to behavior or pages and add a case for what
they change, and where the README documents the settings for other
ports.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@slominskir
slominskir merged commit 6014e60 into main Oct 28, 2026
5 checks passed
@slominskir
slominskir deleted the docs/agents-integration-tests branch October 28, 2026 20:21
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.

1 participant