fix(s2n-quic-dc): handle spurious TCP acceptor worker wakeups (#2434)

dc/s2n-quic-dc/src/stream/server/tokio/tcp/manager.rs

@@ -254,6 +254,13 @@ where
         let mut cf = ControlFlow::Continue(());
 
         let entry = &mut self.workers[idx];
+
+        // Only poll the worker if it's active. We can end up here if we've pruned a worker right before
+        // the tokio runtime notifies us the stream is ready.
+        if !entry.worker.is_active() {
+            return cf;
+        }
+
         let mut task_cx = task::Context::from_waker(&entry.waker);
         let Poll::Ready(res) = entry.worker.poll(&mut task_cx, cx, publisher, clock) else {
             debug_assert!(entry.worker.is_active());

dc/s2n-quic-dc/src/stream/server/tokio/tcp/manager/tests.rs

@@ -171,9 +171,8 @@ impl Harness {
         let Entry { worker, waker, .. } = &mut self.manager.inner.workers[idx];
         let is_active = worker.is_active();
 
-        if is_active {
-            waker.wake_by_ref();
-        }
+        // wake the worker even if it's not active so we can test more paths
+        waker.wake_by_ref();
 
         is_active
     }