From e8e05aa176be4babf5c28a7697e2d057077d2794 Mon Sep 17 00:00:00 2001 From: sowgro Date: Sun, 9 Nov 2025 01:06:16 -0500 Subject: make id system consistent --- src/test/java/design/model/GolferTest.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'src/test/java') diff --git a/src/test/java/design/model/GolferTest.java b/src/test/java/design/model/GolferTest.java index 4dce705..3a79070 100644 --- a/src/test/java/design/model/GolferTest.java +++ b/src/test/java/design/model/GolferTest.java @@ -103,7 +103,8 @@ public class GolferTest { { Golfer testGolfer = new Golfer("John Doe", "jdoesgolf2", "weback4321"); - testGolfer.addClub("John Doe", "The Slammer", ClubType.DRIVER); + Club c = new Club("John Doe", "The Slammer", ClubType.DRIVER); + testGolfer.addClub(c); Club addedClub = testGolfer.getClubs()[0]; assertTrue(testGolfer.hasClub(addedClub)); assertTrue(testGolfer.hasClubs()); @@ -114,7 +115,8 @@ public class GolferTest { { Golfer testGolfer = new Golfer("John Doe", "jdoesgolf2", "weback4321"); - testGolfer.addClub("John Doe", "The Slammer", ClubType.DRIVER); + Club c = new Club("John Doe", "The Slammer", ClubType.DRIVER); + testGolfer.addClub(c); Club addedClub = testGolfer.getClubs()[0]; testGolfer.removeClub(addedClub); assertFalse(testGolfer.hasClub(addedClub)); -- cgit v1.2.3 From 5ad9bc936062c870778709fd6ac997a9098c4b96 Mon Sep 17 00:00:00 2001 From: sowgro Date: Mon, 10 Nov 2025 13:29:50 -0500 Subject: fix some tests --- src/test/java/design/model/ClubTest.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'src/test/java') diff --git a/src/test/java/design/model/ClubTest.java b/src/test/java/design/model/ClubTest.java index 9cbfa87..9ac8130 100644 --- a/src/test/java/design/model/ClubTest.java +++ b/src/test/java/design/model/ClubTest.java @@ -15,7 +15,7 @@ public class ClubTest { @Test void testConstructor() { - Club testClub = new Club(0, "John Doe", "The Slammer", ClubType.DRIVER); + Club testClub = new Club("John Doe", "The Slammer", ClubType.DRIVER); assertEquals(0, testClub.getId()); assertEquals("John Doe", testClub.getManufacture()); @@ -26,35 +26,35 @@ public class ClubTest { @Test void testGetClubType() { - Club testClub = new Club(0, "John Doe", "The Slammer", ClubType.DRIVER); + Club testClub = new Club("John Doe", "The Slammer", ClubType.DRIVER); assertEquals(ClubType.DRIVER, testClub.getClubType()); } @Test void testGetNickname() { - Club testClub = new Club(0, "John Doe", "The Slammer", ClubType.DRIVER); + Club testClub = new Club("John Doe", "The Slammer", ClubType.DRIVER); assertEquals("The Slammer", testClub.getNickname()); } @Test void testGetManufacture() { - Club testClub = new Club(0, "John Doe", "The Slammer", ClubType.DRIVER); + Club testClub = new Club("John Doe", "The Slammer", ClubType.DRIVER); assertEquals("John Doe", testClub.getManufacture()); } @Test void testGetId() { - Club testClub = new Club(0, "John Doe", "The Slammer", ClubType.DRIVER); - assertEquals(0, testClub.getId()); + Club testClub = new Club("John Doe", "The Slammer", ClubType.DRIVER); + assertEquals(-1, testClub.getId()); } @Test void testToString() { - Club testClub = new Club(0, "John Doe", "The Slammer", ClubType.DRIVER); + Club testClub = new Club("John Doe", "The Slammer", ClubType.DRIVER); String expectedString = "#0 The Slammer - John Doe (DRIVER)"; assertEquals(expectedString, testClub.toString()); } -- cgit v1.2.3