Test units added and improved

pom.xml

@@ -14,4 +14,23 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     </properties>
 
+    <dependencyManagement>
+        <dependencies>
+            <dependency>
+                <groupId>org.junit</groupId>
+                <artifactId>junit-bom</artifactId>
+                <version>5.8.2</version>
+                <type>pom</type>
+                <scope>import</scope>
+            </dependency>
+        </dependencies>
+    </dependencyManagement>
+
+    <dependencies>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter</artifactId>
+            <scope>test</scope>
+        </dependency>
+    </dependencies>
 </project>

src/test/java/Phonebook_Test.java

@@ -6,6 +6,146 @@
  * Hint: Your test coverage should be 100%
  * Attention:  We would check your commits and it will affect your score
  */
+import org.junit.jupiter.api.Test;
+import static org.junit.jupiter.api.Assertions.*;
 public class Phonebook_Test {
+    @Test
+    public void testConstructor() {
+        Person person = new Person("John Doe", "09123456789");
+        assertEquals("John Doe", person.getName());
+        assertEquals("09123456789", person.getPhone());
+        assertFalse(person.isHidden());
+    }
 
+    @Test
+    public void testGetName() {
+        Person person = new Person("John Doe", "09123456789");
+        assertEquals("John Doe", person.getName());
+    }
+
+    @Test
+    public void testGetPhone() {
+        Person person = new Person("John Doe", "09123456789");
+        assertEquals("09123456789", person.getPhone());
+    }
+
+    @Test
+    public void testGetId() {
+        Person person = new Person("John Doe", "09123456789");
+        assertEquals(3, person.getId());
+    }
+    @Test
+    public void testIsHidden() {
+        Person person = new Person("John Doe", "09123456789");
+        assertFalse(person.isHidden());
+    }
+
+    @Test
+    public void testSetName() {
+        Person person = new Person("John Doe", "09123456789");
+        person.setName("Jane Doe");
+        assertEquals("Jane Doe", person.getName());
+    }
+
+    @Test
+    public void testSetPhone() {
+        Person person = new Person("John Doe", "09123456789");
+        person.setPhone("09234567891");
+        assertEquals("09234567891", person.getPhone());
+
+    }
+    @Test
+    public void testSetHidden() {
+        Person person = new Person("John Doe", "09123456789");
+        person.setHidden();
+        assertTrue(person.isHidden());
+    }
+    @Test
+    public void testSetUnHidden() {
+        Person person = new Person("John Doe", "09123456789");
+        person.setHidden();
+        person.setUnHidden();
+        assertFalse(person.isHidden());
+    }
+    @Test
+    public void testToString() {
+        Person person   = new Person("John Doe", "09123456789");
+        assertEquals("Name: John Doe - Phone number: 09123456789", person.toString());
+    }
+    @Test
+    public void testAddContact() {
+        Phonebook phonebook = new Phonebook();
+        Person person = new Person("John Doe", "09123456789");
+        phonebook.addContact(person);
+        assertTrue(phonebook.getAllContacts().contains(person));
+    }
+    @Test
+    public void testGetPhoneNumber() {
+        Phonebook phonebook = new Phonebook();
+        Person person = new Person("John Doe", "09123456789");
+        phonebook.addContact(person);
+        assertEquals("09123456789", phonebook.getPhoneNumber("John Doe"));
+    }
+    @Test
+    public void testGetContact() {
+        Phonebook phonebook = new Phonebook();
+        Person person = new Person("John Doe", "09123456789");
+        phonebook.addContact(person);
+        assertEquals(1, phonebook.getContact("John Doe"));
+    }
+    @Test
+    public void testUpdateContactName() {
+        Phonebook phonebook = new Phonebook();
+        Person person = new Person("John Doe", "09123456789");
+        phonebook.addContact(person);
+        assertEquals(1, phonebook.updateContactName("John Doe", "Jane Doe"));
+        assertEquals("Jane Doe", phonebook.getAllContacts().get(0).getName());
+    }
+    @Test
+    public void testUpdateContactPhoneNumber() {
+        Phonebook phonebook = new Phonebook();
+        Person person = new Person("John Doe", "09123456789");
+        phonebook.addContact(person);
+        assertEquals(1, phonebook.updateContactPhoneNumber("John Doe", "09234567890"));
+        assertEquals("09234567890", phonebook.getAllContacts().get(0).getPhone());
+    }
+    @Test
+    public void testDeleteContact() {
+        Phonebook phonebook = new Phonebook();
+        Person person = new Person("John Doe", "09123456789");
+        phonebook.addContact(person);
+        assertEquals(1, phonebook.deleteContact("John Doe"));
+        assertFalse(phonebook.getAllContacts().contains(person));
+    }
+    @Test
+    public void testGetAllContacts() {
+        Phonebook phonebook = new Phonebook();
+        Person person1 = new Person("John Doe", "09123456789");
+        Person person2 = new Person("Jane Smith", "09234567890");
+        phonebook.addContact(person1);
+        phonebook.addContact(person2);
+        assertEquals(2, phonebook.getAllContacts().size());
+    }
+    @Test
+    public void testSetAllContactsHidden() {
+        Phonebook phonebook = new Phonebook();
+        Person person1 = new Person("John Doe", "09123456789");
+        Person person2 = new Person("Jane Smith", "09234567890");
+        phonebook.addContact(person1);
+        phonebook.addContact(person2);
+        phonebook.setAllContactsHidden();
+        assertTrue(person1.isHidden() && person2.isHidden());
+    }
+    @Test
+    public void testSetAllContactsUnHidden() {
+        Phonebook phonebook = new Phonebook();
+        Person person1 = new Person("John Doe", "09123456789");
+        Person person2 = new Person("Jane Smith", "09234567890");
+        phonebook.addContact(person1);
+        phonebook.addContact(person2);
+        phonebook.setAllContactsHidden();
+        phonebook.setAllContactsUnHidden();
+        assertFalse(person1.isHidden() && person2.isHidden());
+    }
 }
+