refactor: restructure the flat package into seperate subpackages and classes

Author: ahmadnull

justfile

@@ -28,8 +28,9 @@ test-show-only-failed: (test "test" "-PtestLogEvents=failed")
 
 [group('chore')]
 clean-whitespace:
-    sed 's/[ \t]*$//' lib/src/main/java/io/github/ahmadnull/dataserialization/* -i
-    sed 's/[ \t]*$//' lib/src/test/java/io/github/ahmadnull/dataserialization/* -i
+    # Recursive, the package directories contain sub packages since the library
+    # was split per format. `just format` already covers this via spotless.
+    find lib/src -name '*.java' -exec sed -i 's/[ \t]*$//' {} +
 
 [group('chore')]
 format:

lib/build.gradle.kts

@@ -24,9 +24,13 @@ dependencies {
 sourceSets {
     // The shared conformance corpora live in the repository root, outside of any
     // module, and are copied onto the test classpath so the tests can list and
-    // read them as resources.
+    // read them as resources. Every format registers its own directory here, the
+    // suites must not be nested into one another to keep the corpus paths flat.
     named("test") {
-        resources.srcDir(rootProject.file("test_suites/json"))
+        resources.srcDirs(
+            rootProject.file("test_suites/json"),
+            // rootProject.file("test_suites/toml"),
+        )
     }
 }
 
@@ -50,6 +54,11 @@ tasks.withType<Test> {
     // Use JUnit Platform for unit tests.
     useJUnitPlatform()
 
+    // The module descriptor is asserted from ModuleApiSurfaceTest, which reads
+    // module-info.class off the test classpath. The tests deliberately stay on
+    // the classpath: inferModulePath only takes effect when the main runtime is
+    // a jar, while the test task consumes the class directory.
+
     // ./gradlew test -PtestLogEvents=failed logs nothing but the failures,
     // any other value keeps the default event set.
     val testLogEvents = when (providers.gradleProperty("testLogEvents").orNull) {

lib/src/main/java/io/github/ahmadnull/dataserialization/DataPath.java

@@ -6,7 +6,7 @@
 
 public class DataPath {
 
-    public enum Seperator {
+    public enum Separator {
         Dot {
             public String toString() {
                 return squareBracketsRegex + Pattern.quote(".") + ']';
@@ -24,22 +24,22 @@ public String toString() {
     }
 
     public static Object getByPath(Map<String, Object> map, String path) {
-        return getByPath(map, DataPath.Seperator.Dot, path);
+        return getByPath(map, DataPath.Separator.Dot, path);
     }
 
-    public static Object getByPath(Map<String, Object> map, DataPath.Seperator seperator, String path) {
-        String seperatorRegex = seperator.toString();
+    public static Object getByPath(Map<String, Object> map, DataPath.Separator separator, String path) {
+        String separatorRegex = separator.toString();
 
-        String[] splittedPath = path.split(seperatorRegex);
+        String[] splittedPath = path.split(separatorRegex);
 
-        return getByPath(map, seperator, splittedPath);
+        return getByPath(map, separator, splittedPath);
     }
 
     public static Object getByPath(Map<String, Object> map, String... path) {
-        return getByPath(map, DataPath.Seperator.Dot, path);
+        return getByPath(map, DataPath.Separator.Dot, path);
     }
 
-    public static Object getByPath(Map<String, Object> map, DataPath.Seperator seperator, String... path) {
+    public static Object getByPath(Map<String, Object> map, DataPath.Separator separator, String... path) {
         Object node = map.get(path[0]);
 
         int length = path.length;
@@ -56,22 +56,22 @@ public static Object getByPath(Map<String, Object> map, DataPath.Seperator seper
     }
 
     public static Object getByPath(List<Object> list, String path) {
-        return getByPath(list, DataPath.Seperator.Dot, path);
+        return getByPath(list, DataPath.Separator.Dot, path);
     }
 
-    public static Object getByPath(List<Object> list, DataPath.Seperator seperator, String path) {
-        String seperatorRegex = seperator.toString();
+    public static Object getByPath(List<Object> list, DataPath.Separator separator, String path) {
+        String separatorRegex = separator.toString();
 
-        String[] splittedPath = path.split(seperatorRegex);
+        String[] splittedPath = path.split(separatorRegex);
 
-        return getByPath(list, seperator, splittedPath);
+        return getByPath(list, separator, splittedPath);
     }
 
     public static Object getByPath(List<Object> list, String... path) {
-        return getByPath(list, DataPath.Seperator.Dot, path);
+        return getByPath(list, DataPath.Separator.Dot, path);
     }
 
-    public static Object getByPath(List<Object> list, DataPath.Seperator seperator, String... path) {
+    public static Object getByPath(List<Object> list, DataPath.Separator separator, String... path) {
         Object node = list.get(Integer.parseInt(path[0]));
 
         int length = path.length;