From 345bcbc2e11bbf47b2807944811d3ab91793a47d Mon Sep 17 00:00:00 2001
From: dupontbertrand <dupontbertrand59@gmail.com>
Date: Fri, 20 Mar 2026 10:29:29 +0100
Subject: [PATCH] tests: use waitUntil instead of sleep for async onStop tests

---
 packages/ddp-server/livedata_server_tests.js | 24 ++++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)

diff --git a/packages/ddp-server/livedata_server_tests.js b/packages/ddp-server/livedata_server_tests.js
index 6b66c6bc67..dbb49c63ee 100644
--- a/packages/ddp-server/livedata_server_tests.js
+++ b/packages/ddp-server/livedata_server_tests.js
@@ -617,10 +617,18 @@ Tinytest.addAsync(
 
     const { clientConn } = await getTestConnections(test);
     const sub = clientConn.subscribe('test_async_onstop_cleanup', trackerId);
-    await sleep(100);
+
+    await waitUntil(
+      () => sub.ready(),
+      { description: 'subscription is ready' }
+    );
 
     sub.stop();
-    await sleep(200);
+
+    await waitUntil(
+      () => asyncCleanupTracker[trackerId] === true,
+      { description: 'async onStop callback completed after unsubscribe' }
+    );
 
     test.isTrue(
       asyncCleanupTracker[trackerId],
@@ -640,10 +648,18 @@ Tinytest.addAsync(
 
     const { clientConn } = await getTestConnections(test);
     clientConn.subscribe('test_async_onstop_cleanup', trackerId);
-    await sleep(100);
+
+    await waitUntil(
+      () => clientConn.status().connected,
+      { description: 'client is connected' }
+    );
 
     clientConn.disconnect();
-    await sleep(300);
+
+    await waitUntil(
+      () => asyncCleanupTracker[trackerId] === true,
+      { description: 'async onStop callback completed after disconnect' }
+    );
 
     test.isTrue(
       asyncCleanupTracker[trackerId],