Skip to content

test: diagnose flaky empty classpath in MavenLauncherTest.spoonMavenLauncherTest - #6915

Open
monperrus wants to merge 1 commit into
INRIA:masterfrom
monperrus:maven-launcher-classpath-diagnostic
Open

monperrus wants to merge 1 commit into
INRIA:masterfrom
monperrus:maven-launcher-classpath-diagnostic

Conversation

@monperrus

Copy link
Copy Markdown
Collaborator

`MavenLauncherTest.spoonMavenLauncherTest` failed once in #6890 (`expected: not equal but was: <0>` on the ALL_SOURCE classpath) and passed on rerun.

An empty classpath means a `spoon.classpath*.tmp` file existed but was empty (a missing file throws instead). Cached files younger than 1h are reused without invoking Maven, and the test copies the whole working directory, including its cached files, into `spooned/`. A stale empty file is the prime suspect: planting an empty `spoon.classpath.tmp` at the repo root reproduces the exact failure locally.

This PR only adds a failure message listing path, size, mtime and content of the cached classpath files in both the copied project and the working directory, so the next occurrence tells us where the empty file came from.

🤖 Generated with Claude Code

…auncherTest

The ALL_SOURCE classpath assertion failed once in CI (INRIA#6890) with an empty
classpath and passed on rerun. An empty classpath means a spoon.classpath*.tmp
file existed but was empty; since cached files younger than 1h are reused
without invoking Maven, a stale empty file (copied from the working directory
along with the project) is the prime suspect.

On failure, report path, size, mtime and content of the cached classpath files
in both the copied project and the working directory.

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

This branch has not been deployed

No deployments
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