Test

Author: LexManos

.github/workflows/test.yml

@@ -0,0 +1,43 @@
+name: Test
+
+on:
+  pull_request:
+  push:
+    branches: [ 'test' ]
+    paths-ignore:
+      - 'README.md'
+
+permissions:
+  contents: read
+
+jobs:
+  test:
+    continue-on-error: true
+    strategy:
+      matrix:
+        command: [
+            "--maven --forge --version 1.21.11-61.1.0",
+            "--maven --forge --version 1.20.1-47.4.0",
+            "--maven --forge --version 1.12.2-14.23.5.2859 --mappings snapshot:20171003-1.12",
+            "--maven --client --version 1.20.1",
+            "--maven --server --version 1.20.1",
+            "--maven --mc --version 1.20.1",
+            "--maven --client --version 26.1-snapshot-1",
+            "--maven --server --version 26.1-snapshot-1"
+        ]
+        
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout sources
+        uses: actions/checkout@v6
+        with:
+          fetch-tags: true
+      - name: Setup Java
+        uses: actions/setup-java@v4
+        with:
+          distribution: temurin
+          java-version: 25
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v5
+      - name: Test Run
+        run: ./gradlew run --args="--debug ${{ matrix.command }}"

build.gradle

@@ -2,7 +2,7 @@ import org.gradle.api.plugins.jvm.JvmTestSuite
 import com.github.jengelman.gradle.plugins.shadow.tasks.ShadowJar
 
 plugins {
-    id 'java'
+    id 'application'
     id 'jvm-test-suite'
     id 'idea'
     id 'eclipse'
@@ -52,6 +52,10 @@ license {
     exclude '**/ComparableVersion.java' // Apache class used as source-level dep as to not need the hundred+ other classes the library pulls in
 }
 
+application {
+    mainClass = 'net.minecraftforge.mcmaven.cli.Main'
+}
+
 tasks.named('jar', Jar) {
     manifest {
         attributes([

settings.gradle

@@ -40,7 +40,7 @@ dependencyResolutionManagement.versionCatalogs.register('libs') {
     // Utilities
     library 'utils-download', 'net.minecraftforge', 'download-utils'  version '0.4.0'
     library 'utils-files',    'net.minecraftforge', 'file-utils'      version '0.3.3'
-    library 'utils-hash',     'net.minecraftforge', 'hash-utils'      version '0.2.1'
+    library 'utils-hash',     'net.minecraftforge', 'hash-utils'      version '0.2.2'
     library 'utils-data',     'net.minecraftforge', 'json-data-utils' version '0.4.3'
     library 'utils-logging',  'net.minecraftforge', 'log-utils'       version '0.5.1'
     library 'utils-os',       'net.minecraftforge', 'os-utils'        version '0.1.0'

src/main/java/net/minecraftforge/mcmaven/cli/MavenTask.java

@@ -115,11 +115,14 @@ static OptionParser run(String[] args, boolean getParser) throws Exception {
             "File to write extended output data to. Not compatible with bulk operations.")
             .withRequiredArg().ofType(File.class);
 
+        var debugO = parser.accepts("debug", "Sets the log level to DEBUG");
+
         var shorthandOptions = new HashMap<String, OptionSpecBuilder>();
         var artifacts = Map.of(
             "forge",  Constants.FORGE_ARTIFACT,
             "fml",    Constants.FMLONLY_ARTIFACT,
             "mc",     "net.minecraft:joined",
+            "joined", "net.minecraft:joined",
             "client", "net.minecraft:client",
             "server", "net.minecraft:server",
             "mapping-data", "net.minecraft:mappings"
@@ -153,6 +156,8 @@ static OptionParser run(String[] args, boolean getParser) throws Exception {
         }
 
         // global options
+        if (options.has(debugO))
+            LOGGER.setEnabled(Logger.Level.DEBUG);
         if (options.has(offlineO))
             Mavenizer.setOffline();
         if (options.has(cacheOnlyO))

src/main/java/net/minecraftforge/mcmaven/impl/Mavenizer.java

@@ -57,15 +57,40 @@ public static void assertNotCacheOnly() {
         if (cacheOnly) {
             throw new IllegalArgumentException("Cache is out of date! Please run without --cache-only");
         } else if (!cacheMiss) {
-            LOGGER.debug("Cache miss!", new Exception("Cache miss! Stacktrace for Information Only"));
+            LOGGER.info("Cache miss!", new Exception("Cache miss! Stacktrace for Information Only"));
             cacheMiss = true;
             LOGGER.release();
         }
     }
 
+    private static void debug(String header, String data) {
+        if (data.isEmpty())
+            LOGGER.debug(header);
+        else {
+            var lines = data.split("\n");
+            if (lines.length == 1)
+                LOGGER.debug(header + ' ' + lines[0]);
+            else {
+                LOGGER.debug(header);
+                LOGGER.push();
+                for (var line : lines)
+                    LOGGER.debug(line);
+                LOGGER.pop();
+            }
+        }
+    }
+
     public static boolean checkCache(File output, HashStore cache) {
         if (!ignoreCache && output.exists() && cache.isSame())
             return true;
+        if (LOGGER.isEnabled(Logger.Level.DEBUG)) {
+            LOGGER.debug("Cache miss: " + output.getAbsolutePath());
+            LOGGER.push();
+                LOGGER.debug("Exists: " + output.exists());
+                debug("Old:", cache.dumpOld());
+                debug("New:", cache.dump());
+            LOGGER.pop();
+        }
         Mavenizer.assertNotCacheOnly();
         return false;
     }