Expand test coverage with new listener and placeholder tests

Adds PlaceholdersManagerTest (8 tests) and AdvancementListenerTest
(25 tests) for previously untested classes, plus 6 branch-coverage
tests in AdvancementsManagerTest for getScore, addAdvancement, and
checkIslandSize edge cases. Full suite: 112 tests passing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: tastybento

src/test/java/world/bentobox/boxed/AdvancementsManagerTest.java

@@ -293,4 +293,69 @@ public void testGetScoreAdvancement() {
         assertEquals(9, am.getScore(advancement));
     }
 
+    /**
+     * Test method for {@link world.bentobox.boxed.AdvancementsManager#getScore(org.bukkit.advancement.Advancement)}.
+     * Root advancements fall back to settings.default-root-increase (0 in the shipped config).
+     */
+    @Test
+    public void testGetScoreAdvancementRoot() {
+        when(advancement.getKey()).thenReturn(NamespacedKey.fromString("story/root"));
+        assertEquals(0, am.getScore(advancement));
+    }
+
+    /**
+     * Test method for {@link world.bentobox.boxed.AdvancementsManager#getScore(org.bukkit.advancement.Advancement)}.
+     * Recipe advancements always score settings.unknown-recipe-increase (0 in the shipped config).
+     */
+    @Test
+    public void testGetScoreAdvancementRecipe() {
+        when(advancement.getKey()).thenReturn(NamespacedKey.fromString("recipes/brewing/blaze_powder"));
+        assertEquals(0, am.getScore(advancement));
+    }
+
+    /**
+     * Test method for {@link world.bentobox.boxed.AdvancementsManager#addAdvancement(org.bukkit.entity.Player, org.bukkit.advancement.Advancement)}.
+     * No island for this player means no expansion and a zero score.
+     */
+    @Test
+    public void testAddAdvancementPlayerAdvancementNullIsland() {
+        when(im.getIsland(world, player.getUniqueId())).thenReturn(null);
+        assertEquals(0, am.addAdvancement(player, advancement));
+        verify(island, never()).setProtectionRange(org.mockito.ArgumentMatchers.anyInt());
+    }
+
+    /**
+     * Test method for {@link world.bentobox.boxed.AdvancementsManager#addAdvancement(org.bukkit.entity.Player, org.bukkit.advancement.Advancement)}.
+     * Visitors (rank below MEMBER_RANK) cannot expand the island.
+     */
+    @Test
+    public void testAddAdvancementPlayerAdvancementVisitorRank() {
+        when(island.getRank(player.getUniqueId())).thenReturn(RanksManager.VISITOR_RANK);
+        assertEquals(0, am.addAdvancement(player, advancement));
+        verify(island, never()).setProtectionRange(org.mockito.ArgumentMatchers.anyInt());
+    }
+
+    /**
+     * Test method for {@link world.bentobox.boxed.AdvancementsManager#addAdvancement(org.bukkit.entity.Player, org.bukkit.advancement.Advancement)}.
+     * An advancement that's already been recorded on the island cannot grant a second expansion.
+     */
+    @Test
+    public void testAddAdvancementPlayerAdvancementAlreadyHas() {
+        // Seed the island with the exact same namespaced key the manager will try to record.
+        am.addAdvancement(island, advancement.getKey().toString());
+        assertEquals(0, am.addAdvancement(player, advancement));
+    }
+
+    /**
+     * Test method for {@link world.bentobox.boxed.AdvancementsManager#checkIslandSize(world.bentobox.bentobox.database.objects.Island)}.
+     * Positive diff case: one scoring advancement grows a size-1 island to size 10.
+     */
+    @Test
+    public void testCheckIslandSizePositiveDiff() {
+        when(island.getProtectionRange()).thenReturn(1);
+        am.addAdvancement(island, "adventure/honey_block_slide");
+        assertEquals(9, am.checkIslandSize(island));
+        verify(island).setProtectionRange(10);
+    }
+
 }

src/test/java/world/bentobox/boxed/PlaceholdersManagerTest.java

@@ -0,0 +1,108 @@
+package world.bentobox.boxed;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.mockito.Mockito.when;
+
+import java.util.List;
+import java.util.Optional;
+import java.util.UUID;
+
+import org.bukkit.Location;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.mockito.Mock;
+
+import world.bentobox.bentobox.api.user.User;
+import world.bentobox.boxed.objects.IslandAdvancements;
+
+/**
+ * @author tastybento
+ */
+public class PlaceholdersManagerTest extends CommonTestSetup {
+
+    @Mock
+    private Boxed addon;
+    @Mock
+    private AdvancementsManager advManager;
+    @Mock
+    private IslandAdvancements islandAdv;
+    @Mock
+    private User user;
+    @Mock
+    private Location userLocation;
+
+    private PlaceholdersManager phm;
+    private UUID uuid;
+
+    @Override
+    @BeforeEach
+    public void setUp() throws Exception {
+        super.setUp();
+
+        uuid = UUID.randomUUID();
+        when(user.getUniqueId()).thenReturn(uuid);
+        when(user.getLocation()).thenReturn(userLocation);
+
+        when(addon.getIslands()).thenReturn(im);
+        when(addon.getOverWorld()).thenReturn(world);
+        when(addon.getAdvManager()).thenReturn(advManager);
+
+        when(advManager.getIsland(island)).thenReturn(islandAdv);
+        when(islandAdv.getAdvancements()).thenReturn(List.of("a", "b", "c"));
+
+        phm = new PlaceholdersManager(addon);
+    }
+
+    @Override
+    @AfterEach
+    public void tearDown() throws Exception {
+        super.tearDown();
+    }
+
+    @Test
+    public void testGetCountNullUser() {
+        assertEquals("", phm.getCount(null));
+    }
+
+    @Test
+    public void testGetCountNullUuid() {
+        when(user.getUniqueId()).thenReturn(null);
+        assertEquals("", phm.getCount(user));
+    }
+
+    @Test
+    public void testGetCountNoIsland() {
+        when(im.getIsland(world, user)).thenReturn(null);
+        assertEquals("", phm.getCount(user));
+    }
+
+    @Test
+    public void testGetCountReturnsAdvancementCount() {
+        when(im.getIsland(world, user)).thenReturn(island);
+        assertEquals("3", phm.getCount(user));
+    }
+
+    @Test
+    public void testGetCountByLocationNullUser() {
+        assertEquals("", phm.getCountByLocation(null));
+    }
+
+    @Test
+    public void testGetCountByLocationNullLocation() {
+        when(user.getLocation()).thenReturn(null);
+        assertEquals("", phm.getCountByLocation(user));
+    }
+
+    @Test
+    public void testGetCountByLocationNoIslandAtLocation() {
+        when(im.getIslandAt(userLocation)).thenReturn(Optional.empty());
+        assertEquals("", phm.getCountByLocation(user));
+    }
+
+    @Test
+    public void testGetCountByLocationReturnsAdvancementCount() {
+        when(im.getIslandAt(userLocation)).thenReturn(Optional.of(island));
+        assertEquals("3", phm.getCountByLocation(user));
+    }
+}