fix(mod): better logging for 1.7.10, default to github caching

.github/workflows/lifecycle.yml

@@ -256,6 +256,7 @@ jobs:
         timeout-minutes: 3
         uses: ./
         with:
+          cache-mc: "true"
           mc: ${{ matrix.version.mc }}
           mc-runtime-test: none
           modloader: ${{ matrix.version.modloader }}

.github/workflows/run-gametests.yml

@@ -60,6 +60,7 @@ jobs: # TODO: add cleanup job
         timeout-minutes: 3
         uses: ./
         with:
+          cache-mc: "true"
           mc: ${{ matrix.mc }}
           modloader: ${{ matrix.modloader }}
           regex: ${{ matrix.regex }}

.github/workflows/test-local-action.yml

@@ -69,4 +69,3 @@ jobs:
           fabric-api: ${{ github.event.inputs.fabric-api }}
           fabric-gametest-api: ${{ github.event.inputs.fabric-gametest-api }}
           hmc-version: ${{ github.event.inputs.hmc-version }}
-          cache-mc: false

1_7_10/src/main/java/me/earth/mc_runtime_test/mixin/MixinNetworkManager.java

@@ -0,0 +1,37 @@
+package me.earth.mc_runtime_test.mixin;
+
+import io.netty.channel.ChannelHandlerContext;
+import io.netty.util.concurrent.GenericFutureListener;
+import net.minecraft.network.NetworkManager;
+import net.minecraft.network.Packet;
+import org.apache.logging.log4j.Logger;
+import org.spongepowered.asm.mixin.Final;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Shadow;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
+
+@Mixin(NetworkManager.class)
+@SuppressWarnings("StringConcatenationArgumentToLogCall")
+public class MixinNetworkManager {
+    @Shadow
+    @Final
+    private static Logger logger;
+
+    @Inject(method = "scheduleOutboundPacket", at = @At("HEAD"))
+    protected void scheduleOutboundPacketHook(Packet packet, GenericFutureListener<?>[] futureListeners, CallbackInfo ci) {
+        logger.info("Sending packet: " + packet.getClass().getName());
+    }
+
+    @Inject(method = "channelRead0", at = @At("HEAD"))
+    protected void channelRead0Hook(ChannelHandlerContext context, Packet packet, CallbackInfo ci) {
+        logger.info("Packet received: " + packet.getClass().getName());
+    }
+
+    @Inject(method = "exceptionCaught", at = @At("HEAD"))
+    private void exceptionCaughtHook(ChannelHandlerContext context, Throwable throwable, CallbackInfo ci) {
+        logger.error("Exception caught", throwable);
+    }
+
+}

1_7_10/src/main/resources/mc_runtime_test.mixins.json

@@ -4,7 +4,8 @@
   "package": "me.earth.mc_runtime_test.mixin",
   "compatibilityLevel": "JAVA_8",
   "client": [
-    "MixinMinecraft"
+    "MixinMinecraft",
+    "MixinNetworkManager"
   ],
   "injectors": {
     "defaultRequire": 1

README.md

@@ -117,6 +117,9 @@ The following table summarizes the available inputs for customization:
 
 ---
 
+## Caching
+Mc-Runtime-Test optionally caches `.minecraft` to improve execution time, for this we use `blacksmith/cache` simply follow the instructions [here](https://docs.blacksmith.sh/introduction/quickstart) to enable blacksmith for your repositories and enable the `cache-mc` input.
+
 ## Running Your Own Tests
 MC-Runtime-Test supports Minecraft’s [Game-Test Framework](https://www.minecraft.net/en-us/creator/article/get-started-gametest-framework). It executes `/test runall` upon joining a world.
 

action.yml

@@ -44,7 +44,7 @@ inputs:
     default: "2.5.1"
   cache-mc:
     description: Cache .minecraft
-    default: "true"
+    default: "false"
 
 runs:
   using: composite