From 88bda60551644712da25614bbe51f58289c70328 Mon Sep 17 00:00:00 2001
From: "Erik C. Thauvin" <erik@thauvin.net>
Date: Mon, 19 May 2025 11:50:24 -0700
Subject: [PATCH] Add test tool options

---
 .../bld/extension/JacocoReportOperation.java  | 112 ++++++++++++++++++
 .../extension/JacocoReportOperationTest.java  |  21 ++++
 2 files changed, 133 insertions(+)

diff --git a/src/main/java/rife/bld/extension/JacocoReportOperation.java b/src/main/java/rife/bld/extension/JacocoReportOperation.java
index 1fae903..c135439 100644
--- a/src/main/java/rife/bld/extension/JacocoReportOperation.java
+++ b/src/main/java/rife/bld/extension/JacocoReportOperation.java
@@ -61,6 +61,10 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
      * The location of the source files.
      */
     final private Collection<File> sourceFiles_ = new ArrayList<>();
+    /**
+     * The test tool options.
+     */
+    final private Collection<String> testToolOptions_ = new ArrayList<>();
     /**
      * The location of the CSV report.
      */
@@ -212,6 +216,15 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
         return csv(csv.toFile());
     }
 
+    /**
+     * Returns the CSV report location.
+     *
+     * @return the CSV report location
+     */
+    public File csv() {
+        return csv_;
+    }
+
     /**
      * Sets the file to write execution data to.
      *
@@ -243,6 +256,15 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
         return destFile(destFile.toFile());
     }
 
+    /**
+     * Returns the file to write execution data to.
+     *
+     * @return the file to write execution data to
+     */
+    public File destFile() {
+        return destFile_;
+    }
+
     /**
      * Sets the source file encoding. The platform encoding is used by default.
      *
@@ -254,6 +276,15 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
         return this;
     }
 
+    /**
+     * Returns the source file encoding.
+     *
+     * @return the source file encoding
+     */
+    public String encoding() {
+        return encoding_;
+    }
+
     /**
      * Sets the locations of the JaCoCo *.exec files to read.
      *
@@ -356,6 +387,10 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
                     "org.jacoco.agent-" + JaCoCo.VERSION.substring(0, JaCoCo.VERSION.lastIndexOf('.'))
                             + "-runtime.jar").toFile(), "destfile=" + destFile_.getPath());
 
+            if (!testToolOptions_.isEmpty()) {
+                testOperation.testToolOptions().addAll(testToolOptions_);
+            }
+
             testOperation.execute();
 
             if (LOGGER.isLoggable(Level.INFO) && !silent()) {
@@ -439,6 +474,24 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
         return html(html.toFile());
     }
 
+    /**
+     * Returns the HTML report location.
+     *
+     * @return the HTML report location
+     */
+    public File html() {
+        return html_;
+    }
+
+    /**
+     * Return the status of the quiet flag.
+     *
+     * @return {@code true} if enabled, {@code false} otherwise
+     */
+    public boolean isQuiet() {
+        return silent();
+    }
+
     private ExecFileLoader loadExecFiles() throws IOException {
         var loader = new ExecFileLoader();
         if (execFiles_.isEmpty() && LOGGER.isLoggable(Level.WARNING) && !silent()) {
@@ -466,6 +519,15 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
         return this;
     }
 
+    /**
+     * Returns the name used for the report.
+     *
+     * @return the report name.
+     */
+    public String name() {
+        return reportName_;
+    }
+
     /**
      * Suppresses all output.
      *
@@ -594,6 +656,47 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
         return this;
     }
 
+    /**
+     * Returns the tab stop width for the source pages.
+     *
+     * @return the tab stop width
+     */
+    public int tabWidth() {
+        return tabWidth_;
+    }
+
+    /**
+     * Returns the test tool options.
+     *
+     * @return the test tool options
+     */
+    public Collection<String> testToolOptions() {
+        return testToolOptions_;
+    }
+
+    /**
+     * Sets the test tool options.
+     *
+     * @param options The options to set
+     * @return this operation instance
+     * @see #testToolOptions(Collection)
+     */
+    public JacocoReportOperation testToolOptions(String... options) {
+        return testToolOptions(List.of(options));
+    }
+
+    /**
+     * Sets the test tool options.
+     *
+     * @param options The options to set
+     * @return this operation instance
+     * @see #testToolOptions(String...)
+     */
+    public JacocoReportOperation testToolOptions(Collection<String> options) {
+        testToolOptions_.addAll(options);
+        return this;
+    }
+
     private void writeReports(IBundleCoverage bundle, ExecFileLoader loader)
             throws IOException {
         if (LOGGER.isLoggable(Level.INFO) && !silent()) {
@@ -642,4 +745,13 @@ public class JacocoReportOperation extends AbstractOperation<JacocoReportOperati
     public JacocoReportOperation xml(Path xml) {
         return xml(xml.toFile());
     }
+
+    /**
+     * Returns the XML report location.
+     *
+     * @return the XML report location
+     */
+    public File xml() {
+        return xml_;
+    }
 }
diff --git a/src/test/java/rife/bld/extension/JacocoReportOperationTest.java b/src/test/java/rife/bld/extension/JacocoReportOperationTest.java
index 44ead9d..a5b3dd4 100644
--- a/src/test/java/rife/bld/extension/JacocoReportOperationTest.java
+++ b/src/test/java/rife/bld/extension/JacocoReportOperationTest.java
@@ -123,6 +123,7 @@ class JacocoReportOperationTest {
     }
 
 
+    @Nested
     @DisplayName("Options Tests")
     class OptionsTests {
         public static final String FOO = "foo";
@@ -221,6 +222,26 @@ class JacocoReportOperationTest {
             op.xml(FOO);
             assertThat(op.xml()).isEqualTo(fooFile);
         }
+
+        @Nested
+        @DisplayName("Test Tool Options Tests")
+        class TestToolOptionsTests {
+            public static final String BAR = "bar";
+
+            @Test
+            void testToolOptionsAsArray() {
+                var op = new JacocoReportOperation();
+                op = op.testToolOptions(FOO, BAR);
+                assertThat(op.testToolOptions()).contains(FOO, BAR);
+            }
+
+            @Test
+            void testToolOptionsAsList() {
+                var op = new JacocoReportOperation();
+                op.testToolOptions(List.of(FOO, BAR));
+                assertThat(op.testToolOptions()).contains(FOO, BAR);
+            }
+        }
     }
 
     @Nested