Upstream/bentobox
1
0
Fork 0
mirror of https://github.com/BentoBoxWorld/BentoBox.git synced 2025-01-09 18:08:27 +01:00
Code Issues Projects Releases Wiki Activity

Prevents items from disappearing when blocked from breaking blocks

Browse Source 
https://github.com/BentoBoxWorld/BentoBox/issues/846
This commit is contained in:
tastybento 2019-07-18 22:43:35 -07:00
parent e99f84f7c7
commit 74372b391a
2 changed files with 35 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/world/bentobox/bentobox/listeners/flags/protection/BreakBlocksListener.java
View File

@ -114,7 +114,7 @@ public class BreakBlocksListener extends FlagListener {
Projectile p = (Projectile) e.getDamager(); Projectile p = (Projectile) e.getDamager();
if (p.getShooter() instanceof Player && !checkIsland(e, (Player)p.getShooter(), e.getEntity().getLocation(), Flags.BREAK_BLOCKS)) { if (p.getShooter() instanceof Player && !checkIsland(e, (Player)p.getShooter(), e.getEntity().getLocation(), Flags.BREAK_BLOCKS)) {
e.getEntity().setFireTicks(0); e.getEntity().setFireTicks(0);
e.getDamager().remove(); p.setFireTicks(0);
} }
} }
} }

64
src/test/java/world/bentobox/bentobox/listeners/flags/protection/BreakBlocksListenerTest.java
View File

@ -3,7 +3,10 @@ package world.bentobox.bentobox.listeners.flags.protection;
import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue; import static org.junit.Assert.assertTrue;
import static org.mockito.Matchers.any; import static org.mockito.Matchers.any;
import static org.mockito.Matchers.eq;
import static org.mockito.Mockito.mock; import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when; import static org.mockito.Mockito.when;
import java.util.Arrays; import java.util.Arrays;
@ -47,6 +50,7 @@ import org.junit.After;
import org.junit.Before; import org.junit.Before;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.mockito.Mock;
import org.mockito.Mockito; import org.mockito.Mockito;
import org.mockito.stubbing.Answer; import org.mockito.stubbing.Answer;
import org.powermock.api.mockito.PowerMockito; import org.powermock.api.mockito.PowerMockito;
@ -76,14 +80,21 @@ import world.bentobox.bentobox.util.Util;
@PrepareForTest( {BentoBox.class, Flags.class, Util.class, Bukkit.class} ) @PrepareForTest( {BentoBox.class, Flags.class, Util.class, Bukkit.class} )
public class BreakBlocksListenerTest { public class BreakBlocksListenerTest {
@Mock
private Location location; private Location location;
@Mock
private BentoBox plugin; private BentoBox plugin;
@Mock
private Notifier notifier; private Notifier notifier;
private BreakBlocksListener bbl; private BreakBlocksListener bbl;
@Mock
private Player player; private Player player;
@Mock
private World world; private World world;
@Mock
private Island island; private Island island;
@Mock
private IslandWorldManager iwm; private IslandWorldManager iwm;
@ -93,11 +104,9 @@ public class BreakBlocksListenerTest {
@Before @Before
public void setUp() throws Exception { public void setUp() throws Exception {
// Set up plugin // Set up plugin
plugin = mock(BentoBox.class);
Whitebox.setInternalState(BentoBox.class, "instance", plugin); Whitebox.setInternalState(BentoBox.class, "instance", plugin);
Server server = mock(Server.class); Server server = mock(Server.class);
world = mock(World.class);
when(server.getLogger()).thenReturn(Logger.getAnonymousLogger()); when(server.getLogger()).thenReturn(Logger.getAnonymousLogger());
when(server.getWorld("world")).thenReturn(world); when(server.getWorld("world")).thenReturn(world);
when(server.getVersion()).thenReturn("BSB_Mocking"); when(server.getVersion()).thenReturn("BSB_Mocking");
@ -115,7 +124,8 @@ public class BreakBlocksListenerTest {
when(itemFactory.getItemMeta(any())).thenReturn(meta); when(itemFactory.getItemMeta(any())).thenReturn(meta);
when(Bukkit.getItemFactory()).thenReturn(itemFactory); when(Bukkit.getItemFactory()).thenReturn(itemFactory);
when(Bukkit.getLogger()).thenReturn(Logger.getAnonymousLogger()); when(Bukkit.getLogger()).thenReturn(Logger.getAnonymousLogger());
location = mock(Location.class);
// Location
when(location.getWorld()).thenReturn(world); when(location.getWorld()).thenReturn(world);
when(location.getBlockX()).thenReturn(0); when(location.getBlockX()).thenReturn(0);
when(location.getBlockY()).thenReturn(0); when(location.getBlockY()).thenReturn(0);
@ -126,8 +136,7 @@ public class BreakBlocksListenerTest {
when(plugin.getFlagsManager()).thenReturn(flagsManager); when(plugin.getFlagsManager()).thenReturn(flagsManager);
// Worlds // Island World Manager
iwm = mock(IslandWorldManager.class);
when(iwm.inWorld(any(World.class))).thenReturn(true); when(iwm.inWorld(any(World.class))).thenReturn(true);
when(iwm.inWorld(any(Location.class))).thenReturn(true); when(iwm.inWorld(any(Location.class))).thenReturn(true);
when(plugin.getIWM()).thenReturn(iwm); when(plugin.getIWM()).thenReturn(iwm);
@ -152,31 +161,28 @@ public class BreakBlocksListenerTest {
// World Settings // World Settings
WorldSettings ws = mock(WorldSettings.class); WorldSettings ws = mock(WorldSettings.class);
when(iwm.getWorldSettings(Mockito.any())).thenReturn(ws); when(iwm.getWorldSettings(any())).thenReturn(ws);
Map<String, Boolean> worldFlags = new HashMap<>(); Map<String, Boolean> worldFlags = new HashMap<>();
when(ws.getWorldFlags()).thenReturn(worldFlags); when(ws.getWorldFlags()).thenReturn(worldFlags);
// Island manager // Island manager
IslandsManager im = mock(IslandsManager.class); IslandsManager im = mock(IslandsManager.class);
when(plugin.getIslands()).thenReturn(im); when(plugin.getIslands()).thenReturn(im);
island = mock(Island.class);
Optional<Island> optional = Optional.of(island); Optional<Island> optional = Optional.of(island);
when(im.getProtectedIslandAt(Mockito.any())).thenReturn(optional); when(im.getProtectedIslandAt(any())).thenReturn(optional);
// Default is that everything is allowed // Default is that everything is allowed
when(island.isAllowed(Mockito.any(), Mockito.any())).thenReturn(true); when(island.isAllowed(any(), any())).thenReturn(true);
// Notifier // Notifier
notifier = mock(Notifier.class);
when(plugin.getNotifier()).thenReturn(notifier); when(plugin.getNotifier()).thenReturn(notifier);
PowerMockito.mockStatic(Util.class); PowerMockito.mockStatic(Util.class);
when(Util.getWorld(Mockito.any())).thenReturn(mock(World.class)); when(Util.getWorld(any())).thenReturn(mock(World.class));
// Addon // Addon
when(iwm.getAddon(Mockito.any())).thenReturn(Optional.empty()); when(iwm.getAddon(any())).thenReturn(Optional.empty());
// Player // Player
player = mock(Player.class);
when(player.getLocation()).thenReturn(location); when(player.getLocation()).thenReturn(location);
when(player.getUniqueId()).thenReturn(UUID.randomUUID()); when(player.getUniqueId()).thenReturn(UUID.randomUUID());
when(player.getName()).thenReturn("tastybento"); when(player.getName()).thenReturn("tastybento");
@ -208,13 +214,13 @@ public class BreakBlocksListenerTest {
*/ */
@Test @Test
public void testOnBlockBreakNotAllowed() { public void testOnBlockBreakNotAllowed() {
when(island.isAllowed(Mockito.any(), Mockito.any())).thenReturn(false); when(island.isAllowed(any(), any())).thenReturn(false);
Block block = mock(Block.class); Block block = mock(Block.class);
when(block.getLocation()).thenReturn(location); when(block.getLocation()).thenReturn(location);
BlockBreakEvent e = new BlockBreakEvent(block, player); BlockBreakEvent e = new BlockBreakEvent(block, player);
bbl.onBlockBreak(e); bbl.onBlockBreak(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(notifier).notify(Mockito.any(), Mockito.eq("protection.protected")); verify(notifier).notify(any(), eq("protection.protected"));
} }
/** /**
@ -235,14 +241,14 @@ public class BreakBlocksListenerTest {
*/ */
@Test @Test
public void testOnBreakHangingNotAllowed() { public void testOnBreakHangingNotAllowed() {
when(island.isAllowed(Mockito.any(), Mockito.any())).thenReturn(false); when(island.isAllowed(any(), any())).thenReturn(false);
Hanging hanging = mock(Hanging.class); Hanging hanging = mock(Hanging.class);
when(hanging.getLocation()).thenReturn(location); when(hanging.getLocation()).thenReturn(location);
RemoveCause cause = RemoveCause.ENTITY; RemoveCause cause = RemoveCause.ENTITY;
HangingBreakByEntityEvent e = new HangingBreakByEntityEvent(hanging, player, cause); HangingBreakByEntityEvent e = new HangingBreakByEntityEvent(hanging, player, cause);
bbl.onBreakHanging(e); bbl.onBreakHanging(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(notifier).notify(Mockito.any(), Mockito.eq("protection.protected")); verify(notifier).notify(any(), eq("protection.protected"));
} }
/** /**
@ -312,7 +318,7 @@ public class BreakBlocksListenerTest {
*/ */
@Test @Test
public void testOnPlayerInteractHitCakeSpawnerDragonEggNotOK() { public void testOnPlayerInteractHitCakeSpawnerDragonEggNotOK() {
when(island.isAllowed(Mockito.any(), Mockito.any())).thenReturn(false); when(island.isAllowed(any(), any())).thenReturn(false);
ItemStack item = mock(ItemStack.class); ItemStack item = mock(ItemStack.class);
Block block = mock(Block.class); Block block = mock(Block.class);
when(block.getLocation()).thenReturn(location); when(block.getLocation()).thenReturn(location);
@ -328,7 +334,7 @@ public class BreakBlocksListenerTest {
e = new PlayerInteractEvent(player, Action.LEFT_CLICK_BLOCK, item, block, BlockFace.EAST); e = new PlayerInteractEvent(player, Action.LEFT_CLICK_BLOCK, item, block, BlockFace.EAST);
bbl.onPlayerInteract(e); bbl.onPlayerInteract(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(notifier, Mockito.times(3)).notify(Mockito.any(), Mockito.eq("protection.protected")); verify(notifier, times(3)).notify(any(), eq("protection.protected"));
} }
/** /**
@ -348,13 +354,13 @@ public class BreakBlocksListenerTest {
*/ */
@Test @Test
public void testOnVehicleDamageEventNotAllowed() { public void testOnVehicleDamageEventNotAllowed() {
when(island.isAllowed(Mockito.any(), Mockito.any())).thenReturn(false); when(island.isAllowed(any(), any())).thenReturn(false);
Vehicle vehicle = mock(Vehicle.class); Vehicle vehicle = mock(Vehicle.class);
when(vehicle.getLocation()).thenReturn(location); when(vehicle.getLocation()).thenReturn(location);
VehicleDamageEvent e = new VehicleDamageEvent(vehicle, player, 10); VehicleDamageEvent e = new VehicleDamageEvent(vehicle, player, 10);
bbl.onVehicleDamageEvent(e); bbl.onVehicleDamageEvent(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(notifier).notify(Mockito.any(), Mockito.eq("protection.protected")); verify(notifier).notify(any(), eq("protection.protected"));
} }
/** /**
@ -421,7 +427,7 @@ public class BreakBlocksListenerTest {
*/ */
@Test @Test
public void testOnEntityDamageNotAllowed() { public void testOnEntityDamageNotAllowed() {
when(island.isAllowed(Mockito.any(), Mockito.any())).thenReturn(false); when(island.isAllowed(any(), any())).thenReturn(false);
DamageCause cause = DamageCause.ENTITY_ATTACK; DamageCause cause = DamageCause.ENTITY_ATTACK;
Entity damagee = mock(ArmorStand.class); Entity damagee = mock(ArmorStand.class);
when(damagee.getLocation()).thenReturn(location); when(damagee.getLocation()).thenReturn(location);
@ -439,7 +445,7 @@ public class BreakBlocksListenerTest {
e = new EntityDamageByEntityEvent(damager, damagee, cause, 10); e = new EntityDamageByEntityEvent(damager, damagee, cause, 10);
bbl.onEntityDamage(e); bbl.onEntityDamage(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(notifier, Mockito.times(3)).notify(Mockito.any(), Mockito.eq("protection.protected")); verify(notifier, times(3)).notify(any(), eq("protection.protected"));
} }
/** /**
@ -492,7 +498,7 @@ public class BreakBlocksListenerTest {
*/ */
@Test @Test
public void testOnEntityDamageNotAllowedProjectile() { public void testOnEntityDamageNotAllowedProjectile() {
when(island.isAllowed(Mockito.any(), Mockito.any())).thenReturn(false); when(island.isAllowed(any(), any())).thenReturn(false);
DamageCause cause = DamageCause.ENTITY_ATTACK; DamageCause cause = DamageCause.ENTITY_ATTACK;
Entity damagee = mock(ArmorStand.class); Entity damagee = mock(ArmorStand.class);
when(damagee.getLocation()).thenReturn(location); when(damagee.getLocation()).thenReturn(location);
@ -501,23 +507,21 @@ public class BreakBlocksListenerTest {
EntityDamageByEntityEvent e = new EntityDamageByEntityEvent(damager, damagee, cause, 10); EntityDamageByEntityEvent e = new EntityDamageByEntityEvent(damager, damagee, cause, 10);
bbl.onEntityDamage(e); bbl.onEntityDamage(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(damagee).setFireTicks(0); verify(damagee).setFireTicks(0);
damagee = mock(ItemFrame.class); damagee = mock(ItemFrame.class);
when(damagee.getLocation()).thenReturn(location); when(damagee.getLocation()).thenReturn(location);
e = new EntityDamageByEntityEvent(damager, damagee, cause, 10); e = new EntityDamageByEntityEvent(damager, damagee, cause, 10);
bbl.onEntityDamage(e); bbl.onEntityDamage(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(damagee).setFireTicks(0); verify(damagee).setFireTicks(0);
damagee = mock(EnderCrystal.class); damagee = mock(EnderCrystal.class);
when(damagee.getLocation()).thenReturn(location); when(damagee.getLocation()).thenReturn(location);
e = new EntityDamageByEntityEvent(damager, damagee, cause, 10); e = new EntityDamageByEntityEvent(damager, damagee, cause, 10);
bbl.onEntityDamage(e); bbl.onEntityDamage(e);
assertTrue(e.isCancelled()); assertTrue(e.isCancelled());
Mockito.verify(notifier, Mockito.times(3)).notify(Mockito.any(), Mockito.eq("protection.protected")); verify(notifier, times(3)).notify(any(), eq("protection.protected"));
Mockito.verify(damager, Mockito.times(3)).remove(); verify(damagee).setFireTicks(0);
Mockito.verify(damagee).setFireTicks(0);
} }
} }