fix: issues on file normalization

Author: jpedroh

src/main/services/dataCollectors/fileSyntacticNormalization/BaseFileSyntacticNormalizationDataCollector.groovy

@@ -27,6 +27,16 @@ abstract class BaseFileSyntacticNormalizationDataCollector implements DataCollec
         def files = MergeScenarioCollector.collectNonFastForwardMergeScenarios(project, mergeCommit)
         files.parallelStream().forEach(file -> {
             LOG.debug("Starting to run file normalization in file ${inputFile}")
+
+            if (!Files.exists(file.resolve(inputFile))) {
+                LOG.debug("Skipping normalization because file ${file.resolve(inputFile)} do not exist")
+                return
+            }
+            if (!Files.exists(file.resolve(outputFile))) {
+                LOG.debug("Skipping normalization because file ${file.resolve(outputFile)} do not exist")
+                return
+            }
+
             def isSuccess = runNormalizationOnFile(file.resolve(inputFile), file.resolve(outputFile))
             if (!isSuccess) {
                 LOG.debug("Failed to run file normalization in file ${inputFile}, falling back to copy the file")

src/main/services/dataCollectors/fileSyntacticNormalization/JDimeFileSyntacticNormalizationDataCollector.groovy

@@ -26,6 +26,6 @@ class JDimeFileSyntacticNormalizationDataCollector extends BaseFileSyntacticNorm
 
         def output = ProcessRunner.startProcess(processBuilder)
         def hasCompleted = output.waitFor(1, TimeUnit.HOURS)
-        return hasCompleted && output.exitValue() != 0
+        return hasCompleted && output.exitValue() == 0
     }
 }

src/main/services/dataCollectors/fileSyntacticNormalization/SporkFileSyntacticNormalizationDataCollector.groovy

@@ -4,7 +4,6 @@ import org.apache.logging.log4j.LogManager
 import org.apache.logging.log4j.Logger
 import util.ProcessRunner
 
-import java.nio.file.Files
 import java.nio.file.Path
 import java.util.concurrent.TimeUnit
 
@@ -27,7 +26,7 @@ class SporkFileSyntacticNormalizationDataCollector extends BaseFileSyntacticNorm
         process.getInputStream().eachLine(LOG::trace)
         process.getErrorStream().eachLine(LOG::warn)
         def hasCompleted = process.waitFor(1, TimeUnit.HOURS)
-        return hasCompleted && Files.exists(outputFile)
+        return hasCompleted && process.exitValue() == 0
     }
 
     private static List<String> getBuildParameters(Path inputFile, Path outputFile) {