Fixed tests in IslandResetCommandTest

This commit is contained in:
Florian CUNY 2019-04-08 11:34:51 +02:00
parent 79941b5e63
commit af2f600621

View File

@ -1,17 +1,5 @@
package world.bentobox.bentobox.api.commands.island; package world.bentobox.bentobox.api.commands.island;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Collections;
import java.util.HashMap;
import java.util.Map;
import java.util.UUID;
import org.bukkit.Bukkit; import org.bukkit.Bukkit;
import org.bukkit.World; import org.bukkit.World;
import org.bukkit.entity.Player; import org.bukkit.entity.Player;
@ -25,7 +13,6 @@ import org.powermock.api.mockito.PowerMockito;
import org.powermock.core.classloader.annotations.PrepareForTest; import org.powermock.core.classloader.annotations.PrepareForTest;
import org.powermock.modules.junit4.PowerMockRunner; import org.powermock.modules.junit4.PowerMockRunner;
import org.powermock.reflect.Whitebox; import org.powermock.reflect.Whitebox;
import world.bentobox.bentobox.BentoBox; import world.bentobox.bentobox.BentoBox;
import world.bentobox.bentobox.Settings; import world.bentobox.bentobox.Settings;
import world.bentobox.bentobox.api.commands.CompositeCommand; import world.bentobox.bentobox.api.commands.CompositeCommand;
@ -39,6 +26,18 @@ import world.bentobox.bentobox.managers.SchemsManager;
import world.bentobox.bentobox.managers.island.NewIsland; import world.bentobox.bentobox.managers.island.NewIsland;
import world.bentobox.bentobox.schems.Clipboard; import world.bentobox.bentobox.schems.Clipboard;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Collections;
import java.util.HashMap;
import java.util.Map;
import java.util.UUID;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
/** /**
* @author tastybento * @author tastybento
* *
@ -116,8 +115,6 @@ public class IslandResetCommandTest {
iwm = mock(IslandWorldManager.class); iwm = mock(IslandWorldManager.class);
when(iwm.getFriendlyName(Mockito.any())).thenReturn("BSkyBlock"); when(iwm.getFriendlyName(Mockito.any())).thenReturn("BSkyBlock");
when(plugin.getIWM()).thenReturn(iwm); when(plugin.getIWM()).thenReturn(iwm);
when(iwm.getResetLimit(Mockito.any())).thenReturn(3);
// Schems manager - custom schem // Schems manager - custom schem
SchemsManager sm = mock(SchemsManager.class); SchemsManager sm = mock(SchemsManager.class);
@ -186,7 +183,7 @@ public class IslandResetCommandTest {
// Now has no team // Now has no team
when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false); when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false);
// Give the user some resets // Give the user some resets
when(pm.getResets(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1); when(pm.getResetsLeft(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(2);
// Set so no confirmation required // Set so no confirmation required
when(s.isResetConfirmation()).thenReturn(false); when(s.isResetConfirmation()).thenReturn(false);
@ -220,8 +217,6 @@ public class IslandResetCommandTest {
when(im.isOwner(Mockito.any(), Mockito.eq(uuid))).thenReturn(true); when(im.isOwner(Mockito.any(), Mockito.eq(uuid))).thenReturn(true);
// Now has no team // Now has no team
when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false); when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false);
// Give the user some resets
when(pm.getResets(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1);
// Set so no confirmation required // Set so no confirmation required
when(s.isResetConfirmation()).thenReturn(false); when(s.isResetConfirmation()).thenReturn(false);
@ -239,7 +234,7 @@ public class IslandResetCommandTest {
PowerMockito.mockStatic(NewIsland.class); PowerMockito.mockStatic(NewIsland.class);
when(NewIsland.builder()).thenReturn(builder); when(NewIsland.builder()).thenReturn(builder);
// Test with unlimited resets // Test with unlimited resets
when(iwm.getResetLimit(Mockito.any())).thenReturn(-1); when(pm.getResetsLeft(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(-1);
// Reset // Reset
assertTrue(irc.execute(user, irc.getLabel(), new ArrayList<>())); assertTrue(irc.execute(user, irc.getLabel(), new ArrayList<>()));
@ -259,7 +254,7 @@ public class IslandResetCommandTest {
// Now has no team // Now has no team
when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false); when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false);
// Give the user some resets // Give the user some resets
when(pm.getResets(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1); when(pm.getResetsLeft(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1);
// Set so no confirmation required // Set so no confirmation required
when(s.isResetConfirmation()).thenReturn(false); when(s.isResetConfirmation()).thenReturn(false);
@ -288,7 +283,6 @@ public class IslandResetCommandTest {
// Send command again to confirm // Send command again to confirm
assertTrue(irc.execute(user, irc.getLabel(), new ArrayList<>())); assertTrue(irc.execute(user, irc.getLabel(), new ArrayList<>()));
} }
@Test @Test
@ -301,7 +295,7 @@ public class IslandResetCommandTest {
// Now has no team // Now has no team
when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false); when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false);
// Give the user some resets // Give the user some resets
when(pm.getResets(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1); when(pm.getResetsLeft(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1);
// Set so no confirmation required // Set so no confirmation required
when(s.isResetConfirmation()).thenReturn(false); when(s.isResetConfirmation()).thenReturn(false);
@ -325,8 +319,6 @@ public class IslandResetCommandTest {
// Reset // Reset
assertFalse(irc.execute(user, irc.getLabel(), new ArrayList<>())); assertFalse(irc.execute(user, irc.getLabel(), new ArrayList<>()));
Mockito.verify(user).sendMessage("commands.island.create.unable-create-island"); Mockito.verify(user).sendMessage("commands.island.create.unable-create-island");
} }
@Test @Test
@ -339,7 +331,7 @@ public class IslandResetCommandTest {
// Now has no team // Now has no team
when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false); when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false);
// Give the user some resets // Give the user some resets
when(pm.getResets(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1); when(pm.getResetsLeft(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1);
// Set so no confirmation required // Set so no confirmation required
when(s.isResetConfirmation()).thenReturn(false); when(s.isResetConfirmation()).thenReturn(false);
@ -373,7 +365,7 @@ public class IslandResetCommandTest {
// Now has no team // Now has no team
when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false); when(im.inTeam(Mockito.any(), Mockito.eq(uuid))).thenReturn(false);
// Give the user some resets // Give the user some resets
when(pm.getResets(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1); when(pm.getResetsLeft(Mockito.eq(world), Mockito.eq(uuid))).thenReturn(1);
// Set so no confirmation required // Set so no confirmation required
when(s.isResetConfirmation()).thenReturn(false); when(s.isResetConfirmation()).thenReturn(false);
@ -397,7 +389,6 @@ public class IslandResetCommandTest {
assertTrue(irc.execute(user, irc.getLabel(), Collections.singletonList("custom"))); assertTrue(irc.execute(user, irc.getLabel(), Collections.singletonList("custom")));
// Verify that build new island was called and the number of resets left shown // Verify that build new island was called and the number of resets left shown
Mockito.verify(builder).build(); Mockito.verify(builder).build();
// This should not be shown Mockito.verify(user).sendMessage("commands.island.reset.resets-left", "[number]", "1");
Mockito.verify(user, Mockito.never()).sendMessage("commands.island.reset.resets-left", "[number]", "1");
} }
} }