Adds Mojang-mapped 1.20.5 test profile. Fix chat colour argument test for 1.20.5. Attempt to get item predicate tests working on 1.20.5

Author: JorelAli

commandapi-platforms/commandapi-bukkit/commandapi-bukkit-test/commandapi-bukkit-test-tests/pom.xml

@@ -205,6 +205,53 @@
 				</dependency> -->
 			</dependencies>
 		</profile>
+		<profile>
+			<id>Minecraft_1_20_5_Mojang</id>
+			<dependencies>
+				<dependency>
+					<!-- Needed for 1.19.4 Brigadier logging. This is also present in
+					 commandapi-core, but isn't inherited here automatically, so we add
+					 it here -->
+					<groupId>org.apache.logging.log4j</groupId>
+					<artifactId>log4j-api</artifactId>
+					<version>2.23.1</version>
+					<scope>provided</scope>
+				</dependency>
+				<dependency>
+					<groupId>dev.jorel</groupId>
+					<artifactId>commandapi-bukkit-nms-dependency-mojang-mapped</artifactId>
+					<version>${project.version}</version>
+					<type>pom</type>
+				</dependency>
+				<dependency>
+					<groupId>dev.jorel</groupId>
+					<artifactId>commandapi-bukkit-shade</artifactId>
+					<version>${project.version}</version>
+				</dependency>
+				<dependency>
+					<groupId>dev.jorel</groupId>
+					<artifactId>commandapi-bukkit-test-impl-1.20.5</artifactId>
+					<version>${project.version}</version>
+					<classifier>mojang-mapped</classifier>
+					<scope>test</scope>
+				</dependency>
+				<dependency>
+					<groupId>org.spigotmc</groupId>
+					<artifactId>spigot</artifactId>
+					<classifier>remapped-mojang</classifier>
+					<version>1.20.5-R0.1-SNAPSHOT</version>
+				</dependency>
+				<!-- Ideally we'd include paper here, but this
+				causes MockBukkit to fall apart because it then
+				tries to register potion effects when the server
+				is null and I CBA to sort that out right now -->
+				<!-- <dependency>
+					<groupId>io.papermc.paper</groupId>
+					<artifactId>paper-api</artifactId>
+					<version>1.20.2-R0.1-SNAPSHOT</version>
+				</dependency> -->
+			</dependencies>
+		</profile>
 		<profile>
 			<id>Minecraft_1_20_3</id>
 			<dependencies>

commandapi-platforms/commandapi-bukkit/commandapi-bukkit-test/commandapi-bukkit-test-tests/src/test/java/dev/jorel/commandapi/test/arguments/ArgumentChatColorTests.java

@@ -78,9 +78,9 @@ void executionTestWithChatColorArgument() {
 			if(color.isFormat() && !color.equals(ChatColor.RESET)) {
 				String colorName = color.name().toLowerCase();
 				if (version.greaterThanOrEqualTo(MCVersion.V1_20_5)) {
-					assertCommandFailsWith(player, "test " + colorName, "Unknown color '" + colorName + "'");
-				} else {
 					assertCommandFailsWith(player, "test " + colorName, "Unknown color '" + colorName + "' at position 10: test magic<--[HERE]");
+				} else {
+					assertCommandFailsWith(player, "test " + colorName, "Unknown color '" + colorName + "'");
 				}
 			}
 		}

commandapi-platforms/commandapi-bukkit/commandapi-bukkit-test/commandapi-bukkit-test-tests/src/test/java/dev/jorel/commandapi/test/arguments/ArgumentItemStackPredicateTests.java

@@ -14,6 +14,7 @@
 
 import be.seeseemelk.mockbukkit.entity.PlayerMock;
 import dev.jorel.commandapi.CommandAPICommand;
+import dev.jorel.commandapi.MCVersion;
 import dev.jorel.commandapi.arguments.ItemStackPredicateArgument;
 import dev.jorel.commandapi.test.MockPlatform;
 import dev.jorel.commandapi.test.Mut;
@@ -55,10 +56,18 @@ void executionTestWithItemStackPredicateArgument() {
 
 		PlayerMock player = server.addPlayer();
 
-		// /test dirt
-		server.dispatchCommand(player, "test dirt");
-		assertTrue(results.get().test(new ItemStack(Material.DIRT)));
-		
+		// I don't know why, but for some reason, in 1.20.5 you can't have
+		// an item predicate that doesn't contain a # character?
+		if (version.lessThan(MCVersion.V1_20_5)) {
+			// /test dirt
+			server.dispatchCommand(player, "test dirt");
+			assertTrue(results.get().test(new ItemStack(Material.DIRT)));
+		} else {
+			// /test #axes
+			server.dispatchCommand(player, "test #axes");
+			assertTrue(results.get().test(new ItemStack(Material.DIAMOND_AXE)));
+		}
+
 		// TODO: Not sure why this fails, but it does!
 		// /test dirt
 //		server.dispatchCommand(player, "test dirt{Count:3b}");