From 447957ee4e256e6182dbaf461fa40f17c40eca6e Mon Sep 17 00:00:00 2001 From: Julius Pfrommer Date: Wed, 14 Oct 2026 15:27:58 +0200 Subject: [PATCH] test(server): expect session cleanup during shutdown --- tests/server/check_session.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/server/check_session.c b/tests/server/check_session.c index 56ad60638..1cbf880dc 100644 --- a/tests/server/check_session.c +++ b/tests/server/check_session.c @@ -150,7 +150,7 @@ observeDeleteCloseSession(UA_Server *server_, UA_AccessControl *ac, originalDeleteCloseSession(server_, ac, sessionId, sessionContext); } -START_TEST(Session_deleteStoppedServerCleansSessionSynchronously) { +START_TEST(Session_shutdownCleansSessionBeforeStopped) { UA_Client *client = UA_Client_newForUnitTest(); ck_assert_uint_eq(UA_Client_connect(client, "opc.tcp://localhost:4840"), UA_STATUSCODE_GOOD); @@ -162,8 +162,9 @@ START_TEST(Session_deleteStoppedServerCleansSessionSynchronously) { UA_atomic_store(&running, false); THREAD_JOIN(server_thread); - ck_assert_uint_eq(UA_Server_run_shutdown(server), UA_STATUSCODE_GOOD); ck_assert_uint_eq(deleteCloseSessionCalls, 0); + ck_assert_uint_eq(UA_Server_run_shutdown(server), UA_STATUSCODE_GOOD); + ck_assert_uint_eq(deleteCloseSessionCalls, 1); ck_assert_uint_eq(UA_Server_delete(server), UA_STATUSCODE_GOOD); server = NULL; ck_assert_uint_eq(deleteCloseSessionCalls, 1); @@ -1028,7 +1029,7 @@ static Suite* testSuite_Session(void) { tcase_add_test(tc_session, Session_updateLifetime_ShallWork); tcase_add_test(tc_session, Session_notificationCallback); tcase_add_test(tc_session, - Session_deleteStoppedServerCleansSessionSynchronously); + Session_shutdownCleansSessionBeforeStopped); tcase_add_test(tc_session, Session_serviceBeginCallbackClosesCurrentSession); tcase_add_test(tc_session, Session_accessControlCloseSessionIsReentrant); tcase_add_test(tc_session, Session_createdNotificationCloseReturnsSessionClosed);