Add some more tests for WrappedGameProfile

Author: dmulloy2

ProtocolLib/src/main/java/com/comphenix/protocol/reflect/FuzzyReflection.java

@@ -604,7 +604,6 @@ public Set<Constructor<?>> getConstructors() {
 	// Prevent duplicate fields
 
 	// @SafeVarargs
-	@SuppressWarnings("unchecked")
 	private static <T> Set<T> setUnion(T[]... array) {
 		Set<T> result = new LinkedHashSet<T>();
 

ProtocolLib/src/main/java/com/comphenix/protocol/utility/Util.java

@@ -67,7 +67,6 @@ public static List<Player> getOnlinePlayers() {
 	 * @return The list
 	 */
 	// @SafeVarargs
-	@SuppressWarnings("unchecked")
 	public static <E> List<E> asList(E... elements) {
 		List<E> list = new ArrayList<E>(elements.length);
 		for (E element : elements) {

ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/WrappedGameProfile.java

@@ -250,6 +250,7 @@ public String getName() {
 	 * 
 	 * @return Property map.
 	 */
+	// In the protocol hack and 1.8 it is a ForwardingMultimap
 	@SuppressWarnings({ "unchecked", "rawtypes" })
 	public Multimap<String, WrappedSignedProperty> getProperties() {
 		Multimap<String, WrappedSignedProperty> result = propertyMap;

ProtocolLib/src/main/java/com/comphenix/protocol/wrappers/nbt/NbtFactory.java

@@ -450,7 +450,6 @@ public static NbtCompound ofCompound(String name) {
 	 * @return The new filled NBT list.
 	 */
 	// @SafeVarargs
-	@SuppressWarnings("unchecked")
 	public static <T> NbtList<T> ofList(String name, T... elements) {
 		return WrappedList.fromArray(name, elements);
 	}

ProtocolLib/src/test/java/com/comphenix/protocol/wrappers/WrappedGameProfileTest.java

@@ -9,16 +9,29 @@
 
 import com.comphenix.protocol.BukkitInitialization;
 import com.google.common.base.Charsets;
+import com.google.common.collect.Multimap;
+import com.mojang.authlib.GameProfile;
+import com.mojang.authlib.properties.Property;
+import com.mojang.authlib.properties.PropertyMap;
 
 public class WrappedGameProfileTest {
 
 	@BeforeClass
 	public static void initializeBukkit() {
 		BukkitInitialization.initializePackage();
 	}
-	
-	@SuppressWarnings("deprecation")
+
 	@Test
+	public void testWrapper() {
+		GameProfile profile = new GameProfile(UUID.nameUUIDFromBytes("ProtocolLib".getBytes(Charsets.UTF_8)), "ProtocolLib");
+		WrappedGameProfile wrapper = WrappedGameProfile.fromHandle(profile);
+
+		assertEquals(profile.getId(), wrapper.getUUID());
+		assertEquals(profile.getName(), wrapper.getName());
+	}
+
+	@Test
+	@SuppressWarnings("deprecation")
 	public void testSkinUpdate() {
 		final UUID uuid = UUID.nameUUIDFromBytes("123".getBytes(Charsets.UTF_8));
 
@@ -31,4 +44,41 @@ public void testSkinUpdate() {
 	public void testNullFailure() {
 		new WrappedGameProfile((String)null, null);
 	}
-}
+
+	@Test
+	public void testGetProperties() {
+		GameProfile profile = new GameProfile(UUID.nameUUIDFromBytes("ProtocolLib".getBytes(Charsets.UTF_8)), "ProtocolLib");
+
+		String name = "test";
+		String value = "test";
+		String signature = null;
+
+		profile.getProperties().put(name, new Property(name, value, signature));
+
+		WrappedGameProfile wrapper = WrappedGameProfile.fromHandle(profile);
+		Multimap<String, WrappedSignedProperty> properties = wrapper.getProperties();
+		WrappedSignedProperty property = properties.get(name).iterator().next();
+
+		assertEquals(property.getName(), name);
+		assertEquals(property.getValue(), value);
+		assertEquals(property.getSignature(), signature);
+	}
+
+	@Test
+	public void testAddProperties() {
+		String name = "test";
+		String value = "test";
+		String signature = null;
+
+		WrappedGameProfile wrapper = new WrappedGameProfile(UUID.nameUUIDFromBytes("ProtocolLib".getBytes(Charsets.UTF_8)), "ProtocolLib");
+		wrapper.getProperties().put(name, new WrappedSignedProperty(name, value, signature));
+
+		GameProfile profile = (GameProfile) wrapper.getHandle();
+		PropertyMap properties = profile.getProperties();
+		Property property = properties.get(name).iterator().next();
+
+		assertEquals(property.getName(), name);
+		assertEquals(property.getValue(), value);
+		assertEquals(property.getSignature(), signature);
+	}
+}