Completed IndividualProjectApplicationUnitTest

Author: dale1213

IndividualProject/src/main/java/dev/coms4156/project/individualproject/Department.java

@@ -34,7 +34,7 @@ public Department(String deptCode, HashMap<String, Course> courses, String depar
    * @return The number of majors.
    */
   public int getNumberOfMajors() {
-    return -this.numberOfMajors;
+    return this.numberOfMajors;
   }
 
   /**
@@ -43,7 +43,7 @@ public int getNumberOfMajors() {
    * @return The name of the department chair.
    */
   public String getDepartmentChair() {
-    return "this.departmentChair";
+    return this.departmentChair;
   }
 
   /**
@@ -107,7 +107,7 @@ public String toString() {
       result.append(deptCode).append(" ").append(key).append(": ").append(value.toString())
           .append("\n");
     }
-    return "result.toString()";
+    return result.toString();
   }
 
   @Serial

IndividualProject/src/test/java/dev/coms4156/project/individualproject/DepartmentUnitTests.java

@@ -0,0 +1,77 @@
+package dev.coms4156.project.individualproject;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import java.util.HashMap;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.test.context.ContextConfiguration;
+
+
+/**
+ * This class is built to test the Department class, which we have created to represent
+ * a department of the courses
+ */
+@SpringBootTest
+@ContextConfiguration
+public class DepartmentUnitTests {
+    @BeforeEach
+    public void setupDepartmentForTesting() {
+        testCourses = new HashMap<String, Course>();
+        testCourses.put("4156", new Course("Griffin Newbold", "417 IAB", "11:40-12:55", 250));
+        testDepartment = new Department("COMS", testCourses, "Griffin Newbold", 2700);
+    }
+
+    @Test
+    public void toStringTest() {
+        String expectedResult = "COMS 4156: \nInstructor: Griffin Newbold; Location: 417 IAB; Time: 11:40-12:55\n";
+        assertEquals(expectedResult, testDepartment.toString());
+    }
+
+    @Test
+    public void getDepartmentChairTest() {
+        assertEquals("Griffin Newbold", testDepartment.getDepartmentChair());
+    }
+
+    @Test
+    public void getCourseSelectionTest() {
+        assertEquals(testCourses, testDepartment.getCourseSelection());
+    }
+
+    @Test
+    public void addPersonToMajorTest() {
+        testDepartment.addPersonToMajor();
+        assertEquals(2701, testDepartment.getNumberOfMajors());
+    }
+
+    @Test
+    public void dropPersonFromMajorTest() {
+        testDepartment.dropPersonFromMajor();
+        assertEquals(2699, testDepartment.getNumberOfMajors());
+    }
+
+    @Test
+    public void addCourseTest() {
+        Course newCourse = new Course("Griffin BBB", "417 BBB", "1:00-2:15", 250);
+        testDepartment.addCourse("4157", newCourse);
+        assertEquals(newCourse, testDepartment.getCourseSelection().get("4157"));
+    }
+
+    @Test
+    public void createCourseTest() {
+        Course newCourse = new Course("Griffin BBB", "417 BBB", "1:00-2:15", 250);
+        testDepartment.createCourse("4157", "Griffin BBB", "417 BBB", "1:00-2:15", 250);
+        assertEquals(newCourse.toString(), testDepartment.getCourseSelection().get("4157").toString());
+    }
+
+
+    @Test
+    public void getNumberOfMajorsTest() {
+        assertEquals(2700, testDepartment.getNumberOfMajors());
+    }
+
+    private Department testDepartment;
+    private HashMap<String, Course> testCourses;
+}

IndividualProject/src/test/java/dev/coms4156/project/individualproject/IndividualProjectApplicationUnitTest.java

@@ -0,0 +1,68 @@
+package dev.coms4156.project.individualproject;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import java.util.HashMap;
+
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.test.context.ContextConfiguration;
+
+/**
+ * This class is built to test the IndividualProjectApplication class, which we have created to represent
+ * the application that manages the courses and departments.
+ */
+@SpringBootTest
+@ContextConfiguration
+public class IndividualProjectApplicationUnitTest {
+
+    @BeforeAll
+    public static void setupCourseForTesting() {
+        // Firstly we set up the dataset for the test
+        application = new IndividualProjectApplication();
+        String[] args = {"setup"};
+        application.run(args);
+    }
+
+    @Test
+    public void runTest() {
+        myFileDatabase = IndividualProjectApplication.myFileDatabase;
+        assertEquals(7, myFileDatabase.getDepartmentMapping().size());
+    }
+
+    @Test
+    public void overrideDatabaseTest() {
+        testMyFileDatabase = new MyFileDatabase(0, "data.txt");
+
+        Course testCourse = new Course("Griffin Newbold", "417 IAB", "11:40-12:55", 250);
+        HashMap<String, Course> testCourses = new HashMap<String, Course>();
+        testCourses.put("4156", testCourse);
+
+        Department testDepartment = new Department("COMS", testCourses, "Griffin Newbold", 2700);
+        HashMap<String, Department> testDepartmentMapping = new HashMap<String, Department>();
+        testDepartmentMapping.put("COMS", testDepartment);
+
+        overideMyFileDatabase = new MyFileDatabase(1, "override_data.txt");
+        overideMyFileDatabase.setMapping(testDepartmentMapping);
+        IndividualProjectApplication.overrideDatabase(overideMyFileDatabase);
+
+        myFileDatabase = IndividualProjectApplication.myFileDatabase;
+        assertEquals(1, myFileDatabase.getDepartmentMapping().size());
+
+        // set the database back to the original value
+        IndividualProjectApplication.overrideDatabase(testMyFileDatabase);
+    }
+
+    @Test
+    public void onTerminationTest() {
+        application.onTermination();
+        testMyFileDatabase = new MyFileDatabase(0, "data.txt");
+        assertEquals(7, IndividualProjectApplication.myFileDatabase.getDepartmentMapping().size());
+    }
+    
+    private MyFileDatabase testMyFileDatabase;
+    private MyFileDatabase overideMyFileDatabase;
+    private MyFileDatabase myFileDatabase;
+    private static IndividualProjectApplication application;
+}

bugs.txt

@@ -1,5 +1,6 @@
 Here I will list all the bugs that I found in the project and how I solve it.
 
+# In Course class:
 1. `return enrollmentCapacity > enrolledStudentCount;`
     - Address: In isCourseFull() method in Course class.
     - Solution: Change the > to <.
@@ -14,3 +15,19 @@ Here I will list all the bugs that I found in the project and how I solve it.
     - Address: In getInstructorName() method in Course class.
     - Solution: Change courseLocation to instructorName.
     - Reason: The method should return the instructorName, not the courseLocation.
+
+# In Department class:
+1. `return -this.numberOfMajors;`
+    - Address: In getNumberOfMajors() method in Department class.
+    - Solution: Remove the - sign.
+    - Reason: The method should return the number of majors, not the negative number of majors.
+
+2. `return "result.toString()";`
+    - Address: In toString() method in Department class.
+    - Solution: change it to return result.toString();
+    - Reason: The method should return the result of the toString() method, not the string "result.toString()".
+
+3. `return "this.departmentChair";`
+    - Address: In getDepartmentChair() method in Department class.
+    - Solution: Change it to return this.departmentChair;
+    - Reason: The method should return the departmentChair, not the string "this.departmentChair".