From d2637471beb459cf4af39406da120ca6fa87a5c9 Mon Sep 17 00:00:00 2001
From: Aman Bhandal <aman_bhandal@live.ca>
Date: Sun, 25 Feb 2018 13:44:32 -0500
Subject: [PATCH 1/2] [TEST] Added tests for Person, BaseEntity, and
 NamedEntity

---
 .../petclinic/model/BaseEntityTest.java       | 30 +++++++++++++++++++
 .../petclinic/model/NamedEntityTest.java      | 26 ++++++++++++++++
 .../samples/petclinic/model/PersonTest.java   | 29 ++++++++++++++++++
 3 files changed, 85 insertions(+)
 create mode 100644 src/test/java/org/springframework/samples/petclinic/model/BaseEntityTest.java
 create mode 100644 src/test/java/org/springframework/samples/petclinic/model/NamedEntityTest.java
 create mode 100644 src/test/java/org/springframework/samples/petclinic/model/PersonTest.java

diff --git a/src/test/java/org/springframework/samples/petclinic/model/BaseEntityTest.java b/src/test/java/org/springframework/samples/petclinic/model/BaseEntityTest.java
new file mode 100644
index 000000000..2dcbd8525
--- /dev/null
+++ b/src/test/java/org/springframework/samples/petclinic/model/BaseEntityTest.java
@@ -0,0 +1,30 @@
+package org.springframework.samples.petclinic.model;
+
+import org.junit.Before;
+import org.junit.Test;
+
+import static org.junit.Assert.*;
+
+public class BaseEntityTest {
+
+    private BaseEntity baseEntityInstance;
+
+    @Before
+    public void setUp(){
+        baseEntityInstance = new BaseEntity();
+    }
+
+    // Expects the baseEntity to have Integer of 100
+    @Test
+    public void testGetAndSetId() {
+        baseEntityInstance.setId(100);
+        assertEquals((Integer)100, baseEntityInstance.getId());
+    }
+
+    // Expects the baseEntity ID to be null
+    @Test
+    public void TestIsNew() {
+        baseEntityInstance.isNew();
+        assertNull(baseEntityInstance.getId());
+    }
+}
diff --git a/src/test/java/org/springframework/samples/petclinic/model/NamedEntityTest.java b/src/test/java/org/springframework/samples/petclinic/model/NamedEntityTest.java
new file mode 100644
index 000000000..32315a5fb
--- /dev/null
+++ b/src/test/java/org/springframework/samples/petclinic/model/NamedEntityTest.java
@@ -0,0 +1,26 @@
+package org.springframework.samples.petclinic.model;
+
+import org.junit.Before;
+import org.junit.Test;
+
+import javax.naming.Name;
+
+import static org.junit.Assert.*;
+
+public class NamedEntityTest {
+
+    private NamedEntity namedObj;
+
+    @Before
+    public void setUp(){
+        namedObj = new NamedEntity();
+    }
+
+    // Testing the DAO of NamedEntity class by verifying the setter, getter and toString of Name
+    @Test
+    public void testSetAndGetName() {
+        namedObj.setName("Junior The Senior");
+        assertEquals("Junior The Senior", namedObj.getName());
+        assertEquals("Junior The Senior", namedObj.toString());
+    }
+}
diff --git a/src/test/java/org/springframework/samples/petclinic/model/PersonTest.java b/src/test/java/org/springframework/samples/petclinic/model/PersonTest.java
new file mode 100644
index 000000000..af8f7ac2b
--- /dev/null
+++ b/src/test/java/org/springframework/samples/petclinic/model/PersonTest.java
@@ -0,0 +1,29 @@
+package org.springframework.samples.petclinic.model;
+
+import org.junit.Before;
+import org.junit.Test;
+
+import static org.junit.Assert.*;
+
+public class PersonTest {
+
+    private Person personObj;
+
+    @Before
+    public void setUp(){
+        personObj = new Person();
+    }
+
+    // Testing the DAO of Person class by verifying the setter and getter of FirstName
+    @Test
+    public void testSetAndGetFirstName() {
+        personObj.setFirstName("Johnny");
+    }
+
+    // Testing the DAO of Person class by verifying the setter and getter of FirstName
+    @Test
+    public void testSetAndGetLastName() {
+        personObj.setLastName("Oliver");
+        assertEquals("Oliver", personObj.getLastName());
+    }
+}

From aa595b12ba34f82d9fc94ebd12360b8eff7b9743 Mon Sep 17 00:00:00 2001
From: Aman Bhandal <aman_bhandal@live.ca>
Date: Sun, 25 Feb 2018 14:26:58 -0500
Subject: [PATCH 2/2] [TEST] Added missing assertEqual for getFirstName

---
 .../org/springframework/samples/petclinic/model/PersonTest.java  | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/test/java/org/springframework/samples/petclinic/model/PersonTest.java b/src/test/java/org/springframework/samples/petclinic/model/PersonTest.java
index af8f7ac2b..8f98ea77f 100644
--- a/src/test/java/org/springframework/samples/petclinic/model/PersonTest.java
+++ b/src/test/java/org/springframework/samples/petclinic/model/PersonTest.java
@@ -18,6 +18,7 @@ public class PersonTest {
     @Test
     public void testSetAndGetFirstName() {
         personObj.setFirstName("Johnny");
+        assertEquals("Johnny", personObj.getFirstName());
     }
 
     // Testing the DAO of Person class by verifying the setter and getter of FirstName