Fix Forge datagen and add tests for them

Author: YaLTeR

.github/workflows/gradle.yml

@@ -34,6 +34,9 @@ jobs:
           path: forge/build/libs/
           if-no-files-found: error
 
+      - name: Test datagen
+        run: ./gradlew -p forge runData
+
   build-neoforge:
     runs-on: ubuntu-latest
     steps:
@@ -56,6 +59,9 @@ jobs:
           path: neoforge/build/libs/
           if-no-files-found: error
 
+      - name: Test datagen
+        run: ./gradlew -p neoforge runClientData
+
   build-fabric:
     runs-on: ubuntu-latest
     steps:

forge/build.gradle

@@ -40,6 +40,14 @@ minecraft {
 
         register('client')
         register('server')
+
+        // For testing datagen.
+        register('data') {
+            args '--mod', 'mousetweaks',
+                    '--all',
+                    '--output', layout.buildDirectory.dir('generated/resources').get().asFile.absolutePath,
+                    '--existing', file('../src/main/resources').absolutePath
+        }
     }
 }
 

neoforge/build.gradle

@@ -57,6 +57,14 @@ runs {
         systemProperty 'forge.enabledGameTestNamespaces', "mousetweaks"
         argument '--nogui'
     }
+
+    // For testing datagen.
+    clientData {
+        arguments.addAll '--mod', 'mousetweaks',
+                '--all',
+                '--output', layout.buildDirectory.dir('generated/resources').get().asFile.absolutePath,
+                '--existing', file('../src/main/resources').absolutePath
+    }
 }
 
 dependencies {

src/main/java/yalter/mousetweaks/forge/MouseTweaksForge.java

@@ -1,26 +1,44 @@
 package yalter.mousetweaks.forge;
 
+import net.minecraftforge.api.distmarker.Dist;
 import net.minecraftforge.client.event.ScreenEvent.MouseButtonPressed;
 import net.minecraftforge.client.event.ScreenEvent.MouseButtonReleased;
 import net.minecraftforge.client.event.ScreenEvent.MouseDragged;
 import net.minecraftforge.client.event.ScreenEvent.MouseScrolled;
 import net.minecraftforge.common.MinecraftForge;
 import net.minecraftforge.eventbus.api.bus.BusGroup;
+import net.minecraftforge.eventbus.api.bus.EventBus;
 import net.minecraftforge.eventbus.api.listener.SubscribeEvent;
 import net.minecraftforge.fml.common.Mod;
+import net.minecraftforge.fml.event.lifecycle.FMLClientSetupEvent;
+import net.minecraftforge.fml.javafmlmod.FMLJavaModLoadingContext;
+import net.minecraftforge.fml.loading.FMLEnvironment;
 import yalter.mousetweaks.*;
 
 import java.lang.invoke.MethodHandles;
 
 @Mod(Constants.MOD_ID)
 public class MouseTweaksForge {
-    public MouseTweaksForge() {
-        Main.initialize();
+    public MouseTweaksForge(FMLJavaModLoadingContext context) {
+        if (FMLEnvironment.dist != Dist.CLIENT) {
+            Logger.Log("Disabled because not running on the client.");
+            return;
+        }
+
         BusGroup.DEFAULT.register(MethodHandles.lookup(), this);
 
         MinecraftForge.registerConfigScreen(ConfigScreen::new);
     }
 
+    @Mod.EventBusSubscriber(modid = Constants.MOD_ID, value = Dist.CLIENT)
+    public static class ClientModEvents {
+        @SubscribeEvent
+        public static void onClientSetup(FMLClientSetupEvent event) {
+            // Minecraft.getInstance() is now non-null in this event handler.
+            Main.initialize();
+        }
+    }
+
     @SubscribeEvent
     public boolean onGuiMouseClickedPre(MouseButtonPressed.Pre event) {
         Logger.DebugLog("onGuiMouseClickedPre button = " + event.getButton());