fix: Redirect the logs from PMD to STDERR when writing to file CF-2115

It seems that when PMD is ran with an output file it logs the info
messages to the STDOUT. But when returning results to the STDOUT
does not do those logs (as most tools)

On the scope of our CLI, we usually propagate a temporary file location
even if we output to the STDOUT, so when such file is propagated to
PMD, we need to make sure to redirect the STDOUT to STDERR

Author: machadoit

tools/pmdRunner.go

@@ -86,11 +86,15 @@ func RunPmd(repositoryToAnalyseDirectory string, pmdBinary string, pathsToCheck
 	// Output file
 	if outputFile != "" {
 		cmd.Args = append(cmd.Args, "-r", outputFile)
+		// When storing results in a file, all the logs output should go to stderr
+		// Note that for formats like SARIF, tools output their results to a temporary file
+		cmd.Stdout = os.Stderr
+	} else {
+		cmd.Stdout = os.Stdout
 	}
 
-	cmd.Dir = repositoryToAnalyseDirectory
 	cmd.Stderr = os.Stderr
-	cmd.Stdout = os.Stdout
+	cmd.Dir = repositoryToAnalyseDirectory
 
 	// Get Java runtime info
 	javaRuntime := config.Runtimes()["java"]