AY2122S1-CS2103T-F12-2 / tp

https://ay2122s1-cs2103t-f12-2.github.io/tp/
MIT License
1 stars 5 forks source link

Fix pe-d bug for Course and Job. #204

Closed Jeromesyl closed 3 years ago

Jeromesyl commented 3 years ago

Bug where users can give blank inputs for Course and Job fields. Fixes #174, #175, #186, #187

codecov-commenter commented 3 years ago

Codecov Report

Merging #204 (fe77053) into master (14010b8) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #204   +/-   ##
=========================================
  Coverage     70.64%   70.64%           
  Complexity      589      589           
=========================================
  Files            85       85           
  Lines          1918     1918           
  Branches        259      259           
=========================================
  Hits           1355     1355           
  Misses          445      445           
  Partials        118      118           
Impacted Files Coverage Δ
...main/java/seedu/intern/model/applicant/Course.java 70.00% <ø> (ø)
...rc/main/java/seedu/intern/model/applicant/Job.java 70.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 14010b8...fe77053. Read the comment docs.