work on all-args constructors, fix no-arg constructors

Author: dmulloy2

PacketWrappers.md

@@ -265,7 +265,7 @@ Cover these scenarios for each wrapper:
 * testNoArgsCreate
     * Construct a fresh wrapper using the no-args constructor
     * Get the NMS handle packet via `getHandle()`
-    * Assert that all fields in the NMS handle are default/empty values (e.g. 0, null, empty list)
+    * Assert that all fields in the NMS handle are not null
 * testModifyExistingPacket
   * Create the NMS handle packet directly from its constructor, passing known test values
   * Create a packet container from the NMS handle via `PacketContainer.fromPacket()`
@@ -278,6 +278,12 @@ Cover these scenarios for each wrapper:
   * Attempt to construct the wrapper using this incorrect packet container
   * Assert that an appropriate exception is thrown (e.g. IllegalArgumentException)
 
+### Appropriate test values
+
+* For numeric fields, use a mix of positive, negative, and zero values (e.g. -5, 0, 3)
+* For enum fields, prefer the 2nd or 3rd constant rather than the first, to ensure the wrapper correctly handles non-default enum values
+  * Additionally, if the enum constant is @Deprecated, do not use it
+
 Here's a good example of a complete test class for `WrappedClientboundSetExperiencePacket`:
 
 ```java

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundAddEntityPacket.java

@@ -31,8 +31,7 @@ public class WrappedClientboundAddEntityPacket extends AbstractPacket {
 
     public WrappedClientboundAddEntityPacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundAddEntityPacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundAnimatePacket.java

@@ -28,8 +28,7 @@ public class WrappedClientboundAnimatePacket extends AbstractPacket {
 
     public WrappedClientboundAnimatePacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundAnimatePacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundBlockChangedAckPacket.java

@@ -18,8 +18,7 @@ public class WrappedClientboundBlockChangedAckPacket extends AbstractPacket {
 
     public WrappedClientboundBlockChangedAckPacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundBlockChangedAckPacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundBlockDestructionPacket.java

@@ -23,8 +23,7 @@ public class WrappedClientboundBlockDestructionPacket extends AbstractPacket {
 
     public WrappedClientboundBlockDestructionPacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundBlockDestructionPacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundBlockEventPacket.java

@@ -23,8 +23,7 @@ public class WrappedClientboundBlockEventPacket extends AbstractPacket {
 
     public WrappedClientboundBlockEventPacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundBlockEventPacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundBlockUpdatePacket.java

@@ -21,8 +21,7 @@ public class WrappedClientboundBlockUpdatePacket extends AbstractPacket {
 
     public WrappedClientboundBlockUpdatePacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundBlockUpdatePacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundChangeDifficultyPacket.java

@@ -20,8 +20,7 @@ public class WrappedClientboundChangeDifficultyPacket extends AbstractPacket {
 
     public WrappedClientboundChangeDifficultyPacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundChangeDifficultyPacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundChunkBatchFinishedPacket.java

@@ -18,8 +18,7 @@ public class WrappedClientboundChunkBatchFinishedPacket extends AbstractPacket {
 
     public WrappedClientboundChunkBatchFinishedPacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundChunkBatchFinishedPacket(PacketContainer packet) {
         super(packet, TYPE);

src/main/java/net/dmulloy2/protocol/wrappers/game/clientbound/WrappedClientboundChunkBatchStartPacket.java

@@ -15,8 +15,7 @@ public class WrappedClientboundChunkBatchStartPacket extends AbstractPacket {
 
     public WrappedClientboundChunkBatchStartPacket() {
         super(new PacketContainer(TYPE), TYPE);
-        handle.getModifier().writeDefaults();
-    }
+            }
 
     public WrappedClientboundChunkBatchStartPacket(PacketContainer packet) {
         super(packet, TYPE);