fix: proper private interface event handler methods deduplication

Author: zernix2077

server/src/main/java/org/allaymc/server/utils/ReflectionUtils.java

@@ -61,7 +61,13 @@ private static void collectAllMethods(Class<?> type, Map<String, Method> methods
     }
 
     public static String buildMethodSignature(Method method) {
-        return method.getName() + Arrays.toString(method.getParameterTypes());
+        var signature = method.getName() + Arrays.toString(method.getParameterTypes());
+
+        if (method.getDeclaringClass().isInterface() && Modifier.isPrivate(method.getModifiers())) {
+            return method.getDeclaringClass().getName() + "#" + signature;
+        }
+
+        return signature;
     }
 
     public static List<Method> getAllStaticVoidParameterlessMethods(Class<?> clazz) {
@@ -115,4 +121,3 @@ public static <T, U> BiMap<T, U> mapStaticFields(Class<?> classA, Class<?> class
         return result;
     }
 }
-

server/src/test/java/org/allaymc/server/utils/ReflectionUtilsTest.java

@@ -35,6 +35,12 @@ void testGetAllMethodsIncludesInterfaceDefaultMethods() {
         assertTrue(methods.stream().anyMatch(m -> m.getName().equals("defaultMethod")));
     }
 
+    @Test
+    void testGetAllMethodsDoesNotDeduplicatePrivateInterfaceMethodsAcrossInterfaces() {
+        var methods = ReflectionUtils.getAllMethods(F.class);
+        assertEquals(2, methods.stream().filter(m -> m.getName().equals("helper")).count());
+    }
+
     @Test
     void testMapStaticFields() {
         var map = ReflectionUtils.mapStaticFields(EnumA.class, EnumB.class);
@@ -74,13 +80,23 @@ default String defaultMethod() {
         }
     }
 
+    interface PrivateMethodA {
+        private void helper() {}
+    }
+
+    interface PrivateMethodB {
+        private void helper() {}
+    }
+
     static class C implements WithDefaultMethod {}
 
+    static class F implements PrivateMethodA, PrivateMethodB {}
+
     enum EnumA {
         A, B, C
     }
 
     enum EnumB {
         A, B
     }
-}
+}