Added test class and made some methods easier to test

src/main/java/world/bentobox/bentobox/api/commands/CompositeCommand.java

@@ -758,7 +758,7 @@ public abstract class CompositeCommand extends Command implements PluginIdentifi
      * @param user    - the User
      * @return result of help command or false if no help defined
      */
-    protected boolean showHelp(CompositeCommand command, User user) {
+    public boolean showHelp(CompositeCommand command, User user) {
         return command.getSubCommand("help")
                 .map(helpCommand -> helpCommand.execute(user, helpCommand.getLabel(), new ArrayList<>())).orElse(false);
     }

src/main/java/world/bentobox/bentobox/api/commands/admin/AdminMaxHomesCommand.java

@@ -9,7 +9,6 @@ import java.util.UUID;
 
 import com.google.common.primitives.Ints;
 
-import world.bentobox.bentobox.BentoBox;
 import world.bentobox.bentobox.api.commands.CompositeCommand;
 import world.bentobox.bentobox.api.commands.ConfirmableCommand;
 import world.bentobox.bentobox.api.localization.TextVariables;
@@ -32,8 +31,8 @@ import world.bentobox.bentobox.util.Util;
 
 public class AdminMaxHomesCommand extends ConfirmableCommand {
 
-    private Integer maxHomes;
+    Integer maxHomes;
-    private Map<String, Island> islands = new HashMap<>();
+    Map<String, Island> islands = new HashMap<>();
 
     public AdminMaxHomesCommand(CompositeCommand parent) {
         super(parent, "setmaxhomes");
@@ -146,7 +145,7 @@ public class AdminMaxHomesCommand extends ConfirmableCommand {
 
     }
 
-    private Map<String, Island> getNameIslandMap(User user) {
+    Map<String, Island> getNameIslandMap(User user) {
         Map<String, Island> islandMap = new HashMap<>();
         int index = 0;
         for (Island island : getIslands().getIslands(getWorld(), user.getUniqueId())) {

src/test/java/world/bentobox/bentobox/api/commands/admin/AdminDeleteCommandTest.java

@@ -68,8 +68,6 @@ public class AdminDeleteCommandTest {
     @Mock
     private @Nullable Island island;
 
-    /**
-     */
     @Before
     public void setUp() throws Exception {
         PowerMockito.mockStatic(Bukkit.class, Mockito.RETURNS_MOCKS);

src/test/java/world/bentobox/bentobox/api/commands/admin/AdminMaxHomesCommandTest.java

@@ -0,0 +1,686 @@
+package world.bentobox.bentobox.api.commands.admin;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.eq;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.spy;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.Optional;
+import java.util.UUID;
+
+import org.bukkit.Bukkit;
+import org.bukkit.Location;
+import org.bukkit.World;
+import org.bukkit.entity.Player;
+import org.bukkit.scheduler.BukkitScheduler;
+import org.bukkit.scheduler.BukkitTask;
+import org.eclipse.jdt.annotation.Nullable;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mock;
+import org.mockito.Mockito;
+import org.powermock.api.mockito.PowerMockito;
+import org.powermock.core.classloader.annotations.PrepareForTest;
+import org.powermock.modules.junit4.PowerMockRunner;
+import org.powermock.reflect.Whitebox;
+
+import world.bentobox.bentobox.BentoBox;
+import world.bentobox.bentobox.Settings;
+import world.bentobox.bentobox.api.commands.CompositeCommand;
+import world.bentobox.bentobox.api.localization.TextVariables;
+import world.bentobox.bentobox.api.user.User;
+import world.bentobox.bentobox.database.objects.Island;
+import world.bentobox.bentobox.managers.CommandsManager;
+import world.bentobox.bentobox.managers.IslandWorldManager;
+import world.bentobox.bentobox.managers.IslandsManager;
+import world.bentobox.bentobox.managers.LocalesManager;
+import world.bentobox.bentobox.managers.PlayersManager;
+import world.bentobox.bentobox.util.Util;
+
+/**
+ * @author tastybento
+ */
+@RunWith(PowerMockRunner.class)
+@PrepareForTest({ Bukkit.class, BentoBox.class, User.class, Util.class })
+public class AdminMaxHomesCommandTest {
+
+    @Mock
+    private CompositeCommand ac;
+    @Mock
+    private User user;
+    @Mock
+    private IslandsManager im;
+    @Mock
+    private PlayersManager pm;
+    private UUID notUUID;
+    private UUID uuid;
+    @Mock
+    private World world;
+    @Mock
+    private @Nullable Island island;
+    private AdminMaxHomesCommand instance;
+    private String label;
+    private ArrayList<String> args = new ArrayList<>();
+
+    @Before
+    public void setUp() throws Exception {
+        PowerMockito.mockStatic(Bukkit.class, Mockito.RETURNS_MOCKS);
+
+        // Set up plugin
+        BentoBox plugin = mock(BentoBox.class);
+        Whitebox.setInternalState(BentoBox.class, "instance", plugin);
+
+        // Util
+        Util.setPlugin(plugin);
+
+        // Command manager
+        CommandsManager cm = mock(CommandsManager.class);
+        when(plugin.getCommandsManager()).thenReturn(cm);
+
+        // Settings
+        Settings s = mock(Settings.class);
+        when(s.getResetCooldown()).thenReturn(0);
+        when(plugin.getSettings()).thenReturn(s);
+
+        // Player
+        Player p = mock(Player.class);
+        // Sometimes use Mockito.withSettings().verboseLogging()
+        when(user.isOp()).thenReturn(false);
+        uuid = UUID.randomUUID();
+        notUUID = UUID.randomUUID();
+        while (notUUID.equals(uuid)) {
+            notUUID = UUID.randomUUID();
+        }
+        when(user.getUniqueId()).thenReturn(uuid);
+        when(user.getPlayer()).thenReturn(p);
+        when(user.getName()).thenReturn("tastybento");
+        User.setPlugin(plugin);
+
+        // Parent command has no aliases
+        when(ac.getSubCommandAliases()).thenReturn(new HashMap<>());
+        when(ac.getTopLabel()).thenReturn("admin");
+        when(ac.getWorld()).thenReturn(world);
+
+        // Island World Manager
+        IslandWorldManager iwm = mock(IslandWorldManager.class);
+        when(plugin.getIWM()).thenReturn(iwm);
+
+        // Player has island to begin with
+        when(im.hasIsland(any(), any(UUID.class))).thenReturn(true);
+        when(im.hasIsland(any(), any(User.class))).thenReturn(true);
+        // when(im.isOwner(any(),any())).thenReturn(true);
+        // when(im.getOwner(any(),any())).thenReturn(uuid);
+        when(im.getIsland(world, user)).thenReturn(island);
+        when(im.getIslands(world, notUUID)).thenReturn(List.of(island));
+        when(plugin.getIslands()).thenReturn(im);
+
+        // Island
+        when(island.getOwner()).thenReturn(uuid);
+        when(island.hasTeam()).thenReturn(true);
+
+        // Has team
+        when(im.inTeam(any(), eq(uuid))).thenReturn(true);
+
+        when(plugin.getPlayers()).thenReturn(pm);
+
+        // Server & Scheduler
+        BukkitScheduler sch = mock(BukkitScheduler.class);
+        when(Bukkit.getScheduler()).thenReturn(sch);
+        BukkitTask task = mock(BukkitTask.class);
+        when(sch.runTaskLater(any(), any(Runnable.class), any(Long.class))).thenReturn(task);
+
+        // Locales
+        LocalesManager lm = mock(LocalesManager.class);
+        when(lm.get(any(), any())).thenReturn("mock translation");
+        when(plugin.getLocalesManager()).thenReturn(lm);
+
+        instance = spy(new AdminMaxHomesCommand(ac));
+        label = "island";
+    }
+
+    @After
+    public void tearDown() {
+        User.clearUsers();
+        Mockito.framework().clearInlineMocks();
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsIsEmpty() {
+        // Arrange: args is already empty
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        // Verify that showHelp was called
+        verify(instance).showHelp(instance, user);
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSize1_UserNotPlayer() {
+        // Arrange
+        args.add("5");
+        when(user.isPlayer()).thenReturn(false);
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("general.errors.use-in-game");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSize1_WrongWorld() {
+        // Arrange
+        args.add("5");
+        when(user.isPlayer()).thenReturn(true);
+        World userWorld = mock(World.class);
+        World expectedWorld = mock(World.class);
+        when(user.getWorld()).thenReturn(userWorld);
+        doReturn(expectedWorld).when(instance).getWorld();
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("general.errors.wrong-world");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSize1_InvalidMaxHomes() {
+        // Arrange
+        args.add("notanumber");
+        when(user.isPlayer()).thenReturn(true);
+        World world = mock(World.class);
+        when(user.getWorld()).thenReturn(world);
+        doReturn(world).when(instance).getWorld();
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("general.errors.must-be-positive-number", TextVariables.NUMBER, "notanumber");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSize1_UserNotOnIsland() {
+        // Arrange
+        args.add("5");
+        when(user.isPlayer()).thenReturn(true);
+        World world = mock(World.class);
+        when(user.getWorld()).thenReturn(world);
+        doReturn(world).when(instance).getWorld();
+
+        Location location = mock(Location.class);
+        when(user.getLocation()).thenReturn(location);
+
+        when(im.getIslandAt(location)).thenReturn(Optional.empty());
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("general.errors.not-on-island");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSize1_Success() {
+        // Arrange
+        args.add("5");
+        when(user.isPlayer()).thenReturn(true);
+        World world = mock(World.class);
+        when(user.getWorld()).thenReturn(world);
+        doReturn(world).when(instance).getWorld();
+
+        Location location = mock(Location.class);
+        when(user.getLocation()).thenReturn(location);
+
+        Island island = mock(Island.class);
+        when(im.getIslandAt(location)).thenReturn(Optional.of(island));
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertTrue(result);
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSizeGreaterThan1_InvalidPlayer() {
+        // Arrange
+        args.add("UnknownPlayer");
+        args.add("5");
+
+        when(pm.getUUID("UnknownPlayer")).thenReturn(null);
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("general.errors.unknown-player", TextVariables.NAME, "UnknownPlayer");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSizeGreaterThan1_InvalidMaxHomes() {
+        // Arrange
+        args.add("ValidPlayer");
+        args.add("notanumber");
+
+        UUID playerUUID = UUID.randomUUID();
+        when(pm.getUUID("ValidPlayer")).thenReturn(playerUUID);
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("general.errors.must-be-positive-number", TextVariables.NUMBER, "notanumber");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSizeGreaterThan1_TargetPlayerHasNoIslands() {
+        // Arrange
+        args.add("ValidPlayer");
+        args.add("5");
+
+        UUID playerUUID = UUID.randomUUID();
+        when(pm.getUUID("ValidPlayer")).thenReturn(playerUUID);
+
+        User targetUser = mock(User.class);
+        // Mock static method User.getInstance(UUID)
+        // Assuming use of Mockito with inline mocking or PowerMockito
+        PowerMockito.mockStatic(User.class);
+        when(User.getInstance(playerUUID)).thenReturn(targetUser);
+
+        doReturn(new HashMap<String, Island>()).when(instance).getNameIslandMap(targetUser);
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("general.errors.player-has-no-island");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSizeGreaterThan2_UnknownIsland() {
+        // Arrange
+        args.add("ValidPlayer");
+        args.add("5");
+        args.add("UnknownIsland");
+
+        UUID playerUUID = UUID.randomUUID();
+        when(pm.getUUID("ValidPlayer")).thenReturn(playerUUID);
+
+        User targetUser = mock(User.class);
+        // Mock static method User.getInstance(UUID)
+        // Assuming use of Mockito with inline mocking or PowerMockito
+        PowerMockito.mockStatic(User.class);
+        when(User.getInstance(playerUUID)).thenReturn(targetUser);
+
+        Map<String, Island> islandsMap = new HashMap<>();
+        islandsMap.put("Island1", mock(Island.class));
+        doReturn(islandsMap).when(instance).getNameIslandMap(targetUser);
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+        verify(user).sendMessage("commands.admin.maxhomes.errors.unknown-island", TextVariables.NAME, "UnknownIsland");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#canExecute(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testArgsSizeGreaterThan1_Success() {
+        // Arrange
+        args.add("ValidPlayer");
+        args.add("5");
+
+        UUID playerUUID = UUID.randomUUID();
+        when(pm.getUUID("ValidPlayer")).thenReturn(playerUUID);
+
+        User targetUser = mock(User.class);
+        // Mock static method User.getInstance(UUID)
+        PowerMockito.mockStatic(User.class);
+        when(User.getInstance(playerUUID)).thenReturn(targetUser);
+
+        Map<String, Island> islandsMap = new HashMap<>();
+        islandsMap.put("", mock(Island.class)); // Assuming empty string key as in code
+        doReturn(islandsMap).when(instance).getNameIslandMap(targetUser);
+
+        // Act
+        boolean result = instance.canExecute(user, label, args);
+
+        // Assert
+        assertTrue(result);
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#setup()}.
+     */
+    @Test
+    public void testSetup() {
+        assertEquals("mod.maxhomes", instance.getPermission());
+        assertFalse(instance.isOnlyPlayer());
+        assertEquals("commands.admin.maxhomes.parameters", instance.getParameters());
+        assertEquals("commands.admin.maxhomes.description", instance.getDescription());
+
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testTabComplete_ArgsSize2_ReturnsPlayerNames() {
+        // Arrange
+        args.add("someArg"); // args.size() == 1
+        args.add(""); // args.size() == 2
+
+        // Mock Util.getOnlinePlayerList(user)
+        List<String> onlinePlayers = Arrays.asList("PlayerOne", "PlayerTwo");
+        PowerMockito.mockStatic(Util.class);
+        when(Util.getOnlinePlayerList(user)).thenReturn(onlinePlayers);
+
+        // Act
+        Optional<List<String>> result = instance.tabComplete(user, label, args);
+
+        // Assert
+        assertTrue(result.isPresent());
+        assertEquals(onlinePlayers, result.get());
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testTabComplete_ArgsSizeGreaterThan3_ReturnsIslandNames() {
+        // Arrange
+        args.add("someArg");
+        args.add("anotherArg");
+        args.add("thirdArg");
+        args.add(""); // args.size() == 4 (>3)
+        String lastArg = args.get(args.size() - 1);
+
+        Map<String, Island> nameIslandMap = new HashMap<>();
+        nameIslandMap.put("IslandOne", mock(Island.class));
+        nameIslandMap.put("IslandTwo", mock(Island.class));
+        doReturn(nameIslandMap).when(instance).getNameIslandMap(user);
+
+        // Create the list of island names
+        List<String> islandNames = new ArrayList<>(nameIslandMap.keySet());
+
+        // Mock Util.tabLimit()
+        List<String> limitedIslandNames = Arrays.asList("IslandOne", "IslandTwo");
+        PowerMockito.mockStatic(Util.class);
+        when(Util.tabLimit(islandNames, lastArg)).thenReturn(limitedIslandNames);
+
+        // Act
+        Optional<List<String>> result = instance.tabComplete(user, label, args);
+
+        // Assert
+        assertTrue(result.isPresent());
+        assertEquals(limitedIslandNames, result.get());
+
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testTabComplete_Otherwise_ReturnsListOfOne() {
+        // Arrange
+        args.add(""); // args.size() == 1
+
+        // Act
+        Optional<List<String>> result = instance.tabComplete(user, label, args);
+
+        // Assert
+        assertTrue(result.isPresent());
+        assertEquals(Collections.singletonList("1"), result.get());
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testTabComplete_ArgsSize3_ReturnsListOfOne() {
+        // Arrange
+        args.add("someArg");
+        args.add("anotherArg");
+        args.add(""); // args.size() == 3
+
+        // Act
+        Optional<List<String>> result = instance.tabComplete(user, label, args);
+
+        // Assert
+        assertTrue(result.isPresent());
+        assertEquals(Collections.singletonList("1"), result.get());
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testExecuteWithEmptyIslands_ShouldReturnFalse() {
+        // Arrange
+        instance.maxHomes = 5; // Set maxHomes to a valid number
+        instance.islands = new HashMap<>(); // Empty islands map
+
+        // Act
+        boolean result = instance.execute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testExecuteWithMaxHomesLessThanOne_ShouldReturnFalse() {
+        // Arrange
+        instance.maxHomes = 0; // Invalid maxHomes
+        Island island = mock(Island.class);
+        Map<String, Island> islandsMap = new HashMap<>();
+        islandsMap.put("TestIsland", island);
+        instance.islands = islandsMap;
+
+        // Act
+        boolean result = instance.execute(user, label, args);
+
+        // Assert
+        assertFalse(result);
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testExecuteSuccessful_SingleIsland() {
+        // Arrange
+        instance.maxHomes = 5;
+        Island island = mock(Island.class);
+        Map<String, Island> islandsMap = new HashMap<>();
+        islandsMap.put("TestIsland", island);
+        instance.islands = islandsMap;
+
+        // Act
+        boolean result = instance.execute(user, label, args);
+
+        // Assert
+        assertTrue(result);
+        verify(island).setMaxHomes(5);
+        verify(user).sendMessage("commands.admin.maxhomes.max-homes-set", TextVariables.NAME, "TestIsland",
+                TextVariables.NUMBER, "5");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testExecuteSuccessful_MultipleIslands() {
+        // Arrange
+        instance.maxHomes = 3;
+        Island island1 = mock(Island.class);
+        Island island2 = mock(Island.class);
+        Map<String, Island> islandsMap = new HashMap<>();
+        islandsMap.put("IslandOne", island1);
+        islandsMap.put("IslandTwo", island2);
+        instance.islands = islandsMap;
+
+        // Act
+        boolean result = instance.execute(user, label, args);
+
+        // Assert
+        assertTrue(result);
+        verify(island1).setMaxHomes(3);
+        verify(island2).setMaxHomes(3);
+        verify(user).sendMessage("commands.admin.maxhomes.max-homes-set", TextVariables.NAME, "IslandOne",
+                TextVariables.NUMBER, "3");
+        verify(user).sendMessage("commands.admin.maxhomes.max-homes-set", TextVariables.NAME, "IslandTwo",
+                TextVariables.NUMBER, "3");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testExecuteAfterSuccessfulCanExecute() {
+        // Arrange
+        args.add("5");
+        when(user.isPlayer()).thenReturn(true);
+        World world = mock(World.class);
+        when(user.getWorld()).thenReturn(world);
+        doReturn(world).when(instance).getWorld();
+
+        Location location = mock(Location.class);
+        when(user.getLocation()).thenReturn(location);
+
+        Island island = mock(Island.class);
+        when(im.getIslandAt(location)).thenReturn(Optional.of(island));
+
+        // Act
+        boolean canExecuteResult = instance.canExecute(user, label, args);
+        boolean executeResult = instance.execute(user, label, args);
+
+        // Assert
+        assertTrue(canExecuteResult);
+        assertTrue(executeResult);
+        verify(island).setMaxHomes(5);
+        verify(user).sendMessage("commands.admin.maxhomes.max-homes-set", TextVariables.NAME, "", TextVariables.NUMBER,
+                "5");
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testExecuteWithInvalidMaxHomesAfterCanExecute() {
+        // Arrange
+        args.add("-1");
+        when(user.isPlayer()).thenReturn(true);
+        World world = mock(World.class);
+        when(user.getWorld()).thenReturn(world);
+        doReturn(world).when(instance).getWorld();
+
+        // Act
+        boolean canExecuteResult = instance.canExecute(user, label, args);
+        boolean executeResult = instance.execute(user, label, args);
+
+        // Assert
+        assertFalse(canExecuteResult);
+        assertFalse(executeResult);
+    }
+
+    /**
+     * Test method for {@link world.bentobox.bentobox.api.commands.admin.AdminMaxHomesCommand#tabComplete(world.bentobox.bentobox.api.user.User, java.lang.String, java.util.List)}.
+     */
+    @Test
+    public void testExecuteWithMultipleIslandsAfterCanExecute() {
+        // Arrange
+        args.add("ValidPlayer");
+        args.add("4");
+
+        UUID playerUUID = UUID.randomUUID();
+        when(pm.getUUID("ValidPlayer")).thenReturn(playerUUID);
+
+        User targetUser = mock(User.class);
+        PowerMockito.mockStatic(User.class);
+        when(User.getInstance(playerUUID)).thenReturn(targetUser);
+
+        Island island1 = mock(Island.class);
+        Island island2 = mock(Island.class);
+        Map<String, Island> islandsMap = new HashMap<>();
+        islandsMap.put("IslandA", island1);
+        islandsMap.put("IslandB", island2);
+        doReturn(islandsMap).when(instance).getNameIslandMap(targetUser);
+
+        // Act
+        boolean canExecuteResult = instance.canExecute(user, label, args);
+        boolean executeResult = instance.execute(user, label, args);
+
+        // Assert
+        assertTrue(canExecuteResult);
+        assertTrue(executeResult);
+        verify(island1).setMaxHomes(4);
+        verify(island2).setMaxHomes(4);
+        verify(user).sendMessage("commands.admin.maxhomes.max-homes-set", TextVariables.NAME, "IslandA",
+                TextVariables.NUMBER, "4");
+        verify(user).sendMessage("commands.admin.maxhomes.max-homes-set", TextVariables.NAME, "IslandB",
+                TextVariables.NUMBER, "4");
+    }
+
+}