Add additional tests for CraftBlockData

2 files changed, 45 insertions, 0 deletions

diff --git a/src/test/java/org/bukkit/BlockDataConversionTest.java b/src/test/java/org/bukkit/BlockDataConversionTest.java
new file mode 100644
index 00000000..46cc26b3
--- /dev/null
+++ b/src/test/java/org/bukkit/BlockDataConversionTest.java
@@ -0,0 +1,38 @@
+package org.bukkit;
+
+import java.util.ArrayList;
+import java.util.List;
+import net.minecraft.server.Block;
+import net.minecraft.server.IBlockData;
+import net.minecraft.server.IRegistry;
+import org.bukkit.craftbukkit.block.data.CraftBlockData;
+import org.bukkit.support.AbstractTestingBase;
+import org.junit.Assert;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
+
+/**
+ * This test class ensures that all Blocks (as registered in IRegistry.BLOCK)
+ * can be converted into their CraftBlockData equivalent.
+ */
+@RunWith(Parameterized.class)
+public class BlockDataConversionTest extends AbstractTestingBase {
+
+    @Parameterized.Parameters(name = "{index}: {0}")
+    public static List<Object[]> args() {
+        List<Object[]> list = new ArrayList<>();
+        for (Block block : (Iterable<Block>) IRegistry.BLOCK) {
+            list.add(new Object[]{block.getBlockData()});
+        }
+        return list;
+    }
+
+    @Parameterized.Parameter(0) public IBlockData data;
+
+    @Test
+    public void testNotNull() {
+        Assert.assertNotNull(data);
+        Assert.assertNotNull(CraftBlockData.fromData(data));
+    }
+}
diff --git a/src/test/java/org/bukkit/PerMaterialTest.java b/src/test/java/org/bukkit/PerMaterialTest.java
index 82b5b1af..7967c4c3 100644
--- a/src/test/java/org/bukkit/PerMaterialTest.java
+++ b/src/test/java/org/bukkit/PerMaterialTest.java
@@ -229,4 +229,11 @@ public class PerMaterialTest extends AbstractTestingBase {
             assertThat(material.getBlastResistance(), is(CraftMagicNumbers.getBlock(material).getDurability()));
         }
     }
+
+    @Test
+    public void testBlockDataCreation() {
+        if (material.isBlock()) {
+            assertNotNull(material.createBlockData());
+        }
+    }
 }