Added reflection testing

Voinea-Radu · Sep 5, 2024 · cd60fd0 · cd60fd0
1 parent d4458b7
commit cd60fd0
Show file tree

Hide file tree

Showing 15 changed files with 219 additions and 33 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -40,6 +40,8 @@ dependencies {
     // Tests
     testImplementation(platform(libs.junit.bom))
     testImplementation(libs.junit.jupiter)
+    testImplementation(libs.slf4j.log4j)
+    testImplementation(libs.log4j)
 }
 
 tasks {

diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml
@@ -17,10 +17,12 @@ jetbrains_annotations = { module = "org.jetbrains:annotations", version = "24.1.
 # Dependencies
 gson = { module = "com.google.code.gson:gson", version = "2.11.0" }
 jedis = { module = "redis.clients:jedis", version = "5.1.5" }
-slf4j = { module = "org.slf4j:slf4j-api", version = "2.0.16" }
 apache_commons_compress = { module = "org.apache.commons:commons-compress", version = "1.27.1" }
 apache_commons_lang3 = { module = "org.apache.commons:commons-lang3", version = "3.16.0" }
 apache_commons_pool2 = { module = "org.apache.commons:commons-pool2", version = "2.12.0" }
 apache_commons_io = { module = "commons-io:commons-io", version = "2.16.1" }
+slf4j = { module = "org.slf4j:slf4j-api", version = "2.0.16" }
+slf4j_log4j = { module = "org.slf4j:slf4j-log4j12", version = "2.0.16" }
+log4j = { module = "org.apache.logging.log4j:log4j-core", version = "2.23.1" }
 
 
diff --git a/src/main/java/com.voinearadu/logger/Logger.java b/src/main/java/com.voinearadu/logger/Logger.java
@@ -48,6 +48,11 @@ public static void debug(Object object) {
         log(Level.DEBUG, object, ConsoleColor.WHITE, 1);
     }
 
+    @SuppressWarnings("unused")
+    public static void debug(Object object, ConsoleColor color) {
+        log(Level.DEBUG, object, color, 1);
+    }
+
     public static void good(Object object) {
         log(Level.INFO, object, ConsoleColor.GREEN, 1);
     }

diff --git a/src/main/java/com.voinearadu/reflections/Reflections.java b/src/main/java/com.voinearadu/reflections/Reflections.java
@@ -2,6 +2,7 @@
 
 import com.voinearadu.logger.Logger;
 import lombok.Getter;
+import org.apache.commons.io.FileUtils;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
@@ -21,11 +22,23 @@
 public class Reflections {
 
     private final List<File> zipFiles;
+    private final List<File> folders;
     private final ClassLoader classLoader;
     private final List<String> searchDomain;
 
-    public Reflections(List<File> zipFiles, ClassLoader classLoader, String... searchDomain) {
+    public Reflections(@NotNull List<File> zipFiles, @NotNull List<File> folders, @NotNull ClassLoader classLoader, @NotNull String... searchDomain) {
         this.zipFiles = zipFiles;
+        this.folders = folders;
+
+        String[] classPathEntries = System.getProperty("java.class.path").split(File.pathSeparator);
+        for (String classPathEntry : classPathEntries) {
+            if (classPathEntry.endsWith(".jar") || classPathEntry.endsWith(".zip")) {
+                zipFiles.add(new File(classPathEntry));
+            } else {
+                folders.add(new File(classPathEntry));
+            }
+        }
+
         this.classLoader = classLoader;
         this.searchDomain = Arrays.stream(searchDomain).map(domain -> {
             if (!domain.endsWith(".")) {
@@ -35,7 +48,6 @@ public Reflections(List<File> zipFiles, ClassLoader classLoader, String... searc
         }).toList();
     }
 
-    @SuppressWarnings("unused")
     public static @Nullable Field getField(@NotNull Class<?> clazz, String fieldName) {
         Field field = null;
 
@@ -84,20 +96,24 @@ public Reflections(List<File> zipFiles, ClassLoader classLoader, String... searc
 
     @SuppressWarnings("unused")
     public Reflections from(String... searchDomain) {
-        return new Reflections(zipFiles, classLoader, searchDomain);
+        return new Reflections(zipFiles, folders, classLoader, searchDomain);
     }
 
     public @NotNull Set<Class<?>> getClasses() {
         Set<Class<?>> classes = new HashSet<>();
 
         for (File zipFile : zipFiles) {
-            classes.addAll(getClasses(zipFile));
+            classes.addAll(getClassesFromZip(zipFile));
+        }
+
+        for (File zipFile : folders) {
+            classes.addAll(getClassesFromFolder(zipFile));
         }
 
         return classes;
     }
 
-    private @NotNull Set<Class<?>> getClasses(File zipFile) {
+    private @NotNull Set<Class<?>> getClassesFromZip(File zipFile) {
         Set<Class<?>> classes = new HashSet<>();
 
         try (ZipFile zip = new ZipFile(zipFile)) {
@@ -106,39 +122,32 @@ public Reflections from(String... searchDomain) {
             while (entries.hasMoreElements()) {
                 ZipEntry entry = entries.nextElement();
 
-                if (entry == null)
-                    break;
-
-                if (entry.isDirectory()) {
+                if (entry == null || entry.isDirectory()) {
                     continue;
                 }
 
-                String name = entry.getName();
+                Class<?> clazz = processFile(entry.getName());
 
-                if (!name.endsWith(".class")) {
-                    continue;
-                }
-
-                name = name.replace("/", ".");
-                name = name.replace(".class", "");
-
-                if (searchDomain.stream().noneMatch(name::startsWith)) {
-                    continue;
+                if (clazz != null) {
+                    classes.add(clazz);
                 }
+            }
+        } catch (Exception error) {
+            Logger.error(error);
+        }
 
-                String simpleClassName = name.substring(name.lastIndexOf('.') + 1);
+        return classes;
+    }
 
-                // Skip Mixin classes
-                if (simpleClassName.contains("Mixin")) {
-                    continue;
-                }
+    private @NotNull Set<Class<?>> getClassesFromFolder(File folder) {
+        Set<Class<?>> classes = new HashSet<>();
 
+        try {
+            for (File file : FileUtils.listFiles(folder, new String[]{"class"}, true)) {
+                Class<?> clazz = processFile(file.getAbsolutePath().replace(folder.getAbsolutePath() + File.separator, ""));
 
-                try {
-                    classes.add(classLoader.loadClass(name));
-                } catch (Throwable throwable) {
-                    Logger.error("Failed to load class " + name + " from " + zipFile.getName());
-                    Logger.error(throwable);
+                if (clazz != null) {
+                    classes.add(clazz);
                 }
             }
         } catch (Exception error) {
@@ -148,7 +157,36 @@ public Reflections from(String... searchDomain) {
         return classes;
     }
 
-    @SuppressWarnings("unused")
+    private @Nullable Class<?> processFile(String fileName) {
+        if (!fileName.endsWith(".class")) {
+            return null;
+        }
+
+        fileName = fileName.replace("/", ".");
+        fileName = fileName.replace("\\", ".");
+        fileName = fileName.replace(".class", "");
+
+        if (searchDomain.stream().noneMatch(fileName::startsWith)) {
+            return null;
+        }
+
+        String simpleClassName = fileName.substring(fileName.lastIndexOf('.') + 1);
+
+        // Skip Mixin classes
+        if (simpleClassName.contains("Mixin")) {
+            return null;
+        }
+
+        try {
+            return classLoader.loadClass(fileName);
+        } catch (Throwable throwable) {
+            Logger.error("Failed to load class " + fileName + " from " + fileName);
+            Logger.error(throwable);
+        }
+
+        return null;
+    }
+
     public @NotNull Set<Class<?>> getTypesAnnotatedWith(@NotNull Class<? extends Annotation> annotation) {
         Set<Class<?>> classes = new HashSet<>();
 
@@ -161,7 +199,6 @@ public Reflections from(String... searchDomain) {
         return classes;
     }
 
-    @SuppressWarnings("unused")
     public @NotNull Set<Method> getMethodsAnnotatedWith(@NotNull Class<? extends Annotation> annotation) {
         Set<Method> methods = new HashSet<>();
 

diff --git a/src/test/java/com/voinearadu/event_manager/EventManagerTests.java b/src/test/java/com/voinearadu/event_manager/EventManagerTests.java
@@ -3,6 +3,7 @@
 import com.voinearadu.event_manager.dto.TestComplexEvent;
 import com.voinearadu.event_manager.dto.TestEvent;
 import com.voinearadu.event_manager.manager.TestEventListener;
+import org.apache.log4j.BasicConfigurator;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
@@ -17,6 +18,7 @@ public class EventManagerTests {
 
     @BeforeAll
     public static void setup() {
+        BasicConfigurator.configure();
         eventManager.register(new TestEventListener());
     }
 

diff --git a/src/test/java/com/voinearadu/file_manager/FileManagerTests.java b/src/test/java/com/voinearadu/file_manager/FileManagerTests.java
@@ -4,6 +4,7 @@
 import com.google.gson.GsonBuilder;
 import com.voinearadu.file_manager.dto.files.FileObject;
 import com.voinearadu.file_manager.manager.FileManager;
+import org.apache.log4j.BasicConfigurator;
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
@@ -20,6 +21,7 @@ public class FileManagerTests {
 
     @BeforeAll
     public static void init() {
+        BasicConfigurator.configure();
         Gson gson = new GsonBuilder() //NOPMD - suppressed GsonCreatedForEachMethodCall
                 .create();
         fileManager = new FileManager(() -> gson, "tmp");

diff --git a/src/test/java/com/voinearadu/file_manager/GsonTests.java b/src/test/java/com/voinearadu/file_manager/GsonTests.java
@@ -13,6 +13,7 @@
 import com.voinearadu.file_manager.dto.serializable.SerializableMap;
 import com.voinearadu.file_manager.dto.serializable.SerializableObject;
 import lombok.Getter;
+import org.apache.log4j.BasicConfigurator;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
@@ -32,6 +33,7 @@ public class GsonTests {
 
     @BeforeAll
     public static void init() {
+        BasicConfigurator.configure();
         ClassLoader classLoader = GsonTests.class.getClassLoader();
 
         SerializableListGsonTypeAdapter serializableListGsonTypeAdapter = new SerializableListGsonTypeAdapter(classLoader);

diff --git a/src/test/java/com/voinearadu/lambda/LambdaRunnableExecutorTest.java b/src/test/java/com/voinearadu/lambda/LambdaRunnableExecutorTest.java
@@ -2,6 +2,7 @@
 
 import com.voinearadu.lambda.lambda.*;
 import lombok.SneakyThrows;
+import org.apache.log4j.BasicConfigurator;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
@@ -17,8 +18,8 @@ public class LambdaRunnableExecutorTest {
 
     @BeforeAll
     public static void init() {
+        BasicConfigurator.configure();
     }
-
     @Test
     public void testLambdaExecutors() {
         ArgLambdaExecutor<List<String>> addEmpty = (list) -> list.add("empty");

diff --git a/src/test/java/com/voinearadu/logger/LoggerTest.java b/src/test/java/com/voinearadu/logger/LoggerTest.java
@@ -1,6 +1,8 @@
 package com.voinearadu.logger;
 
 import lombok.SneakyThrows;
+import org.apache.log4j.BasicConfigurator;
+import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.MethodOrderer;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.TestMethodOrder;
@@ -14,6 +16,11 @@
 @TestMethodOrder(MethodOrderer.OrderAnnotation.class)
 public class LoggerTest {
 
+    @BeforeAll
+    public static void init() {
+        BasicConfigurator.configure();
+    }
+
     @SneakyThrows
     @Test
     public void testDebugLogger() {

diff --git a/src/test/java/com/voinearadu/message_builder/MessageBuilderTests.java b/src/test/java/com/voinearadu/message_builder/MessageBuilderTests.java
@@ -1,6 +1,8 @@
 package com.voinearadu.message_builder;
 
 
+import org.apache.log4j.BasicConfigurator;
+import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
 import java.util.Arrays;
@@ -11,6 +13,11 @@
 
 public class MessageBuilderTests {
 
+    @BeforeAll
+    public static void init() {
+        BasicConfigurator.configure();
+    }
+
     @Test
     public void testMessageBuilder() {
         MessageBuilder builder1 = new MessageBuilder("This is a %placeholder%");

diff --git a/src/test/java/com/voinearadu/redis_manager/RedisTest.java b/src/test/java/com/voinearadu/redis_manager/RedisTest.java
@@ -17,6 +17,7 @@
 import com.voinearadu.redis_manager.manager.RedisManager;
 import com.voinearadu.redis_manager.manager.TestListener;
 import lombok.Getter;
+import org.apache.log4j.BasicConfigurator;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
@@ -32,6 +33,7 @@ public class RedisTest {
 
     @BeforeAll
     public static void init() {
+        BasicConfigurator.configure();
         MessageBuilderManager.init(false);
 
         ClassLoader classLoader = RedisTest.class.getClassLoader();