(jackrabbit-oak) 02/02: OAK-11570 oak-run fullGC fullGcMaxAge default value is wrongly calculated

[joscorbe]

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

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

commit 709fab2f6506336e6eb0002257bee125d689d9ef
Author: Daniel Iancu <ia...@adobe.com>
AuthorDate: Thu Mar 13 12:20:38 2025 +0200

    OAK-11570 oak-run fullGC fullGcMaxAge default value is wrongly calculated
---
 .../oak/plugins/document/RevisionsCommandTest.java | 32 +++++++++++-----------
 1 file changed, 16 insertions(+), 16 deletions(-)

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 adaef8e28c..777dbad49e 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
@@ -191,18 +191,18 @@ public class RevisionsCommandTest {
         ns.dispose();
 
         String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--entireRepo"));
-        assertTrue(output.contains("DryRun is enabled : true"));
-        assertTrue(output.contains("ResetFullGC is enabled : false"));
-        assertTrue(output.contains("Compaction is enabled : false"));
-        assertTrue(output.contains("starting gc collect"));
-        assertTrue(output.contains("IncludePaths are : [/]"));
-        assertTrue(output.contains("ExcludePaths are : []"));
-        assertTrue(output.contains("FullGcMode is : 0"));
-        assertTrue(output.contains("FullGcDelayFactory is : 2.0"));
-        assertTrue(output.contains("FullGcBatchSize is : 1000"));
-        assertTrue(output.contains("FullGcProgressSize is : 10000"));
+        assertTrue(output.contains("DryRun is enabled : true\n"));
+        assertTrue(output.contains("ResetFullGC is enabled : false\n"));
+        assertTrue(output.contains("Compaction is enabled : false\n"));
+        assertTrue(output.contains("starting gc collect\n"));
+        assertTrue(output.contains("IncludePaths are : [/]\n"));
+        assertTrue(output.contains("ExcludePaths are : []\n"));
+        assertTrue(output.contains("FullGcMode is : 0\n"));
+        assertTrue(output.contains("FullGcDelayFactory is : 2.0\n"));
+        assertTrue(output.contains("FullGcBatchSize is : 1000\n"));
+        assertTrue(output.contains("FullGcProgressSize is : 10000\n"));
         assertTrue(output.contains("FullGcMaxAgeInSecs is : 86400\n"));
-        assertTrue(output.contains("FullGcMaxAgeMillis is : 86400000"));
+        assertTrue(output.contains("FullGcMaxAgeMillis is : 86400000\n"));
     }
 
     @Test
@@ -210,8 +210,8 @@ public class RevisionsCommandTest {
         ns.dispose();
 
         String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--fullGcMaxAge", "10000", "--entireRepo"));
-        assertTrue(output.contains("FullGcMaxAgeInSecs is : 10000"));
-        assertTrue(output.contains("FullGcMaxAgeMillis is : 10000000"));
+        assertTrue(output.contains("FullGcMaxAgeInSecs is : 10000\n"));
+        assertTrue(output.contains("FullGcMaxAgeMillis is : 10000000\n"));
         assertTrue(output.contains("starting gc collect"));
     }
 
@@ -220,7 +220,7 @@ public class RevisionsCommandTest {
         ns.dispose();
 
         String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--fullGcDelayFactor", "2.5", "--entireRepo"));
-        assertTrue(output.contains("FullGcDelayFactory is : 2.5"));
+        assertTrue(output.contains("FullGcDelayFactory is : 2.5\n"));
         assertTrue(output.contains("starting gc collect"));
     }
 
@@ -229,7 +229,7 @@ public class RevisionsCommandTest {
         ns.dispose();
 
         String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--fullGcBatchSize", "200", "--entireRepo"));
-        assertTrue(output.contains("FullGcBatchSize is : 200"));
+        assertTrue(output.contains("FullGcBatchSize is : 200\n"));
         assertTrue(output.contains("starting gc collect"));
     }
 
@@ -238,7 +238,7 @@ public class RevisionsCommandTest {
         ns.dispose();
 
         String output = captureSystemOut(new RevisionsCmd("fullGC", 
"--fullGcProgressSize", "20000", "--entireRepo"));
-        assertTrue(output.contains("FullGcProgressSize is : 20000"));
+        assertTrue(output.contains("FullGcProgressSize is : 20000\n"));
         assertTrue(output.contains("starting gc collect"));
     }