Check if coach actually exists

[TomAlard]

Closes #91

[codecov]

Codecov Report

Merging #103 (0b68234) into development (f56be2f) will increase coverage by 0.53%.
The diff coverage is 71.42%.

@@                Coverage Diff                @@
##             development     #103      +/-   ##
=================================================
+ Coverage          91.16%   91.69%   +0.53%     
- Complexity           114      115       +1     
=================================================
  Files                 21       21              
  Lines                249      253       +4     
  Branches              16       17       +1     
=================================================
+ Hits                 227      232       +5     
  Misses                13       13              
+ Partials               9        8       -1     

Flag	Coverage Δ
all_tests	91.69% <71.42%> (+0.53%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...n/be/osoc/team1/backend/services/StudentService.kt	91.89% <71.42%> (+0.98%)	⬆️
...main/kotlin/be/osoc/team1/backend/entities/User.kt	100.00% <0.00%> (+8.33%)	⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[lars-vc]

Will the coach entity get deleted after the merge? I am afraid it will keep existing so maybe you can also delete it on your branch

[TomAlard]

Will the coach entity get deleted after the merge? I am afraid it will keep existing so maybe you can also delete it on your branch

I merged the dev branch into this one, so the Coach entity is gone now.