(jackrabbit-oak) 02/02: OAK-11795: Added test cases.

[joscorbe]

This is an automated email from the ASF dual-hosted git repository.

joscorbe pushed a commit to branch OAK-11795
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git

commit 077c5e777af7e1fac2b2aba39fb59732c7dbc35f
Author: Jose Cordero <corde...@adobe.com>
AuthorDate: Tue Jul 8 10:39:25 2025 +0200

    OAK-11795: Added test cases.
---
 .../jackrabbit/oak/run/RevisionsCommand.java       |  4 ++--
 .../oak/plugins/document/RevisionsCommandTest.java | 27 ++++++++++++++++++++++
 2 files changed, 29 insertions(+), 2 deletions(-)

diff --git 
a/oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java 
b/oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java
index 76613b6a78..6f765b12d4 100644
--- a/oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java
+++ b/oak-run/src/main/java/org/apache/jackrabbit/oak/run/RevisionsCommand.java
@@ -218,7 +218,7 @@ public class RevisionsCommand implements Command {
                             "nodes for Full GC which are not accessed recently 
(currentTime - lastModifiedTime > fullGcMaxAge). Default: 86400 (one day)")
                     
.withOptionalArg().ofType(Long.class).defaultsTo(TimeUnit.DAYS.toSeconds(1));
             fullGCAuditLoggingEnabled = 
parser.accepts("fullGCAuditLoggingEnabled", "Enable audit logging for Full GC")
-                    .withOptionalArg().ofType(Boolean.class).defaultsTo(FALSE);
+                    .withRequiredArg().ofType(Boolean.class).defaultsTo(FALSE);
             exportMetrics = parser.accepts("exportMetrics",
                     "type, URI to export the metrics and optional metadata all 
delimeted by semi-colon(;)").withRequiredArg();
         }
@@ -321,7 +321,7 @@ public class RevisionsCommand implements Command {
         }
 
         Boolean isFullGCAuditLoggingEnabled() {
-            return options.has(fullGCAuditLoggingEnabled);
+            return options.valueOf(fullGCAuditLoggingEnabled);
         }
 
         boolean exportMetrics() {
diff --git 
a/oak-run/src/test/java/org/apache/jackrabbit/oak/plugins/document/RevisionsCommandTest.java
 
b/oak-run/src/test/java/org/apache/jackrabbit/oak/plugins/document/RevisionsCommandTest.java
index 3fc023b6f8..1bc4669e0f 100644
--- 
a/oak-run/src/test/java/org/apache/jackrabbit/oak/plugins/document/RevisionsCommandTest.java
+++ 
b/oak-run/src/test/java/org/apache/jackrabbit/oak/plugins/document/RevisionsCommandTest.java
@@ -372,6 +372,33 @@ public class RevisionsCommandTest {
         assertTrue(errOutput.contains("--path or --entireRepo option is 
required for fullGC command"));
     }
 
+    @Test
+    public void auditLoggingEnabled() {
+        ns.dispose();
+
+        String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--entireRepo", "--fullGCAuditLoggingEnabled", "true"));
+        assertTrue(output.contains("FullGCAuditLoggingEnabled : true"));
+        assertTrue(output.contains("starting gc collect"));
+    }
+
+    @Test
+    public void auditLoggingDisabled() {
+        ns.dispose();
+
+        String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--entireRepo", "--fullGCAuditLoggingEnabled", "false"));
+        assertTrue(output.contains("FullGCAuditLoggingEnabled : false"));
+        assertTrue(output.contains("starting gc collect"));
+    }
+
+    @Test
+    public void auditLoggingNotSet() {
+        ns.dispose();
+
+        String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--entireRepo"));
+        assertTrue(output.contains("FullGCAuditLoggingEnabled : false"));
+        assertTrue(output.contains("starting gc collect"));
+    }
+
     @Test
     public void fullGCWithEmbeddedWithPath() {
         ns.dispose();