feat: modified tests

Author: htanweer244

lesson_16/objects/objects_app/src/main/java/com/codedifferently/lesson16/HummadTanweer/HobbyLimitExceededException.java

@@ -1,4 +1,5 @@
 package com.codedifferently.lesson16.HummadTanweer;
+
 import java.util.ArrayList;
 
 public class Person {
@@ -9,6 +10,12 @@ public class Person {
   private final String email;
   private final int MAX_HOBBIES = 2;
 
+  public class HobbyLimitExceededException extends Exception {
+    public HobbyLimitExceededException(String message) {
+      super(message);
+    }
+  }
+
   enum Gender {
     MALE,
     FEMALE,
@@ -23,28 +30,17 @@ public Person(String name, int age, Gender gender, String email) {
     this.hobbies = new ArrayList<>();
   }
 
-public void addHobby(String hobby) throws HobbyLimitExceededException {
+  public void addHobby(String hobby) throws HobbyLimitExceededException {
     if (hobbies.size() >= MAX_HOBBIES) {
-        throw new HobbyLimitExceededException("Cannot add more than " + MAX_HOBBIES + " hobbies.");
+      throw new HobbyLimitExceededException("Cannot add more than " + MAX_HOBBIES + " hobbies.");
     }
     hobbies.add(hobby);
-}
+  }
 
   public ArrayList<String> getHobbies() {
     return hobbies;
   }
 
-  public void displayHobbies() {
-    if (hobbies.isEmpty()) {
-      System.out.println(name + " has no hobbies.");
-    } else {
-      System.out.println(name + "'s hobbies:");
-      for (String hobby : hobbies) {
-        System.out.println("- " + hobby);
-      }
-    }
-  }
-
   public boolean isAdult() {
     return age >= 18;
   }
@@ -53,31 +49,7 @@ public String getName() {
     return name;
   }
 
-  public Gender getGender() {
-    return gender;
-  }
-
   public String getEmail() {
     return email;
   }
-
-  public static void main(String[] args) {
-    Person person = new Person("xyz", 29, Gender.MALE, "[email protected]");
-
-    try {
-      person.addHobby("Football");
-      person.addHobby("Pickleball");
-      person.addHobby("Basketball");
-    } catch (HobbyLimitExceededException e) {
-      System.out.println(e.getMessage());
-    }
-
-    person.displayHobbies();
-
-    if (person.isAdult()) {
-      System.out.println(person.getName() + " Adult");
-    } else {
-      System.out.println(person.getName() + " Not an adult");
-    }
-  }
 }

lesson_16/objects/objects_app/src/main/java/com/codedifferently/lesson16/HummadTanweer/Person.java

@@ -4,43 +4,55 @@
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assertions.fail;
-import org.junit.jupiter.api.Test;
 
 import com.codedifferently.lesson16.HummadTanweer.Person.Gender;
+import com.codedifferently.lesson16.HummadTanweer.Person.HobbyLimitExceededException;
+import org.junit.jupiter.api.Test;
 
 public class PersonTest {
   @Test
   void testIsAdult() {
     Person adult = new Person("xyz", 29, Gender.MALE, "[email protected]");
     assertTrue(adult.isAdult(), "True for 29");
-
     Person minor = new Person("abc", 15, Gender.FEMALE, "[email protected]");
     assertFalse(minor.isAdult(), "False for 15");
   }
 
-    @Test
-    void testAddHobby() throws HobbyLimitExceededException {
-        Person person = new Person("xyz", 29, Gender.MALE, "[email protected]");
-        
-        person.addHobby("Football");
-        person.addHobby("Pickleball");
-        
-        assertEquals(2, person.getHobbies().size());
-        assertTrue(person.getHobbies().contains("Football"));
-        assertTrue(person.getHobbies().contains("Pickleball"));
-    }
+  @Test
+  void testAddHobby() throws HobbyLimitExceededException {
+    Person person = new Person("xyz", 29, Gender.MALE, "[email protected]");
 
-    @Test
-    void testAddHobbyExceedsLimit() throws HobbyLimitExceededException {
-        Person person = new Person("xyz", 29, Gender.MALE, "[email protected]");
-    
-        try {
-         person.addHobby("Football");
-         person.addHobby("Pickleball");
-         person.addHobby("Basketball");
-         fail("Expected HobbyLimitExceededException to be thrown");
-        } catch (HobbyLimitExceededException e) {
-            assertEquals("Cannot add more than 2 hobbies.", e.getMessage());
-        }
+    person.addHobby("Football");
+    person.addHobby("Pickleball");
+
+    assertEquals(2, person.getHobbies().size());
+    assertTrue(person.getHobbies().contains("Football"));
+    assertTrue(person.getHobbies().contains("Pickleball"));
+  }
+
+  @Test
+  public void testGetName() {
+    Person person = new Person("xyz", 29, Gender.MALE, "[email protected]");
+    assertEquals("xyz", person.getName());
+  }
+
+  @Test
+  public void getEmail() {
+    Person person = new Person("xyz", 29, Gender.MALE, "[email protected]");
+    assertEquals("[email protected]", person.getEmail());
+  }
+
+  @Test
+  void testAddHobbyExceedsLimit() throws HobbyLimitExceededException {
+    Person person = new Person("xyz", 29, Gender.MALE, "[email protected]");
+
+    try {
+      person.addHobby("Football");
+      person.addHobby("Pickleball");
+      person.addHobby("Basketball");
+      fail("Expected HobbyLimitExceededException to be thrown");
+    } catch (HobbyLimitExceededException e) {
+      assertEquals("Cannot add more than 2 hobbies.", e.getMessage());
     }
+  }
 }