GH-22 Ensure a failure to clean up temporary files does not fail call

Author: Ryan O'Meara

method-analyzer-core/src/main/java/com/synopsys/method/analyzer/core/report/ReportGenerator.java

@@ -44,6 +44,9 @@
 
 import javax.annotation.Nullable;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import com.google.common.collect.Lists;
 import com.google.common.collect.Multimap;
 import com.google.gson.Gson;
@@ -72,6 +75,9 @@ public class ReportGenerator {
 
     private static final Gson GSON = new Gson();
 
+    /** Logger reference to output information to the application log files */
+    private final Logger logger = LoggerFactory.getLogger(getClass());
+
     private final MetaDataReportJson metaDataReport;
 
     /**
@@ -182,10 +188,14 @@ private Path writeReport(Path destinationFile, List<MethodIdJson> uniqueMethodKe
         writeZipFile(destinationFile, outputFileMapping);
 
         // GH-22: Explicitly clean up temporary files once use of them is complete
-        try (Stream<Path> walk = Files.walk(workingDirectory)) {
-            walk.sorted(Comparator.reverseOrder())
-                    .map(Path::toFile)
-                    .forEach(File::delete);
+        try {
+            try (Stream<Path> walk = Files.walk(workingDirectory)) {
+                walk.sorted(Comparator.reverseOrder())
+                        .map(Path::toFile)
+                        .forEach(File::delete);
+            }
+        } catch (IOException e) {
+            logger.warn("Error cleaning up temporary report files", e);
         }
 
         return destinationFile;