Disambiguate test source path resolution

[mkutz]

Summary

• Exclude bin and out directories from test source file search (fixes IntelliJ copying sources into bin)
• Prefer paths containing src when multiple matches are found, resolving ambiguity from output directories not explicitly excluded
• Throw a descriptive error when the source file location is still ambiguous after filtering

Closes #200

Test plan

• Existing tests for build and target exclusion still pass
• New test for bin directory exclusion
• New test for src path preference when multiple matches exist
• New test for ambiguous paths (multiple src matches) throwing error
• New test for missing source file throwing error

[Copilot]

Pull request overview

This PR addresses issue #200 where test source files were sometimes incorrectly referenced from build output directories (like bin) instead of actual source directories. The solution adds exclusions for bin and out directories, implements a disambiguation strategy that prefers paths containing src, and provides clear error messages when the source file cannot be found or is ambiguous.

Changes:

• Extended the directory exclusion regex pattern to exclude bin and out directories in addition to build and target
• Replaced the "first match wins" strategy with a comprehensive disambiguation approach that collects all matches and prefers paths containing src when multiple matches exist
• Added descriptive error handling for missing and ambiguous source file scenarios

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
modules/core/src/main/java/org/approvej/approve/StackTraceTestFinderUtil.java	Updated path filtering regex to exclude bin and out directories; replaced simple first-match logic with disambiguation strategy that prefers src paths
modules/core/src/test/java/org/approvej/approve/StackTraceTestFinderUtilTest.java	Added comprehensive tests for bin directory exclusion, missing source files, src path preference, and ambiguous path scenarios

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The error message indicates how many paths contain 'src' but then shows all matches. This could be confusing if srcMatches.size() differs from matches.size(). Consider either showing only srcMatches in the error message, or clarifying that all matches are being shown. For example: "Found multiple test source files (%d total, %d contain 'src'): %s".formatted(matches.size(), srcMatches.size(), matches)

Suggested change

	"Found multiple test source files (%d contain 'src'): %s"
	.formatted(srcMatches.size(), matches));
	"Found multiple test source files (%d total, %d contain 'src'): %s"
	.formatted(matches.size(), srcMatches.size(), matches));