From 31b7967390e705eed2a440a341b43013f6fbf58a Mon Sep 17 00:00:00 2001
From: "Erik C. Thauvin" <erik@thauvin.net>
Date: Sat, 12 Aug 2023 00:59:54 -0700
Subject: [PATCH] Added more logging

---
 README.md                                           |  0
 .../rife/bld/extension/JacocoReportOperation.java   | 13 +++++++++++--
 .../bld/extension/JacocoReportOperationTest.java    |  1 +
 3 files changed, 12 insertions(+), 2 deletions(-)
 mode change 100755 => 100644 README.md

diff --git a/README.md b/README.md
old mode 100755
new mode 100644
diff --git a/src/main/java/rife/bld/extension/JacocoReportOperation.java b/src/main/java/rife/bld/extension/JacocoReportOperation.java
index f0fcebc..248d07a 100644
--- a/src/main/java/rife/bld/extension/JacocoReportOperation.java
+++ b/src/main/java/rife/bld/extension/JacocoReportOperation.java
@@ -169,7 +169,7 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
 //            project.testOperation().fromProject(project).javaOptions().javaAgent(
 //                    Path.of(project.libBldDirectory().getPath(), "org.jacoco.agent-"
 //                            + JaCoCo.VERSION.substring(0, JaCoCo.VERSION.lastIndexOf('.')) + "-runtime.jar").toFile(),
-//                    "destfile=" + Path.of(buildJacocoExecDir.getPath(), "jacoco.exec"));
+//                    "destfile=" + destFile.getPath());
                 project.testOperation().fromProject(project).javaOptions().add("-javaagent:" +
                         Path.of(project.libBldDirectory().getPath(), "org.jacoco.agent-"
                                 + JaCoCo.VERSION.substring(0, JaCoCo.VERSION.lastIndexOf('.')) + "-runtime.jar")
@@ -180,6 +180,10 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
                     throw new IOException(e);
                 }
 
+                if (LOGGER.isLoggable(Level.INFO) && !quiet) {
+                    LOGGER.log(Level.INFO, "Execution Data: {0}", destFile);
+                }
+
                 if (buildJacocoExec.exists()) {
                     execFiles.add(buildJacocoExec);
                 }
@@ -237,7 +241,7 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
         } else {
             for (var f : execFiles) {
                 if (LOGGER.isLoggable(Level.INFO) && !quiet) {
-                    LOGGER.log(Level.INFO, "Loading execution data file: {0}",
+                    LOGGER.log(Level.INFO, "Loading execution data: {0}",
                             f.getAbsolutePath());
                 }
                 loader.load(f);
@@ -319,6 +323,11 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
                 loader.getExecutionDataStore().getContents());
         visitor.visitBundle(bundle, sourceLocator());
         visitor.visitEnd();
+        if (LOGGER.isLoggable(Level.INFO) && !quiet) {
+            LOGGER.log(Level.INFO, "XML Report: file://{0}", xml);
+            LOGGER.log(Level.INFO, "CSV Report: file://{0}", csv);
+            LOGGER.log(Level.INFO, "HTML Report: file://{0}/index.html", html);
+        }
     }
 
     /**
diff --git a/src/test/java/rife/bld/extension/JacocoReportOperationTest.java b/src/test/java/rife/bld/extension/JacocoReportOperationTest.java
index 7f8d257..b91a3c1 100644
--- a/src/test/java/rife/bld/extension/JacocoReportOperationTest.java
+++ b/src/test/java/rife/bld/extension/JacocoReportOperationTest.java
@@ -63,6 +63,7 @@ class JacocoReportOperationTest {
                     s.contains("<counter type=\"INSTRUCTION\" missed=\"0\" covered=\"3\"/>"))).isTrue();
         }
         assertThat(Path.of(html.getPath(), "com.example", "Examples.java.html")).exists();
+
         deleteOnExit(tempDir.toFile());
     }