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

jamesbognar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/juneau.git


The following commit(s) were added to refs/heads/master by this push:
     new 4956fedc37 Remove debug System.out leftovers and dedupe 
maven-jar-plugin (test-output audit cleanups)
4956fedc37 is described below

commit 4956fedc37ad306347985693fe726700cb5442eb
Author: James Bognar <[email protected]>
AuthorDate: Mon Jul 13 14:58:20 2026 -0400

    Remove debug System.out leftovers and dedupe maven-jar-plugin (test-output 
audit cleanups)
    
    Flagged by the TODO-219 mvn-test-output audit.
    
    Co-authored-by: Cursor <[email protected]>
---
 .../apache/juneau/commons/utils/StringUtils_Test.java |  2 --
 .../juneau/marshall/bson/BsonArrayDebug_Test.java     | 15 ---------------
 .../apache/juneau/marshall/uon/Common_UonTest.java    | 19 ++-----------------
 juneau-rest/juneau-rest-server-rdf/pom.xml            |  4 ----
 4 files changed, 2 insertions(+), 38 deletions(-)

diff --git 
a/juneau-core/juneau-commons/src/test/java/org/apache/juneau/commons/utils/StringUtils_Test.java
 
b/juneau-core/juneau-commons/src/test/java/org/apache/juneau/commons/utils/StringUtils_Test.java
index cd06e36bf6..ae827ecc9e 100755
--- 
a/juneau-core/juneau-commons/src/test/java/org/apache/juneau/commons/utils/StringUtils_Test.java
+++ 
b/juneau-core/juneau-commons/src/test/java/org/apache/juneau/commons/utils/StringUtils_Test.java
@@ -6554,9 +6554,7 @@ class StringUtils_Test extends TestBase {
                // - First '\' sees second '\', appends '\' and skips second '\'
                // - Then appends the second '\' from the string, then 'b'
                // Result: "a\\\\b" (both backslashes preserved)
-               System.out.println("=== Testing unescapeChars with a\\\\b, 
backslash NOT in escaped set ===");
                var result2 = unescapeChars("a\\\\b", AsciiSet.of(","));
-               System.out.println("Result: " + 
Arrays.toString(result2.toCharArray()));
                assertEquals("a\\\\b", result2); // '\' not in escaped set
        }
 
diff --git 
a/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/bson/BsonArrayDebug_Test.java
 
b/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/bson/BsonArrayDebug_Test.java
index df54733a88..7dcdf8323a 100644
--- 
a/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/bson/BsonArrayDebug_Test.java
+++ 
b/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/bson/BsonArrayDebug_Test.java
@@ -29,14 +29,6 @@ class BsonArrayDebug_Test {
        void dumpIntArrayBson() throws Exception {
                var s = 
BsonSerializer.create().keepNullProperties().addBeanTypes().addRootType().build();
                var bytes = s.serialize(ints(1, 2, 3));
-               // Print hex to see structure
-               var sb = new StringBuilder();
-               for (var i = 0; i < Math.min(bytes.length, 80); i++) {
-                       sb.append(String.format("%02x ", bytes[i] & 0xFF));
-                       if ((i + 1) % 16 == 0) sb.append("\n");
-               }
-               System.out.println("First 80 bytes (hex):");
-               System.out.println(sb);
                assertTrue(bytes.length > 0);
        }
 
@@ -45,13 +37,6 @@ class BsonArrayDebug_Test {
                var s = 
BsonSerializer.create().keepNullProperties().addBeanTypes().addRootType().build();
                var x = new 
org.apache.juneau.marshall.collections.JsonList("['abc',123]");
                var bytes = s.serialize(x);
-               var sb = new StringBuilder();
-               for (var i = 0; i < Math.min(bytes.length, 100); i++) {
-                       sb.append(String.format("%02x ", bytes[i] & 0xFF));
-                       if ((i + 1) % 16 == 0) sb.append("\n");
-               }
-               System.out.println("JsonList BSON (hex):");
-               System.out.println(sb);
                assertTrue(bytes.length > 0);
        }
 }
diff --git 
a/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/uon/Common_UonTest.java
 
b/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/uon/Common_UonTest.java
index d3d0e93d39..15dbd947e6 100755
--- 
a/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/uon/Common_UonTest.java
+++ 
b/juneau-core/juneau-marshall/src/test/java/org/apache/juneau/marshall/uon/Common_UonTest.java
@@ -187,23 +187,8 @@ class Common_UonTest extends TestBase {
                r2.r3 = r3;
                r3.r1 = r1;
 
-               // No recursion detection - DEBUG: Check what exception is 
thrown
-               assertThrowsWithMessage(Exception.class, "It's recommended you 
use the MarshallingTraverseContext.BEANTRAVERSE_detectRecursions setting to 
help locate the loop.", ()->{
-                       try {
-                               s.build().serialize(r1);
-                       } catch (Throwable t) {
-                               System.out.println("DEBUG: Exception type: " + 
cn(t));
-                               System.out.println("DEBUG: Exception class: " + 
t.getClass());
-                               System.out.println("DEBUG: Is 
StackOverflowError? " + (t instanceof StackOverflowError));
-                               System.out.println("DEBUG: Is 
SerializeException? " + (t instanceof 
org.apache.juneau.marshall.serializer.SerializeException));
-                               System.out.println("DEBUG: Stack trace (without 
message):");
-                               for (StackTraceElement ste : t.getStackTrace()) 
{
-                                       System.out.println("  " + ste);
-                               }
-                               // Re-throw to let assertThrowsWithMessage 
handle it
-                               throw t;
-                       }
-               });
+               // No recursion detection
+               assertThrowsWithMessage(Exception.class, "It's recommended you 
use the MarshallingTraverseContext.BEANTRAVERSE_detectRecursions setting to 
help locate the loop.", ()->s.build().serialize(r1));
 
                // Recursion detection, no ignore
                s.detectRecursions();
diff --git a/juneau-rest/juneau-rest-server-rdf/pom.xml 
b/juneau-rest/juneau-rest-server-rdf/pom.xml
index 417096412f..73fd340086 100644
--- a/juneau-rest/juneau-rest-server-rdf/pom.xml
+++ b/juneau-rest/juneau-rest-server-rdf/pom.xml
@@ -90,10 +90,6 @@
                                        </execution>
                                </executions>
                        </plugin>
-                       <plugin>
-                               <groupId>org.apache.maven.plugins</groupId>
-                               <artifactId>maven-jar-plugin</artifactId>
-                       </plugin>
                </plugins>
        </build>
 </project>

Reply via email to