Skip to content

Commit

Permalink
[SUREFIRE-2182] Log starter implementation on DEBUG level
Browse files Browse the repository at this point in the history
For fork starter log the underlying configuration class name as well
  • Loading branch information
kwin committed Jun 21, 2023
1 parent 0998f10 commit 8861f50
Showing 1 changed file with 6 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -265,8 +265,10 @@ public abstract class AbstractSurefireMojo extends AbstractMojo implements Suref

/**
* Additional elements to be appended to the classpath.
* Each item must have the format
*
* @since 2.4
* @see <a href="https://docs.oracle.com/javase/8/docs/technotes/tools/windows/classpath.html">Java Class Path</a>
*/
@Parameter(property = "maven.test.additionalClasspath")
private String[] additionalClasspathElements;
Expand Down Expand Up @@ -1166,7 +1168,7 @@ private RunResult executeProvider(
(Properties) System.getProperties().clone();
try {
createCopyAndReplaceForkNumPlaceholder(effectiveProperties, 1).copyToSystemProperties();

getConsoleLogger().debug("Using in-process starter");
InPluginVMSurefireStarter surefireStarter = createInprocessStarter(
provider,
classLoaderConfiguration,
Expand All @@ -1181,6 +1183,9 @@ private RunResult executeProvider(
} else {
ForkConfiguration forkConfiguration = createForkConfiguration(platform, resolvedJavaModularityResult);
if (getConsoleLogger().isDebugEnabled()) {
getConsoleLogger()
.debug("Using fork starter with configuration "
+ forkConfiguration.getClass().getName());
showMap(getEnvironmentVariables(), "environment variable");
showArray(getExcludedEnvironmentVariables(), "excluded environment variable");
}
Expand Down

0 comments on commit 8861f50

Please sign in to comment.