testing + refactoring

student-managment-app/data-model/services/build.gradle.kts

-/*
- * This file was generated by the Gradle 'init' task.
- *
- * This project uses @Incubating APIs which are subject to change.
- */
-
-plugins {
-    id("student.managment.app.java-library-conventions")
-}

student-managment-app/data-model/services/src/test/java/student/managment/app/list/LinkedListTest.java

-/*
- * This Java source file was generated by the Gradle 'init' task.
- */
-package student.managment.app.services;
-
-import org.junit.jupiter.api.Test;
-
-import static org.junit.jupiter.api.Assertions.*;
-
-class LinkedListTest {
-    @Test void testConstructor() {
-        LinkedList list = new LinkedList();
-        assertEquals(0, list.size());
-    }
-
-    @Test void testAdd() {
-        LinkedList list = new LinkedList();
-
-        list.add("one");
-        assertEquals(1, list.size());
-        assertEquals("one", list.get(0));
-
-        list.add("two");
-        assertEquals(2, list.size());
-        assertEquals("two", list.get(1));
-    }
-
-    @Test void testRemove() {
-        LinkedList list = new LinkedList();
-
-        list.add("one");
-        list.add("two");
-        assertTrue(list.remove("one"));
-
-        assertEquals(1, list.size());
-        assertEquals("two", list.get(0));
-
-        assertTrue(list.remove("two"));
-        assertEquals(0, list.size());
-    }
-
-    @Test public void testRemoveMissing() {
-        LinkedList list = new LinkedList();
-
-        list.add("one");
-        list.add("two");
-        assertFalse(list.remove("three"));
-        assertEquals(2, list.size());
-    }
-}

student-managment-app/data-model/src/main/java/student/managment/app/entities/Grade.java

+package student.managment.app.entities;
+
+public class Grade {
+    private String courseName;
+    private double grade;
+
+    public Grade() {
+
+    }
+
+    public Grade(String courseName, double grade) {
+        this.courseName = courseName;
+        this.grade = grade;
+    }
+
+    public String getCourseName() {
+        return courseName;
+    }
+
+    public void setCourseName(String courseName) {
+        this.courseName = courseName;
+    }
+
+    public double getGrade() {
+        return grade;
+    }
+
+    public void setGrade(double grade) {
+        this.grade = grade;
+    }
+}

student-managment-app/data-model/src/main/java/student/managment/app/entities/Student.java

 package student.managment.app.entities;
 
-import java.math.BigDecimal;
-import java.math.RoundingMode;
 import java.util.*;
 
 public class Student {
@@ -47,6 +45,4 @@ public class Student {
     public void addCourse(Course course) {
         this.courses.add(course);
     }
-
-
 }

student-managment-app/repositories/src/main/java/student/managment/app/repositories/StudentRepositoryImpl.java

@@ -20,10 +20,6 @@ public class StudentRepositoryImpl implements StudentRepository {
 
     @Override
     public void add(Student student) {
-//        if (students.containsKey(student.getId())) {
-//            System.out.println("Student with given ID already exists");
-//            return;
-//        }
         this.students.put(student.getId(), student);
     }
 

student-managment-app/services/src/main/java/student/managment/app/services/CourseService.java

@@ -150,4 +150,4 @@ public class CourseService {
                     gradeWithStudents.forEach((grade, students) -> students.forEach(s -> System.out.printf("->      %s %.2f \n",s.getName(), grade)));
                 });
     }
-}
\ No newline at end of file
+}

student-managment-app/services/src/main/java/student/managment/app/services/StudentService.java

@@ -51,4 +51,3 @@ public class StudentService {
     }
 }
 
-

student-managment-app/services/src/test/java/student/managment/app/services/CourseServiceTest.java

 package student.managment.app.services;
 
 import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import student.managment.app.entities.Course;
 import student.managment.app.entities.Teacher;
@@ -17,6 +19,16 @@ public class CourseServiceTest {
         this.teacherService = new TeacherService();
     }
 
+    @BeforeAll
+    public static void beforeAll() {
+
+    }
+
+    @BeforeEach
+    public void beforeEach() {
+
+    }
+
     @Test
     public void whenAddingNewCourseShouldIncreaseSize() {
         this.courseService.addCourse(new Course(1L, "mathematics", 12));