From 0dad719083165570b86c1c7cfba55ac669599c02 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 7 Apr 2026 10:08:53 +0000 Subject: [PATCH 1/3] Initial plan From 8e5c2aff545a41a73cc84cf99de41fb2f6a6bffb Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 7 Apr 2026 10:13:12 +0000 Subject: [PATCH 2/3] Improve Title assertions in CheckPhaseTest using ArgumentCaptor Agent-Logs-Url: https://github.com/BentoBoxWorld/AOneBlock/sessions/e4ba8fa5-e639-4f8e-ae85-ae03b1d9c5ff Co-authored-by: tastybento <4407265+tastybento@users.noreply.github.com> --- .../aoneblock/listeners/CheckPhaseTest.java | 26 +++++++++++++------ 1 file changed, 18 insertions(+), 8 deletions(-) diff --git a/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java b/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java index 71720df..25bb4ad 100644 --- a/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java +++ b/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java @@ -20,8 +20,10 @@ import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; +import org.mockito.ArgumentCaptor; import org.mockito.Mock; +import net.kyori.adventure.text.Component; import net.kyori.adventure.title.Title; import world.bentobox.aoneblock.AOneBlock; @@ -152,8 +154,10 @@ void testSetNewPhase() { verify(previous).getFirstTimeEndCommands(); // Verify phase name change assertEquals("Next Phase", is.getPhaseName()); - // Verify title shown - verify(mockPlayer).showTitle(any(Title.class)); + // Verify title shown with correct phase name + ArgumentCaptor titleCaptor = ArgumentCaptor.forClass(Title.class); + verify(mockPlayer).showTitle(titleCaptor.capture()); + assertEquals(Component.text("Next Phase"), titleCaptor.getValue().title()); } @@ -185,8 +189,10 @@ void testSetNewPhaseSecondTime() { verify(previous, never()).getFirstTimeEndCommands(); // Verify phase name change assertEquals("Next Phase", is.getPhaseName()); - // Verify title shown - verify(mockPlayer).showTitle(any(Title.class)); + // Verify title shown with correct phase name + ArgumentCaptor<Title> titleCaptor2 = ArgumentCaptor.forClass(Title.class); + verify(mockPlayer).showTitle(titleCaptor2.capture()); + assertEquals(Component.text("Next Phase"), titleCaptor2.getValue().title()); } @@ -218,8 +224,10 @@ void testSetNewPhaseNullPlayer() { verify(previous).getFirstTimeEndCommands(); // Verify phase name change assertEquals("Next Phase", is.getPhaseName()); - // Verify title shown - verify(mockPlayer).showTitle(any(Title.class)); + // Verify title shown with correct phase name + ArgumentCaptor<Title> titleCaptor3 = ArgumentCaptor.forClass(Title.class); + verify(mockPlayer).showTitle(titleCaptor3.capture()); + assertEquals(Component.text("Next Phase"), titleCaptor3.getValue().title()); } @@ -251,8 +259,10 @@ void testCheckPhaseNPCPlayer() { verify(previous).getFirstTimeEndCommands(); // Verify phase name change assertEquals("Next Phase", is.getPhaseName()); - // Verify title shown - verify(mockPlayer).showTitle(any(Title.class)); + // Verify title shown with correct phase name + ArgumentCaptor<Title> titleCaptor4 = ArgumentCaptor.forClass(Title.class); + verify(mockPlayer).showTitle(titleCaptor4.capture()); + assertEquals(Component.text("Next Phase"), titleCaptor4.getValue().title()); } From 8bac957d29f7f27a15a7efc5d507ff40809c2e84 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 7 Apr 2026 10:15:14 +0000 Subject: [PATCH 3/3] Rename numbered titleCaptor variables for consistency Agent-Logs-Url: https://github.com/BentoBoxWorld/AOneBlock/sessions/e4ba8fa5-e639-4f8e-ae85-ae03b1d9c5ff Co-authored-by: tastybento <4407265+tastybento@users.noreply.github.com> --- .../aoneblock/listeners/CheckPhaseTest.java | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java b/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java index 25bb4ad..6b2ae3f 100644 --- a/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java +++ b/src/test/java/world/bentobox/aoneblock/listeners/CheckPhaseTest.java @@ -190,9 +190,9 @@ void testSetNewPhaseSecondTime() { // Verify phase name change assertEquals("Next Phase", is.getPhaseName()); // Verify title shown with correct phase name - ArgumentCaptor<Title> titleCaptor2 = ArgumentCaptor.forClass(Title.class); - verify(mockPlayer).showTitle(titleCaptor2.capture()); - assertEquals(Component.text("Next Phase"), titleCaptor2.getValue().title()); + ArgumentCaptor<Title> titleCaptor = ArgumentCaptor.forClass(Title.class); + verify(mockPlayer).showTitle(titleCaptor.capture()); + assertEquals(Component.text("Next Phase"), titleCaptor.getValue().title()); } @@ -225,9 +225,9 @@ void testSetNewPhaseNullPlayer() { // Verify phase name change assertEquals("Next Phase", is.getPhaseName()); // Verify title shown with correct phase name - ArgumentCaptor<Title> titleCaptor3 = ArgumentCaptor.forClass(Title.class); - verify(mockPlayer).showTitle(titleCaptor3.capture()); - assertEquals(Component.text("Next Phase"), titleCaptor3.getValue().title()); + ArgumentCaptor<Title> titleCaptor = ArgumentCaptor.forClass(Title.class); + verify(mockPlayer).showTitle(titleCaptor.capture()); + assertEquals(Component.text("Next Phase"), titleCaptor.getValue().title()); } @@ -260,9 +260,9 @@ void testCheckPhaseNPCPlayer() { // Verify phase name change assertEquals("Next Phase", is.getPhaseName()); // Verify title shown with correct phase name - ArgumentCaptor<Title> titleCaptor4 = ArgumentCaptor.forClass(Title.class); - verify(mockPlayer).showTitle(titleCaptor4.capture()); - assertEquals(Component.text("Next Phase"), titleCaptor4.getValue().title()); + ArgumentCaptor<Title> titleCaptor = ArgumentCaptor.forClass(Title.class); + verify(mockPlayer).showTitle(titleCaptor.capture()); + assertEquals(Component.text("Next Phase"), titleCaptor.getValue().title()); }