Setup module info

Author: the-codinator

pom.xml

@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+  xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
   <modelVersion>4.0.0</modelVersion>
 
   <name>LeetCode Tester</name>
@@ -57,6 +58,13 @@
           <source>${java.version}</source>
           <target>${java.version}</target>
           <compilerArgument>-Xlint</compilerArgument>
+          <annotationProcessorPaths>
+            <path>
+              <groupId>org.projectlombok</groupId>
+              <artifactId>lombok</artifactId>
+              <version>${lombok.version}</version>
+            </path>
+          </annotationProcessorPaths>
         </configuration>
       </plugin>
 
@@ -71,6 +79,15 @@
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
         <version>${plugin.surefire.version}</version>
+        <configuration>
+          <!-- Surefire JUnite Hack for Test execution in Java Modules environment -->
+          <!-- https://github.com/junit-team/junit5/issues/2147 -->
+          <argLine>
+            --add-exports org.codi.LeetCodeTester/org.codi.lct.example=org.junit.platform.commons
+            --add-exports org.junit.platform.commons/org.junit.platform.commons.util=ALL-UNNAMED
+            --add-exports org.junit.platform.commons/org.junit.platform.commons.logging=ALL-UNNAMED
+          </argLine>
+        </configuration>
       </plugin>
 
       <!-- Tools / dependency version management -->
@@ -174,6 +191,7 @@
         <configuration>
           <show>protected</show>
           <nohelp>true</nohelp>
+          <quiet>true</quiet>
           <doclint>-missing</doclint>
         </configuration>
       </plugin>
@@ -252,12 +270,6 @@
     </dependency>
 
     <!-- Testing -->
-    <dependency>
-      <groupId>org.junit.jupiter</groupId>
-      <artifactId>junit-jupiter-engine</artifactId>
-      <version>${junit.version}</version>
-    </dependency>
-
     <dependency>
       <groupId>org.junit.jupiter</groupId>
       <artifactId>junit-jupiter-api</artifactId>
@@ -266,7 +278,7 @@
 
     <dependency>
       <groupId>org.junit.jupiter</groupId>
-      <artifactId>junit-jupiter-params</artifactId>
+      <artifactId>junit-jupiter-engine</artifactId>
       <version>${junit.version}</version>
     </dependency>
   </dependencies>

src/main/java/module-info.java

@@ -0,0 +1,35 @@
+module org.codi.LeetCodeTester {
+    /* Exported from module for usage */
+    // Annotations
+    exports org.codi.lct.annotation;
+    exports org.codi.lct.annotation.settings;
+    // Core Testing
+    exports org.codi.lct.core;
+    exports org.codi.lct.core.tester;
+    // Utilities
+    exports org.codi.lct.ds;
+    exports org.codi.lct.ds.extra;
+
+    /* Module Dependencies */
+    // Lombok (Compile Only)
+    requires static lombok;
+    // Jackson
+    requires com.fasterxml.jackson.core;
+    requires transitive com.fasterxml.jackson.annotation;
+    requires transitive com.fasterxml.jackson.databind;
+    // JUnit
+    requires transitive org.junit.jupiter.api;
+    // Apache Commons
+    requires org.apache.commons.lang3;
+    // Logging
+    requires org.slf4j;
+    // Javadoc
+    requires static org.apiguardian.api;
+
+    /* Expose for reflection & access */
+    exports org.codi.lct.impl.adapter.eq to org.apache.commons.lang3;
+    exports org.codi.lct.impl.testcase to org.junit.platform.commons;
+    opens org.codi.lct.core.tester to org.junit.platform.commons;
+    opens org.codi.lct.impl.adapter.ser to com.fasterxml.jackson.databind;
+    opens org.codi.lct.impl.adapter.deser to com.fasterxml.jackson.databind;
+}

src/main/java/org/codi/lct/core/tester/LCTestCase.java

@@ -9,7 +9,7 @@
 import org.codi.lct.impl.helper.JacksonHelper;
 
 /**
- * Defines a test case based on expected output & provided inputs.
+ * Defines a test case based on expected output and provided inputs.
  * Inputs can be in any format, and the executor will attempt to convert it to the right format prior to execution.
  */
 @Value

src/main/java/org/codi/lct/ds/extra/Pair.java

@@ -1,12 +1,10 @@
 package org.codi.lct.ds.extra;
 
 import lombok.AllArgsConstructor;
-import lombok.EqualsAndHashCode;
-import lombok.NoArgsConstructor;
+import lombok.Data;
 
-@NoArgsConstructor
+@Data
 @AllArgsConstructor
-@EqualsAndHashCode
 @SuppressWarnings("checkstyle:MemberName")
 public class Pair<L, R> {
 

src/main/java/org/codi/lct/impl/helper/ConfigHelper.java

@@ -27,7 +27,7 @@ public class ConfigHelper {
 
     static {
         // Load default properties
-        Properties props = FileHelper.loadProperties(null, "lc-tester.properties");
+        Properties props = FileHelper.loadProperties(null, "lct/lc-tester.properties");
         BASE_CONFIG = LCConfig.builder()
             .trackExecutionTime(getBooleanProperty(props, LCConfig.Fields.trackExecutionTime, false))
             .allowMissingExpectedValues(getBooleanProperty(props, LCConfig.Fields.allowMissingExpectedValues, false))

src/main/java/org/codi/lct/impl/helper/JacksonHelper.java

@@ -59,8 +59,9 @@ public List<Object> readAllValues(MappingIterator<Object> iterator) {
             iterator.forEachRemaining(values::add);
             return values;
         } catch (Exception e) {
-            throw new LCException(
-                "Error reading values from input content: " + iterator.getCurrentLocation().getSourceRef(), e);
+            throw new LCException("Error reading values from input content: " + iterator.getCurrentLocation()
+                .contentReference()
+                .getRawContent(), e);
         }
     }
 

src/test/java/org/codi/lct/example/Example01ASimpleFirstExample.java renamed to src/test/java/org/codi/lct/example/Example01_ASimpleFirstExample_Test.java

@@ -10,7 +10,7 @@
  * Test cases are loaded automatically from the TestCase data file at {@code
  * resources/lct/org/codi/lct/example/Example01ASimpleFirstExample}
  */
-public class Example01ASimpleFirstExample extends LCTester {
+public class Example01_ASimpleFirstExample_Test extends LCTester {
 
     @LCSolution
     public int sum(int[] nums) {

src/test/java/org/codi/lct/example/Example02CustomTestCase.java renamed to src/test/java/org/codi/lct/example/Example02_CustomTestCase_Test.java

@@ -12,7 +12,7 @@
  *
  * Note: each test case is run on a separate test instance
  */
-public class Example02CustomTestCase extends LCTester {
+public class Example02_CustomTestCase_Test extends LCTester {
 
     /**
      * {@link LCTestCaseGenerator} annotation is not required since the signature matches {@code public static

src/test/java/org/codi/lct/example/Example03ManualTestCase.java renamed to src/test/java/org/codi/lct/example/Example03_ManualTestCase_Test.java

@@ -21,7 +21,7 @@
  * To avoid sharing the test instance, create multiple @Test methods instead
  */
 @ExtendWith(LCExtension.class)
-public class Example03ManualTestCase {
+public class Example03_ManualTestCase_Test {
 
     @Test
     @DisplayName("My manual test runner")

src/test/java/org/codi/lct/example/Example04ComplexDataStructures.java renamed to src/test/java/org/codi/lct/example/Example04_ComplexDataStructures_Test.java

@@ -7,7 +7,7 @@
 /**
  * Checkout {@link org.codi.lct.ds} for supported data structures
  */
-public class Example04ComplexDataStructures extends LCTester {
+public class Example04_ComplexDataStructures_Test extends LCTester {
 
     public ListNode inorder(TreeNode root) {
         ListNode preStart = new ListNode(0, null);