Skip to content

chore: fix stale test count, javadoc escape, and baseline gitignore guard - #27

Merged
shossain786 merged 1 commit into
masterfrom
chore/test-doc-and-gitignore-cleanups
Jul 18, 2026
Merged

shossain786 merged 1 commit into
masterfrom
chore/test-doc-and-gitignore-cleanups

Conversation

@shossain786

Copy link
Copy Markdown
Contributor

Follow-up cleanups surfaced by a release-readiness mvn clean verify run of v3.1.1 (480 tests, 0 failures — all green). None block a release; these are hygiene.

Changes

  • CLAUDE.md — corrected the stale "12 test classes" to "39 test classes, 480 tests" so the doc stops under-reporting coverage.
  • DriverManager.java — escaped an unescaped & as &amp; in a javadoc <li> (cleared the javadoc build warning; jar built fine regardless).
  • .gitignore — added src/test/resources/baselines/*.png guard. VisualAssert tests write throwaway timestamped baseline PNGs there during runs; this prevents a failed cleanup from dirtying the tree or accidentally committing throwaway images.

🤖 Generated with Claude Code

…uard

Follow-ups from a release-readiness test run of v3.1.1:

- CLAUDE.md: correct "12 test classes" to "39 test classes, 480 tests".
- DriverManager: escape unescaped `&` as `&amp;` in a javadoc <li>.
- .gitignore: guard src/test/resources/baselines/*.png so a failed
  VisualAssert cleanup can't dirty the tree or commit throwaway images.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@shossain786 shossain786 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

looks fine

@shossain786
shossain786 merged commit 8e6a175 into master Jul 18, 2026
2 checks passed
@shossain786
shossain786 deleted the chore/test-doc-and-gitignore-cleanups branch July 18, 2026 06:32
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