test: fix multi threads unstable tests (#215)

* test: fix multi threads unstable tests Signed-off-by: Zhenchi <zhongzc_arch@outlook.com> * chore: add new contributors Signed-off-by: Zhenchi <zhongzc_arch@outlook.com> --------- Signed-off-by: Zhenchi <zhongzc_arch@outlook.com>
tikv · May 6, 2024 · b987f64 · b987f64
1 parent 9dc5c99
commit b987f64
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/CONTRIBUTORS b/CONTRIBUTORS
@@ -9,8 +9,10 @@ Jiang Yinzuo
 Jonathan Chuang
 Lewis Hallam
 Mark Van de Vyver
+morristai
 piercetrey-figure
 Renkai Ge
+rrichardson
 Stephen Cirner
 shota kizawa
 TennyZhuang

diff --git a/minitrace/tests/lib.rs b/minitrace/tests/lib.rs
@@ -154,15 +154,20 @@ fn multiple_threads_single_span() {
         let root = Span::root("root", SpanContext::random());
         let _g = root.set_local_parent();
 
+        let mut handles = vec![];
+
         for _ in 0..4 {
             let child_span = Span::enter_with_local_parent("cross-thread");
-            scope.spawn(move |_| {
+            let h = scope.spawn(move |_| {
                 let _g = child_span.set_local_parent();
                 four_spans();
             });
+            handles.push(h);
         }
 
         four_spans();
+
+        handles.into_iter().for_each(|h| h.join().unwrap());
     })
     .unwrap();
 
@@ -212,22 +217,28 @@ fn multiple_threads_multiple_spans() {
         let root2 = Span::root("root2", SpanContext::new(TraceId(13), SpanId::default()));
         let local_collector = LocalCollector::start();
 
+        let mut handles = vec![];
+
         for _ in 0..4 {
             let merged = Span::enter_with_parents("merged", vec![&root1, &root2]);
             let _g = merged.set_local_parent();
             let _local = LocalSpan::enter_with_local_parent("local");
-            scope.spawn(move |_| {
+            let h = scope.spawn(move |_| {
                 let local_collector = LocalCollector::start();
 
                 four_spans();
 
                 let local_spans = local_collector.collect();
                 merged.push_child_spans(local_spans);
             });
+
+            handles.push(h);
         }
 
         four_spans();
 
+        handles.into_iter().for_each(|h| h.join().unwrap());
+
         let local_spans = local_collector.collect();
         root1.push_child_spans(local_spans.clone());
         root2.push_child_spans(local_spans);