📓 Add test for #32

build.gradle.kts

@@ -35,6 +35,7 @@ dependencies {
     compileOnly(libs.tab.api)
     compileOnly(libs.packet.events)
     implementation(libs.entity.lib)
+    testImplementation("org.junit.jupiter:junit-jupiter:5.7.1")
 }
 
 tasks {

src/main/java/com/mattmx/nametags/config/TextFormatter.java

@@ -22,21 +22,19 @@ public enum TextFormatter {
     )
     ;
 
-    // Converts legacy hex format &x&9&0&0&c&3&f to modern hex format &#900c3f
+    // Converts legacy hex format &x&9&0&0&c&3&f -> &#900c3f modern hex format
     // https://github.com/Matt-MX/DisplayNameTags/issues/32#issuecomment-2509403581
-    private static String convertLegacyHex(String input) {
-        //regex to match the legacy hex format
-        String legacyHexPattern = "&x(&[0-9a-fA-F]){6}";
-        Pattern pattern = Pattern.compile(legacyHexPattern);
-        Matcher matcher = pattern.matcher(input);
+    private static final Pattern LEGACY_HEX_PATTERN = Pattern.compile("&x(&[0-9a-fA-F]){6}");
+    public static String convertLegacyHex(String input) {
+        Matcher matcher = LEGACY_HEX_PATTERN.matcher(input);
 
         StringBuilder result = new StringBuilder();
         while (matcher.find()) {
             String legacyHex = matcher.group();
-            //extract hex digits from the legacy format
+            // Extract hex digits from the legacy format
             String hexColor = legacyHex.replace("&x", "")
                 .replace("&", "");
-            //replace with modern format
+            // Replace with modern format
             String modernHex = "&#" + hexColor;
             matcher.appendReplacement(result, modernHex);
         }

src/test/java/com/mattmx/nametags/config/TextFormatterTest.java

@@ -0,0 +1,22 @@
+package com.mattmx.nametags.config;
+
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.CsvSource;
+
+public class TextFormatterTest {
+
+    @ParameterizedTest
+    @CsvSource({
+        "&x&c&7&0&0&3&9[Admin],&#c70039[Admin]",
+        "&x&9&0&0&c&3&fCoyotea&r,&#900c3fCoyotea&r",
+        "&x&9&0&0&c&3&fCoyotea,&#900c3fCoyotea",
+        "&x&c&7&0&0&3&9[Admin] &x&9&0&0&c&3&fCoyotea&r,&#c70039[Admin] &#900c3fCoyotea&r"
+    })
+    public void test_convertLegacyHex(String input, String expectedOut) {
+        final String out = TextFormatter.convertLegacyHex(input);
+
+        Assertions.assertEquals(expectedOut, out);
+    }
+
+}