Clean up test classes; use latest API
This commit is contained in:
parent
233e5ceb65
commit
6894695024
2
pom.xml
2
pom.xml
|
@ -76,7 +76,7 @@
|
|||
<spigot.version>1.19.4-R0.1-SNAPSHOT</spigot.version>
|
||||
<!-- Might differ from the last Spigot release for short periods
|
||||
of time -->
|
||||
<paper.version>1.19.4-R0.1-SNAPSHOT</paper.version>
|
||||
<paper.version>1.19.3-R0.1-SNAPSHOT</paper.version>
|
||||
<bstats.version>3.0.0</bstats.version>
|
||||
<vault.version>1.7.1</vault.version>
|
||||
<placeholderapi.version>2.10.9</placeholderapi.version>
|
||||
|
|
|
@ -29,7 +29,6 @@ import org.junit.Test;
|
|||
import org.junit.runner.RunWith;
|
||||
import org.mockito.Mock;
|
||||
import org.mockito.Mockito;
|
||||
import org.mockito.invocation.InvocationOnMock;
|
||||
import org.mockito.stubbing.Answer;
|
||||
import org.powermock.api.mockito.PowerMockito;
|
||||
import org.powermock.core.classloader.annotations.PrepareForTest;
|
||||
|
|
|
@ -324,7 +324,7 @@ public class StandardSpawnProtectionListenerTest {
|
|||
*/
|
||||
@Test
|
||||
public void testOnBucketEmptyDisallowed() {
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.DOWN, null, null);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.DOWN, null, null, null);
|
||||
ssp.onBucketEmpty(e);
|
||||
assertTrue(e.isCancelled());
|
||||
verify(player).sendMessage("protection.spawn-protected");
|
||||
|
@ -336,7 +336,7 @@ public class StandardSpawnProtectionListenerTest {
|
|||
@Test
|
||||
public void testOnBucketEmptyDisallowedNoProtection() {
|
||||
when(ws.isMakeNetherPortals()).thenReturn(true);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.DOWN, null, null);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.DOWN, null, null, null);
|
||||
ssp.onBucketEmpty(e);
|
||||
assertFalse(e.isCancelled());
|
||||
verify(player, never()).sendMessage("protection.spawn-protected");
|
||||
|
@ -348,7 +348,7 @@ public class StandardSpawnProtectionListenerTest {
|
|||
@Test
|
||||
public void testOnBucketEmptyAllowed() {
|
||||
when(player.isOp()).thenReturn(true);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.DOWN, null, null);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.DOWN, null, null, null);
|
||||
ssp.onBucketEmpty(e);
|
||||
assertFalse(e.isCancelled());
|
||||
verify(player, never()).sendMessage("protection.spawn-protected");
|
||||
|
|
|
@ -17,6 +17,7 @@ import org.bukkit.entity.TropicalFish;
|
|||
import org.bukkit.event.player.PlayerBucketEmptyEvent;
|
||||
import org.bukkit.event.player.PlayerBucketFillEvent;
|
||||
import org.bukkit.event.player.PlayerInteractEntityEvent;
|
||||
import org.bukkit.inventory.EquipmentSlot;
|
||||
import org.bukkit.inventory.ItemStack;
|
||||
import org.bukkit.inventory.PlayerInventory;
|
||||
import org.junit.Before;
|
||||
|
@ -65,7 +66,7 @@ public class BucketListenerTest extends AbstractCommonSetup {
|
|||
when(block.getLocation()).thenReturn(location);
|
||||
when(block.getRelative(any())).thenReturn(block);
|
||||
ItemStack item = mock(ItemStack.class);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketEmpty(e);
|
||||
assertFalse(e.isCancelled());
|
||||
}
|
||||
|
@ -80,7 +81,7 @@ public class BucketListenerTest extends AbstractCommonSetup {
|
|||
when(block.getLocation()).thenReturn(location);
|
||||
when(block.getRelative(any())).thenReturn(block);
|
||||
ItemStack item = mock(ItemStack.class);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
PlayerBucketEmptyEvent e = new PlayerBucketEmptyEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketEmpty(e);
|
||||
assertTrue(e.isCancelled());
|
||||
verify(notifier).notify(any(), eq("protection.protected"));
|
||||
|
@ -96,22 +97,22 @@ public class BucketListenerTest extends AbstractCommonSetup {
|
|||
when(block.getRelative(any())).thenReturn(block);
|
||||
ItemStack item = mock(ItemStack.class);
|
||||
when(item.getType()).thenReturn(Material.WATER_BUCKET);
|
||||
PlayerBucketFillEvent e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
PlayerBucketFillEvent e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertFalse(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertFalse(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.LAVA_BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertFalse(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.MILK_BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertFalse(e.isCancelled());
|
||||
}
|
||||
|
@ -127,22 +128,22 @@ public class BucketListenerTest extends AbstractCommonSetup {
|
|||
when(block.getRelative(any())).thenReturn(block);
|
||||
ItemStack item = mock(ItemStack.class);
|
||||
when(item.getType()).thenReturn(Material.WATER_BUCKET);
|
||||
PlayerBucketFillEvent e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
PlayerBucketFillEvent e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertTrue(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertTrue(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.LAVA_BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertTrue(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.MILK_BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertTrue(e.isCancelled());
|
||||
|
||||
|
@ -163,22 +164,22 @@ public class BucketListenerTest extends AbstractCommonSetup {
|
|||
when(block.getRelative(any())).thenReturn(block);
|
||||
ItemStack item = mock(ItemStack.class);
|
||||
when(item.getType()).thenReturn(Material.WATER_BUCKET);
|
||||
PlayerBucketFillEvent e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
PlayerBucketFillEvent e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertFalse(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertTrue(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.LAVA_BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertFalse(e.isCancelled());
|
||||
|
||||
when(item.getType()).thenReturn(Material.MILK_BUCKET);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item);
|
||||
e = new PlayerBucketFillEvent(player, block, block, BlockFace.UP, Material.WATER_BUCKET, item, EquipmentSlot.HAND);
|
||||
l.onBucketFill(e);
|
||||
assertFalse(e.isCancelled());
|
||||
|
||||
|
|
|
@ -105,7 +105,7 @@ public class PlaceBlocksListenerTest extends AbstractCommonSetup {
|
|||
Hanging hanging = mock(Hanging.class);
|
||||
Block block = mock(Block.class);
|
||||
when(block.getLocation()).thenReturn(location);
|
||||
HangingPlaceEvent e = new HangingPlaceEvent(hanging, player, block, BlockFace.EAST, null);
|
||||
HangingPlaceEvent e = new HangingPlaceEvent(hanging, player, block, BlockFace.EAST, null, null);
|
||||
pbl.onHangingPlace(e);
|
||||
assertFalse(e.isCancelled());
|
||||
verify(notifier, never()).notify(any(), eq("protection.protected"));
|
||||
|
@ -120,7 +120,7 @@ public class PlaceBlocksListenerTest extends AbstractCommonSetup {
|
|||
Hanging hanging = mock(Hanging.class);
|
||||
Block block = mock(Block.class);
|
||||
when(block.getLocation()).thenReturn(location);
|
||||
HangingPlaceEvent e = new HangingPlaceEvent(hanging, player, block, BlockFace.EAST, null);
|
||||
HangingPlaceEvent e = new HangingPlaceEvent(hanging, player, block, BlockFace.EAST, null, null);
|
||||
pbl.onHangingPlace(e);
|
||||
assertTrue(e.isCancelled());
|
||||
verify(notifier).notify(any(), eq("protection.protected"));
|
||||
|
|
Loading…
Reference in New Issue