Development/workerpool (#1746)

* [Tests/unit/core] : Fix build 'test_workerpool'.

'ThreadPool' and 'WorkerPool' changes were not (fully) integrated.

* [Tests/unit/core] : Amend 'a65dbb2adf2e9622aae8825c0d88c599a4a252e6'.

- Add 1 to occupation to compensate for the 'internal' timer.

- Reverse the boolean test for calculating occupation.

* [Tests/unit/core] : test for less or equal on timings in 'test_workerpool'.

Clock timings are never exact due to resolution, rounding and scheduling. Assume a Job executed earlier is allowed and preferred.

* [Tests/unit/core] : Disable instant job completion for 'test_workerpool'.

Time resolution and scheduling may prevent instant completion.

* [Tests/unit/core] : disable 'zero' delta times tests in 'test_workerpool'

* Revert "[Tests/unit/core] : disable 'zero' delta times tests in 'test_workerpool'"

This reverts commit b6c300f.

* [Tests/unit/core] : disable 'zero' delta times tests in 'test_workerpool'

---------

Co-authored-by: Pierre Wielders <[email protected]>

Tests/unit/core/test_workerpool.cpp

@@ -675,7 +675,7 @@ namespace Core {
         const uint16_t scheduledTimes[] = {2000}; //In milliseconds
         CheckWorkerPool_ScheduleJobs(1, 1, 0, 0, nullptr, scheduledTimes);
     }
-    TEST(Core_WorkerPool, Check_ScheduleJobs_SinglePool_SingleJob_ZeroTime_DISABLED)
+    TEST(Core_WorkerPool, DISABLED_Check_ScheduleJobs_SinglePool_SingleJob_ZeroTime)
     {
         const uint16_t scheduledTimes[] = {0};
         CheckWorkerPool_ScheduleJobs(1, 1, 0, 0, nullptr, scheduledTimes);
@@ -703,7 +703,7 @@ namespace Core {
         const uint16_t scheduledTimes[] = {2000, 3000, 1000, 2000, 3000};
         CheckWorkerPool_ScheduleJobs(1, maxJobs, 0, 0, nullptr, scheduledTimes);
     }
-    TEST(Core_WorkerPool, Check_ScheduleJobs_SinglePool_MultipleJobs_ZeroTime_DISABLED)
+    TEST(Core_WorkerPool, DISABLED_Check_ScheduleJobs_SinglePool_MultipleJobs_ZeroTime)
     {
         uint8_t maxJobs = 5;
         const uint16_t scheduledTimes[5] = {0};
@@ -724,7 +724,7 @@ namespace Core {
         CheckWorkerPool_ScheduleJobs(2, maxJobs, 0, 0, nullptr, scheduledTimes);
         CheckWorkerPool_ScheduleJobs(2, 2, 0, 0, nullptr, scheduledTimes);
     }
-    TEST(Core_WorkerPool, Check_ScheduleJobs_MultiplePool_MultipleJobs_ZeroTime_DISABLED)
+    TEST(Core_WorkerPool, DISABLED_Check_ScheduleJobs_MultiplePool_MultipleJobs_ZeroTime)
     {
         uint8_t maxJobs = 5;
         const uint16_t scheduledTimes[5] = {0};
@@ -829,7 +829,7 @@ namespace Core {
         const uint16_t rescheduledTimes[] = {2000};
         CheckWorkerPool_RescheduleJobs(1, 1, 0, 0, nullptr, scheduledTimes, rescheduledTimes);
     }
-    TEST(Core_WorkerPool, Check_RescheduleJobs_SinglePool_SingleJob_ZeroTime_DISABLED)
+    TEST(Core_WorkerPool, DISABLED_Check_RescheduleJobs_SinglePool_SingleJob_ZeroTime)
     {
         const uint16_t scheduledTimes[] = {1000};
         const uint16_t rescheduledTimes[] = {0};
@@ -863,7 +863,7 @@ namespace Core {
         const uint16_t rescheduledTimes[] = {2000, 2000, 1000, 2000, 3000};
         CheckWorkerPool_RescheduleJobs(1, 5, 0, 0, nullptr, scheduledTimes, rescheduledTimes);
     }
-    TEST(Core_WorkerPool, Check_RescheduleJobs_SinglePool_MultipleJobs_ZeroTime_DISABLED)
+    TEST(Core_WorkerPool, DISABLED_Check_RescheduleJobs_SinglePool_MultipleJobs_ZeroTime)
     {
         const uint16_t scheduledTimes[] = {1000, 2000, 3000, 2000, 1000};
         const uint16_t rescheduledTimes[5] = {0};
@@ -899,7 +899,7 @@ namespace Core {
         const uint16_t rescheduledTimes[] = {2000, 2000, 1000, 2000, 3000};
         CheckWorkerPool_RescheduleJobs(1, 5, 0, 0, nullptr, scheduledTimes, rescheduledTimes);
     }
-    TEST(Core_WorkerPool, Check_RescheduleJobs_MultiplePool_MultipleJobs_ZeroTime_DISABLED)
+    TEST(Core_WorkerPool, DISABLED_Check_RescheduleJobs_MultiplePool_MultipleJobs_ZeroTime)
     {
         const uint16_t scheduledTimes[] = {1000, 2000, 3000, 2000, 1000};
         const uint16_t rescheduledTimes[5] = {0};