switched to SeqCst atomic ordering in tests (unsure if Relaxed is

correct)
oddity-ai · Mar 21, 2024 · f2c2db8 · f2c2db8
1 parent 9a0b8e9
commit f2c2db8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/runtime/future.rs b/src/runtime/future.rs
@@ -398,14 +398,14 @@ mod tests {
         Future::new({
             let first_future_completed = first_future_completed.clone();
             move || {
-                first_future_completed.store(true, Ordering::Relaxed);
+                first_future_completed.store(true, Ordering::SeqCst);
             }
         })
         .await;
         assert!(
             Future::new({
                 let first_future_completed = first_future_completed.clone();
-                move || first_future_completed.load(Ordering::Relaxed)
+                move || first_future_completed.load(Ordering::SeqCst)
             })
             .await
         );

diff --git a/src/runtime/work.rs b/src/runtime/work.rs
@@ -40,10 +40,10 @@ mod tests {
         let work = Work::new({
             let make_me_true = make_me_true.clone();
             move || {
-                make_me_true.store(true, std::sync::atomic::Ordering::Relaxed);
+                make_me_true.store(true, std::sync::atomic::Ordering::SeqCst);
             }
         });
         work.run();
-        assert!(make_me_true.load(std::sync::atomic::Ordering::Relaxed));
+        assert!(make_me_true.load(std::sync::atomic::Ordering::SeqCst));
     }
 }