Fix the select_all::smoke test

So that it matches the current behavior.
rust-lang · Apr 24, 2018 · 1a52eba · 1a52eba
1 parent f484fff
commit 1a52eba
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/futures/tests/select_all.rs b/futures/tests/select_all.rs
@@ -11,17 +11,20 @@ fn smoke() {
         ok(3),
     ];
 
+    // This test depends on an implementation detail of `select_all` to match
+    // its behavior: it uses `swap_remove` under the hood.
+
     let (i, idx, v) = block_on(select_all(v)).ok().unwrap();
     assert_eq!(i, 1);
     assert_eq!(idx, 0);
 
-    let (i, idx, v) = block_on(select_all(v)).err().unwrap();
-    assert_eq!(i, 2);
-    assert_eq!(idx, 0);
-
     let (i, idx, v) = block_on(select_all(v)).ok().unwrap();
     assert_eq!(i, 3);
     assert_eq!(idx, 0);
 
+    let (i, idx, v) = block_on(select_all(v)).err().unwrap();
+    assert_eq!(i, 2);
+    assert_eq!(idx, 0);
+
     assert!(v.is_empty());
 }