From df17927d453745954950c74574b6f480504017a7 Mon Sep 17 00:00:00 2001 From: Thinkofdeath Date: Sun, 25 Jan 2015 11:59:37 +0000 Subject: Fix PerMaterialTest skipping useful tests for certain blocks --- src/test/java/org/bukkit/PerMaterialTest.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/test/java/org/bukkit/PerMaterialTest.java') diff --git a/src/test/java/org/bukkit/PerMaterialTest.java b/src/test/java/org/bukkit/PerMaterialTest.java index f689cbf4..1b532e14 100644 --- a/src/test/java/org/bukkit/PerMaterialTest.java +++ b/src/test/java/org/bukkit/PerMaterialTest.java @@ -41,10 +41,6 @@ public class PerMaterialTest extends AbstractTestingBase { public static List data() { List list = Lists.newArrayList(); for (Material material : Material.values()) { - if (INVALIDATED_MATERIALS.contains(material)) { - continue; - } - list.add(new Object[] {material}); } return list; @@ -75,6 +71,8 @@ public class PerMaterialTest extends AbstractTestingBase { @Test public void maxDurability() { + if (INVALIDATED_MATERIALS.contains(material)) return; + if (material == Material.AIR) { assertThat((int) material.getMaxDurability(), is(0)); } else if (material.isBlock()){ @@ -85,6 +83,8 @@ public class PerMaterialTest extends AbstractTestingBase { @Test public void maxStackSize() { + if (INVALIDATED_MATERIALS.contains(material)) return; + final ItemStack bukkit = new ItemStack(material); final CraftItemStack craft = CraftItemStack.asCraftCopy(bukkit); if (material == Material.AIR) { -- cgit v1.2.3