Issue #16361: add comment on testAddException

Carbon14-48 · romani · commit bbb00f943b5c · 2026-03-27T19:31:59.000-07:00
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/DefaultLoggerTest.java b/src/test/java/com/puppycrawl/tools/checkstyle/DefaultLoggerTest.java
@@ -308,6 +308,14 @@ public void testNewCtor() throws Exception {
         }
     }
 
+    /**
+     * Direct invocation of {@code addException} is necessary because
+     * {@code Checker} implementation does not call {@code addException}
+     * during normal file processing flow, leaving mutations on lines 179-183
+     * ({@code PrintWriter::println}, {@code getLocalizedMessage},
+     * {@code AuditEvent::getFileName}, {@code Throwable::printStackTrace})
+     * with no coverage if tested only through {@code verifyWithInlineConfigParserAndDefaultLogger}.
+     */
     @Test
     public void testAddException() throws Exception {
         try (MockByteArrayOutputStream errorStream = new MockByteArrayOutputStream()) {

Original file line number	Diff line number	Diff line change
`@@ -308,6 +308,14 @@ public void testNewCtor() throws Exception {`
`308`	`308`	`}`
`309`	`309`	`}`
`310`	`310`
	`311`	`+ /**`
	`312`	`+ * Direct invocation of {@code addException} is necessary because`
	`313`	`+ * {@code Checker} implementation does not call {@code addException}`
	`314`	`+ * during normal file processing flow, leaving mutations on lines 179-183`
	`315`	`+ * ({@code PrintWriter::println}, {@code getLocalizedMessage},`
	`316`	`+ * {@code AuditEvent::getFileName}, {@code Throwable::printStackTrace})`
	`317`	`+ * with no coverage if tested only through {@code verifyWithInlineConfigParserAndDefaultLogger}.`
	`318`	`+ */`
`311`	`319`	`@Test`
`312`	`320`	`public void testAddException() throws Exception {`
`313`	`321`	`try (MockByteArrayOutputStream errorStream = new MockByteArrayOutputStream()) {`