testing/ostest: accept -ENOENT from work_cancel() for unqueued work

Since nuttx commit 5a209a853e ("sched/wqueue: restore -ENOENT from
work_cancel() for unqueued work") work_cancel() and work_cancel_wq()
return -ENOENT when the work is not queued. The tester thread queues
work with zero delay, so the worker may already have consumed it by the
time it is cancelled and the ASSERT(ret == OK) fires;

Accept -ENOENT there and expect it when cancelling idle work in
the API validation test.

Signed-off-by: raiden00pl <raiden00@railab.me>

Author: raiden00pl

testing/ostest/wqueue.c

@@ -768,10 +768,10 @@ static void api_validation_test(FAR struct kwork_wqueue_s *wqueue)
   ASSERT(work_cancel_sync_wq(wqueue, NULL) == -EINVAL);
   ASSERT(work_queue_priority_wq(NULL) == -EINVAL);
 
-  /* Cancelling idle work is intentionally idempotent. */
+  /* Cancelling idle work reports -ENOENT. */
 
-  ASSERT(work_cancel_wq(wqueue, &work) == OK);
-  ASSERT(work_cancel_sync_wq(wqueue, &work) == OK);
+  ASSERT(work_cancel_wq(wqueue, &work) == -ENOENT);
+  ASSERT(work_cancel_sync_wq(wqueue, &work) == -ENOENT);
   ASSERT(work_available(&work));
   printf("wqueue_test: API validation done\n");
 }
@@ -843,15 +843,15 @@ static FAR void *tester(FAR void *arg)
           ret = work_queue_wq(val[1], &work, empty_worker, NULL, 0);
           ASSERT(ret == OK);
           ret = work_cancel_wq(val[1], &work);
-          ASSERT(ret == OK);
+          ASSERT(ret == OK || ret == -ENOENT);
         }
       else
         {
           ret = work_queue((int)(uintptr_t)val[0], &work,
                            empty_worker, NULL, 0);
           ASSERT(ret == OK);
           ret = work_cancel((int)(uintptr_t)val[0], &work);
-          ASSERT(ret == OK);
+          ASSERT(ret == OK || ret == -ENOENT);
         }
 
       usleep((int)(uintptr_t)val[2]);