Closed khsrali closed 3 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 79.32%. Comparing base (
a147f87
) to head (deb4593
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Cheers @agoscinski, I applied/answered your suggestions
Changes include:
tests/conftest.py
MockFirecrest
Slurm.all_jobs
cancel()
for killing jobspoll_active
whenjob_id
is not found, raiseFirecrestException
just like firecrest does. --this includes when the job is finished.conftest.Values
tests/test_calculation.py::test_calculation_basic
tests/test_scheduler.py
transport.kill_jobs
test_scheduler.py
tests pass against server