diff options
author | Joseph Hunkeler <jhunkeler@gmail.com> | 2024-09-17 10:29:13 -0400 |
---|---|---|
committer | Joseph Hunkeler <jhunkeler@gmail.com> | 2024-09-18 23:08:03 -0400 |
commit | 3ef3a2758c572dcdeed7d693f52c5049394f11f5 (patch) | |
tree | 7e0df913c83a421fbb7203bb707c5b7018bafa37 /tests/test_multiprocessing.c | |
parent | 60c0a3c8607b835f73e77d89b5f67a8e6bd5c8b3 (diff) | |
download | stasis-3ef3a2758c572dcdeed7d693f52c5049394f11f5.tar.gz |
Fix test status expectation
* Fix child not returning result of execvp(). task->status is for program status, not fork() status.
Diffstat (limited to 'tests/test_multiprocessing.c')
-rw-r--r-- | tests/test_multiprocessing.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/test_multiprocessing.c b/tests/test_multiprocessing.c index 3d1d972..67574e3 100644 --- a/tests/test_multiprocessing.c +++ b/tests/test_multiprocessing.c @@ -38,7 +38,7 @@ void test_mp_task() { STASIS_ASSERT_FATAL((task = mp_task(pool, task_name, commands[i])) != NULL, "Task should not be NULL"); STASIS_ASSERT(task->pid != 0, "PID should be non-zero at this point"); STASIS_ASSERT(task->parent_pid != MP_POOL_PID_UNUSED, "Parent PID should be non-zero"); - STASIS_ASSERT(task->status == 0, "Status should be zero"); + STASIS_ASSERT(task->status == -1, "Status should be -1 (not started yet)"); STASIS_ASSERT(strcmp(task->ident, task_name) == 0, "Wrong task identity"); STASIS_ASSERT(strstr(task->log_file, pool->log_root) != NULL, "Log file path must be in log_root"); STASIS_ASSERT(task->gate != NULL, "Semaphore should be initialized"); @@ -51,7 +51,7 @@ void test_mp_pool_join() { for (size_t i = 0; i < pool->num_used; i++) { struct MultiProcessingTask *task = &pool->task[i]; STASIS_ASSERT(task->pid == MP_POOL_PID_UNUSED, "Task should be marked as unused"); - STASIS_ASSERT(task->status == 0, "Task should have succeeded"); + STASIS_ASSERT(task->status == 0, "Task status should be zero (success)"); } } |