Skip to content

Add nested module discovery coverage#52

Merged
Will-thom merged 1 commit into
mainfrom
issue-6-nested-module-tests
May 26, 2026
Merged

Add nested module discovery coverage#52
Will-thom merged 1 commit into
mainfrom
issue-6-nested-module-tests

Conversation

@Will-thom
Copy link
Copy Markdown
Collaborator

Summary

  • add a nested Maven module fixture rooted at platform/service-core
  • verify recursive module discovery includes root, parent, and nested module paths
  • verify Surefire report mapping and reproduce command use the nested module path

Validation

  • go test ./...
  • go run ./cmd/prmaven fails -project pkg/prmaven/testdata/nested-module-project

Closes #6

@github-actions
Copy link
Copy Markdown

Hi @Will-thom, thank you for contributing to PR Maven CLI.

I appreciate the time you spent opening this pull request. This project is intentionally shaped for focused, reviewable open source contributions, so your input helps the tool become more useful for Java and Maven teams.

A maintainer will review the context, labels, and next steps as soon as possible. If this is your first contribution here, welcome aboard.

Useful links:

  • Contributing guide: CONTRIBUTING.md
  • Testing guide: docs/testing.md
  • CI/CD guide: docs/ci.md

Thank you for helping build a deterministic, local-first Maven failure triage tool.

@Will-thom Will-thom merged commit e853d1d into main May 26, 2026
21 checks passed
@Will-thom Will-thom deleted the issue-6-nested-module-tests branch May 26, 2026 23:10
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.

[Stage 2] Add nested module discovery tests

1 participant