chore: fixed incomplete code as well made sure foramt was correct

Author: DavisD251

lesson_16/objects/objects_app/src/main/java/com/codedifferently/lesson16/onepiece/NotPirateException.java

@@ -4,31 +4,32 @@
 
 public class Pirate {
   public enum HakiType {
-    Observation,
-    Armament,
-    Conquerors,
-    Advanced_Observation,
-    Advanced_Armament,
-    Advanced_Conquerors,
-    Will_of_D;
+    OBSERVATION,
+    ARMANENT,
+    CONQUERORS,
+    ADVANCED_OBSERVATION,
+    ADVANCED_ARMANENT,
+    ADVANCED_CONQUERORS,
+    WILL_OF_D;
   }
 
   private String name;
   private String crew;
-  private long bounty;
-  private boolean hasDream;
-  private boolean isPirate;
+  private Long bounty;
+  private String role;
+  private Boolean hasDream;
   private HakiType powers;
+
   private final HakiType[] haki = HakiType.values();
   private static final Random cflip = new Random();
 
-  public Pirate(String name, String crew, long bounty) {
+  public Pirate(String name, String crew, Long bounty, String role, Boolean hasDream) {
     this.name = name;
     this.crew = crew;
     this.bounty = bounty;
+    this.role = role;
     this.hasDream = true;
-    this.isPirate = true;
-    this.powers = HakiType.Will_of_D;
+    this.powers = HakiType.WILL_OF_D;
   }
 
   // Getters and setters
@@ -44,12 +45,20 @@ public long getBounty() {
     return bounty;
   }
 
-  public boolean getHasDream() {
-    return hasDream;
+  public String getRole() {
+    return role;
+  }
+
+  public boolean getHasDream() throws HasNoDreamException {
+    if (!hasDream) {
+      throw new HasNoDreamException(name + " has no dream!");
+    }
+    System.out.println("Has Dream");
+    return true;
   }
 
-  public boolean getIsPirate() {
-    return isPirate;
+  public HakiType getPowers() {
+    return powers;
   }
 
   public void rollPowers() {
@@ -71,11 +80,11 @@ public void setBounty(Long bounty) {
     this.bounty = bounty;
   }
 
-  public void setHasDream(boolean hasDream) {
-    this.hasDream = hasDream;
+  public void setRole(String role) {
+    this.role = role;
   }
 
-  public void setIsPirate(boolean isPirate) {
-    this.isPirate = isPirate;
+  public void setHasDream(boolean hasDream) {
+    this.hasDream = hasDream;
   }
 }

lesson_16/objects/objects_app/src/main/java/com/codedifferently/lesson16/onepiece/Pirate.java

@@ -1,6 +1,12 @@
 package com.codedifferently.lesson16.onepiece;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import com.codedifferently.lesson16.onepiece.Pirate.HakiType;
+import java.io.ByteArrayOutputStream;
+import java.io.PrintStream;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -10,73 +16,112 @@ public class PirateTest {
 
   @BeforeEach
   void setUp() {
-    pirate = new Pirate(2, "Zoro", "Swordsman", 900000);
+    pirate = new Pirate("Luffy", "StrawHatPirates", 3000000000L, "Captain", true);
   }
 
   @Test
-  void testGetId() {
-    int actual = pirate.getId();
+  void testGetName() {
+    String actual = pirate.getName();
 
-    assertThat(actual).isEqualTo(2);
+    assertThat(actual).isEqualTo("Luffy");
   }
 
   @Test
-  void testSetId() {
-    pirate.setId(3);
-    int actual = pirate.getId();
+  void testSetName() {
+    pirate.setName("BlackBeard");
+    String actual = pirate.getName();
 
-    assertThat(actual).isEqualTo(3);
+    assertThat(actual).isEqualTo("BlackBeard");
   }
 
   @Test
-  void testGetName() {
-    String actual = pirate.getName();
+  void testGetCrew() {
+    String actual = pirate.getCrew();
 
-    assertThat(actual).isEqualTo("Zoro");
+    assertThat(actual).isEqualTo("StrawHatPirates");
   }
 
   @Test
-  void testSetName() {
-    pirate.setName("Sanji");
-    String actual = pirate.getName();
+  void testSetCrew() {
+    pirate.setCrew("BlackBeardPirates");
+    String actual = pirate.getCrew();
 
-    assertThat(actual).isEqualTo("Sanji");
+    assertThat(actual).isEqualTo("BlackBeardPirates");
   }
 
   @Test
-  void testGetDepartment() {
-    String actual = pirate.getDepartment();
+  void testGetBounty() {
+    Long actual = pirate.getBounty();
 
-    assertThat(actual).isEqualTo("Swordsman");
+    assertThat(actual).isEqualTo(3000000000L);
   }
 
   @Test
-  void testSetDepartment() {
-    pirate.setDepartment("Cook");
-    String actual = pirate.getDepartment();
+  void testSetBounty() {
+    pirate.setBounty(3996000000L);
+    Long actual = pirate.getBounty();
 
-    assertThat(actual).isEqualTo("Cook");
+    assertThat(actual).isEqualTo(3996000000L);
   }
 
   @Test
-  void testGetSalary() {
-    double actual = pirate.getSalary();
+  void testGetRole() {
+    String actual = pirate.getRole();
 
-    assertThat(actual).isEqualTo(900000);
+    assertThat(actual).isEqualTo("Captain");
   }
 
   @Test
-  void testSetSalary() {
-    pirate.setSalary(850000);
-    double actual = pirate.getSalary();
+  void testSetRole() {
+    pirate.setRole("Captain");
+    String actual = pirate.getRole();
 
-    assertThat(actual).isEqualTo(850000);
+    assertThat(actual).isEqualTo("Captain");
   }
 
   @Test
-  void testGetDetails() {
-    String actual = pirate.getDetails();
+  void testgetHasDream() {
+    try {
+      pirate.getHasDream();
+    } catch (HasNoDreamException e) {
+      System.out.println(e.getMessage());
+    }
+  }
+
+  @Test
+  void testRollPowers() {
+    pirate.rollPowers();
+    assertNotNull(pirate.getPowers(), "Powers should not be null after rolling");
+    boolean isValid = false;
+    for (HakiType h : HakiType.values()) {
+      if (pirate.getPowers() == h) {
+        isValid = true;
+        break;
+      }
+    }
+    assertTrue(isValid, "Powers should be a valid HakiType");
+  }
+
+  @Test
+  void testRollPowersOutput() {
+
+    // Capture console output
+    ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
+    System.setOut(new PrintStream(outputStream));
+
+    pirate.rollPowers();
+
+    String output = outputStream.toString().trim();
+    boolean matches = false;
+    for (HakiType h : HakiType.values()) {
+      if (output.equals("Random Haki: " + h)) {
+        matches = true;
+        break;
+      }
+    }
+
+    assertTrue(matches, "Haki Options");
 
-    assertThat(actual).isEqualTo("2 Zoro Swordsman 900000.0");
+    System.setOut(System.out);
   }
 }