From ab284edcbd00043ae343cfb7aad3857c614e5d6b Mon Sep 17 00:00:00 2001
From: Markus KARG <markus@headcrashing.eu>
Date: Sat, 30 Nov 2024 15:43:14 +0100
Subject: [PATCH] Add violation for StringReader

---
 .../src/main/resources/modernizer.xml                 |  6 ++++++
 .../gaul/modernizer_maven_plugin/ModernizerTest.java  | 11 ++++++++++-
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/modernizer-maven-plugin/src/main/resources/modernizer.xml b/modernizer-maven-plugin/src/main/resources/modernizer.xml
index 4ab32c9..3d86840 100644
--- a/modernizer-maven-plugin/src/main/resources/modernizer.xml
+++ b/modernizer-maven-plugin/src/main/resources/modernizer.xml
@@ -425,6 +425,12 @@ violation names use the same format that javap emits.
     <comment>Prefer java.io.OutputStreamWriter(java.io.OutputStream, java.nio.charset.Charset)</comment>
   </violation>
 
+  <violation>
+    <name>java/io/StringReader."&lt;init&gt;":(Ljava/lang/String;)V</name>
+    <version>24</version>
+    <comment>Prefer java.io.Reader.of(java.lang.CharSequence)</comment>
+  </violation>
+
   <violation>
     <name>java/lang/Byte."&lt;init&gt;":(B)V</name>
     <version>5</version>
diff --git a/modernizer-maven-plugin/src/test/java/org/gaul/modernizer_maven_plugin/ModernizerTest.java b/modernizer-maven-plugin/src/test/java/org/gaul/modernizer_maven_plugin/ModernizerTest.java
index 9de99c8..4aa266d 100644
--- a/modernizer-maven-plugin/src/test/java/org/gaul/modernizer_maven_plugin/ModernizerTest.java
+++ b/modernizer-maven-plugin/src/test/java/org/gaul/modernizer_maven_plugin/ModernizerTest.java
@@ -28,6 +28,7 @@
 import java.io.OutputStreamWriter;
 import java.io.PrintStream;
 import java.io.PrintWriter;
+import java.io.StringReader;
 import java.net.HttpURLConnection;
 import java.net.NetworkInterface;
 import java.net.URLDecoder;
@@ -399,7 +400,7 @@ public void testAnnotationViolation() throws Exception {
 
     @Test
     public void testAllViolations() throws Exception {
-        Modernizer modernizer = createModernizer("19");
+        Modernizer modernizer = createModernizer("24");
         Collection<ViolationOccurrence> occurrences = modernizer.check(
                 new ClassReader(AllViolations.class.getName()));
         occurrences.addAll(modernizer.check(
@@ -418,6 +419,8 @@ public void testAllViolations() throws Exception {
             new ClassReader(Java18Violations.class.getName())));
         occurrences.addAll(modernizer.check(
             new ClassReader(Java19Violations.class.getName())));
+        occurrences.addAll(modernizer.check(
+            new ClassReader(Java24Violations.class.getName())));
         // must visit inner classes manually
         occurrences.addAll(modernizer.check(
                 new ClassReader(EnumerationTestClass.class.getName())));
@@ -847,6 +850,12 @@ private static void method() throws Exception {
         }
     }
 
+    private static class Java24Violations {
+        private static void method() throws Exception {
+            new StringReader("");
+        }
+    }
+
     private enum EnumClass {
     }
 }