Changeset eebecdc in mainline for uspace/lib/system/test/system.c

Timestamp:

2025-03-13T18:30:36Z (2 months ago)

Author:

Miroslav Cimerman <mc@…>

Children:

e3e53cc

Parents:

e494d7b (diff), da54714 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

merge upstream/master into helenraid

File:

• uspace/lib/system/test/system.c (modified) (9 diffs)

uspace/lib/system/test/system.c

@@ -1 +1 @@
 /*
- * Copyright (c) 2024 Jiri Svoboda
+ * Copyright (c) 2025 Jiri Svoboda
  * All rights reserved.
  *
@@ -46 +46 @@
 void test_system_conn(ipc_call_t *, void *);
 
-static errno_t test_shutdown(void *);
+static errno_t test_poweroff(void *);
+static errno_t test_restart(void *);
 
 static void test_sys_shutdown_complete(void *);
@@ -52 +53 @@
 
 static system_ops_t test_system_srv_ops = {
-        .shutdown = test_shutdown
+        .poweroff = test_poweroff,
+        .restart = test_restart
 };
 
@@ -66 +68 @@
         errno_t rc;
 
-        bool shutdown_called;
+        bool poweroff_called;
+        bool restart_called;
         bool shutdown_complete_called;
         bool shutdown_failed_called;
@@ -103 +106 @@
 }
 
-/** system_shutdown() with server returning error response works */
-PCUT_TEST(shutdown_failure)
+/** system_poweroff() with server returning error response works */
+PCUT_TEST(poweroff_failure)
 {
         errno_t rc;
@@ -126 +129 @@
 
         resp.rc = ENOMEM;
-        resp.shutdown_called = false;
-
-        rc = system_shutdown(system);
-        PCUT_ASSERT_TRUE(resp.shutdown_called);
+        resp.poweroff_called = false;
+
+        rc = system_poweroff(system);
+        PCUT_ASSERT_TRUE(resp.poweroff_called);
         PCUT_ASSERT_ERRNO_VAL(resp.rc, rc);
 
@@ -138 +141 @@
 }
 
-/** system_shutdown() with server returning success response works */
-PCUT_TEST(shutdown_success)
+/** system_poweroff() with server returning success response works */
+PCUT_TEST(poweroff_success)
 {
         errno_t rc;
@@ -161 +164 @@
 
         resp.rc = EOK;
-        resp.shutdown_called = false;
-
-        rc = system_shutdown(system);
-        PCUT_ASSERT_TRUE(resp.shutdown_called);
+        resp.poweroff_called = false;
+
+        rc = system_poweroff(system);
+        PCUT_ASSERT_TRUE(resp.poweroff_called);
+        PCUT_ASSERT_ERRNO_VAL(resp.rc, rc);
+
+        system_close(system);
+        rc = loc_service_unregister(srv, sid);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+        loc_server_unregister(srv);
+}
+
+/** system_restart() with server returning error response works */
+PCUT_TEST(restart_failure)
+{
+        errno_t rc;
+        service_id_t sid;
+        system_t *system = NULL;
+        test_response_t resp;
+        loc_srv_t *srv;
+
+        async_set_fallback_port_handler(test_system_conn, &resp);
+
+        // FIXME This causes this test to be non-reentrant!
+        rc = loc_server_register(test_system_server, &srv);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = loc_service_register(srv, test_system_svc, &sid);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = system_open(test_system_svc, NULL, NULL, &system);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+        PCUT_ASSERT_NOT_NULL(system);
+
+        resp.rc = ENOMEM;
+        resp.restart_called = false;
+
+        rc = system_restart(system);
+        PCUT_ASSERT_TRUE(resp.restart_called);
+        PCUT_ASSERT_ERRNO_VAL(resp.rc, rc);
+
+        system_close(system);
+        rc = loc_service_unregister(srv, sid);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+        loc_server_unregister(srv);
+}
+
+/** system_restart() with server returning success response works */
+PCUT_TEST(restart_success)
+{
+        errno_t rc;
+        service_id_t sid;
+        system_t *system = NULL;
+        test_response_t resp;
+        loc_srv_t *srv;
+
+        async_set_fallback_port_handler(test_system_conn, &resp);
+
+        // FIXME This causes this test to be non-reentrant!
+        rc = loc_server_register(test_system_server, &srv);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = loc_service_register(srv, test_system_svc, &sid);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        rc = system_open(test_system_svc, NULL, NULL, &system);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+        PCUT_ASSERT_NOT_NULL(system);
+
+        resp.rc = EOK;
+        resp.restart_called = false;
+
+        rc = system_restart(system);
+        PCUT_ASSERT_TRUE(resp.restart_called);
         PCUT_ASSERT_ERRNO_VAL(resp.rc, rc);
 
@@ -279 +352 @@
 }
 
-/** Test system shutdown.
+/** Test system poweroff.
  *
  * @param arg Argument (test_response_t *)
  */
-static errno_t test_shutdown(void *arg)
+static errno_t test_poweroff(void *arg)
 {
         test_response_t *resp = (test_response_t *)arg;
 
-        resp->shutdown_called = true;
+        resp->poweroff_called = true;
+        return resp->rc;
+}
+
+/** Test system restart.
+ *
+ * @param arg Argument (test_response_t *)
+ */
+static errno_t test_restart(void *arg)
+{
+        test_response_t *resp = (test_response_t *)arg;
+
+        resp->restart_called = true;
         return resp->rc;
 }