Fix UpdateRequirementsPacketTests being skipped because MockPlayer#up…

…dateCommands threw UnsupportedOperationException

commandapi-platforms/commandapi-bukkit/commandapi-bukkit-test/commandapi-bukkit-test-tests/src/test/java/dev/jorel/commandapi/test/network/UpdateRequirementsPacketTests.java

@@ -6,6 +6,7 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
+import org.mockito.Mockito;
 
 class UpdateRequirementsPacketTests extends NetworkTestBase {
 
@@ -29,7 +30,12 @@ public void tearDown() {
 
 	@Test
 	void sendReceiveTestWithUpdateRequirementsPacket() {
-		PlayerMock player = server.addPlayer(); // Protocol version currently 0
+		PlayerMock player = Mockito.spy(new PlayerMock(server, "player"));
+		// Interrupt normal calls to updateCommands, because MockPlayer throws an UnimplementedOperationException
+		Mockito.doNothing().when(player).updateCommands();
+		server.addPlayer(player);
+
+		// Protocol version currently 0
 
 		// Error when sending packet to target that has not declared they understand CommandAPI plugin messages
 		assertThrowsWithMessage(
@@ -47,5 +53,8 @@ void sendReceiveTestWithUpdateRequirementsPacket() {
 			new UpdateRequirementsPacket(),
 			new byte[]{0}
 		);
+
+		// Update commands should have been called for this player
+		Mockito.verify(player, Mockito.times(1)).updateCommands();
 	}
 }