Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate junit-platform.properties warning should also print the duplicate URLs #3898

Closed
huxi opened this issue Jul 25, 2024 · 3 comments
Closed

Comments

@huxi
Copy link

huxi commented Jul 25, 2024

Due to a bug in Kafka, we currently see several warnings in our build that look like this:
Discovered 2 'junit-platform.properties' configuration files in the classpath; only the first will be used.

While this is certainly true, I had to write code to find out what's actually going on and pinpoint the issue to Kafka.

This could have been prevented if the warning message would have also included the multiple URLs.

This is not a big thing but a low-hanging fruit to slightly enhance the user experience in such a situation.

@sormuras
Copy link
Member

Sounds like a good proposal to me. It's also related to:

@sbrannen
Copy link
Member

This could have been prevented if the warning message would have also included the multiple URLs.

Sounds reasonable to me.

We should be able to adapt org.junit.platform.launcher.core.LauncherConfigurationParameters.loadClasspathResource(String) easily to support this.

@marcphilipp
Copy link
Member

I'll take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants