fix(tests): leak-free teardown for the disconnect-cancel runtime test

LSan (Linux amd64 leg): 200 bytes — the runtime client (192) plus its
IPC connection (8) — leaked whenever an earlier stage of
daemon_runtime_disconnect_cancels_blocked_non_index_child... failed:
both close blocks were gated on the request thread having completed,
so a failed run skipped the close entirely.

Teardown now closes unconditionally: close_begin interrupts the
transport (forcing the blocked call to complete), the join becomes
safe, and close_finish frees the client on every path.

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>

Author: DeusData

tests/test_daemon_runtime.c

@@ -3648,8 +3648,25 @@ TEST(daemon_runtime_disconnect_cancels_blocked_non_index_child_and_preserves_oth
     }
     bool exited =
         started && cbm_daemon_runtime_service_wait_exited(fixture.service, RUNTIME_TEST_TIMEOUT_MS);
-    if (first && !request_thread_started) {
-        (void)cbm_daemon_runtime_client_close(first, RUNTIME_TEST_TIMEOUT_MS);
+    /* Teardown must be leak-free on EVERY path, including the failure path
+     * where the request thread never completed: close_begin interrupts the
+     * transport, which forces the blocked call to finish, making the join
+     * safe; only close_finish releases the client. Skipping the close while
+     * the thread was still marked running leaked the client + its connection
+     * (LSan, 200 bytes) whenever an earlier stage of this test failed. */
+    if (first) {
+        if (!first_close_begun) {
+            first_close_begun = cbm_daemon_runtime_client_close_begin(first);
+        }
+        if (request_thread_started) {
+            (void)cbm_thread_join(&request_thread);
+            request_thread_started = false;
+        }
+        if (first_close_begun) {
+            (void)cbm_daemon_runtime_client_close_finish(first, RUNTIME_TEST_TIMEOUT_MS);
+        } else {
+            (void)cbm_daemon_runtime_client_close(first, RUNTIME_TEST_TIMEOUT_MS);
+        }
         first = NULL;
     }
     runtime_test_fixture_finish(&fixture);