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>