search

tpaviot / ProcessScheduler

A Python package for automatic and optimized resource scheduling
https://processscheduler.github.io/
GNU General Public License v3.0
59 stars 18 forks source link

Added get_scheduled_tasks to SchedulingSolution #32

Closed dreinon closed 3 years ago

tpaviot commented 3 years ago

Thanks for the contribution @dreinon can you please also push a related unittest? just to be sure the new method works as expected and to prevent a coverage issue. Maybe an additionnal test to https://github.com/tpaviot/ProcessScheduler/blob/master/test/test_solver.py

dreinon commented 3 years ago

Sure! I didn't make any tests because I didn't find tests for get_all_tasks_but_unavailable. Is there any?

tpaviot commented 3 years ago

I'm not sure there is a specific test for the get_all_tasks_but_unavailable method, but it is actually covered by the unittest suite, according to the coverage.io report, see https://codecov.io/gh/tpaviot/ProcessScheduler/src/master/processscheduler/solution.py