Skip to content

Commit eb89d24

Browse files
committed
Revert import changes in tests
After rust-lang#2216, these redundant imports are unneeded. This reverts almost all of rust-lang#2101.
1 parent 0afeda1 commit eb89d24

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

43 files changed

+958
-1481
lines changed

futures/tests/abortable.rs

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
1+
use futures::channel::oneshot;
2+
use futures::executor::block_on;
3+
use futures::future::{abortable, Aborted, FutureExt};
4+
use futures::task::{Context, Poll};
5+
use futures_test::task::new_count_waker;
6+
17
#[test]
28
fn abortable_works() {
3-
use futures::channel::oneshot;
4-
use futures::future::{abortable, Aborted};
5-
use futures::executor::block_on;
6-
79
let (_tx, a_rx) = oneshot::channel::<()>();
810
let (abortable_rx, abort_handle) = abortable(a_rx);
911

@@ -13,11 +15,6 @@ fn abortable_works() {
1315

1416
#[test]
1517
fn abortable_awakens() {
16-
use futures::channel::oneshot;
17-
use futures::future::{abortable, Aborted, FutureExt};
18-
use futures::task::{Context, Poll};
19-
use futures_test::task::new_count_waker;
20-
2118
let (_tx, a_rx) = oneshot::channel::<()>();
2219
let (mut abortable_rx, abort_handle) = abortable(a_rx);
2320

@@ -33,9 +30,6 @@ fn abortable_awakens() {
3330

3431
#[test]
3532
fn abortable_resolves() {
36-
use futures::channel::oneshot;
37-
use futures::future::abortable;
38-
use futures::executor::block_on;
3933
let (tx, a_rx) = oneshot::channel::<()>();
4034
let (abortable_rx, _abort_handle) = abortable(a_rx);
4135

futures/tests/arc_wake.rs

Lines changed: 49 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -1,69 +1,64 @@
1-
mod countingwaker {
2-
use futures::task::{self, ArcWake, Waker};
3-
use std::sync::{Arc, Mutex};
1+
use futures::task::{self, ArcWake, Waker};
2+
use std::sync::{Arc, Mutex};
43

5-
struct CountingWaker {
6-
nr_wake: Mutex<i32>,
7-
}
4+
struct CountingWaker {
5+
nr_wake: Mutex<i32>,
6+
}
87

9-
impl CountingWaker {
10-
fn new() -> Self {
11-
Self {
12-
nr_wake: Mutex::new(0),
13-
}
8+
impl CountingWaker {
9+
fn new() -> Self {
10+
Self {
11+
nr_wake: Mutex::new(0),
1412
}
13+
}
1514

16-
fn wakes(&self) -> i32 {
17-
*self.nr_wake.lock().unwrap()
18-
}
15+
fn wakes(&self) -> i32 {
16+
*self.nr_wake.lock().unwrap()
1917
}
18+
}
2019

21-
impl ArcWake for CountingWaker {
22-
fn wake_by_ref(arc_self: &Arc<Self>) {
23-
let mut lock = arc_self.nr_wake.lock().unwrap();
24-
*lock += 1;
25-
}
20+
impl ArcWake for CountingWaker {
21+
fn wake_by_ref(arc_self: &Arc<Self>) {
22+
let mut lock = arc_self.nr_wake.lock().unwrap();
23+
*lock += 1;
2624
}
25+
}
2726

28-
#[test]
29-
fn create_from_arc() {
30-
let some_w = Arc::new(CountingWaker::new());
27+
#[test]
28+
fn create_from_arc() {
29+
let some_w = Arc::new(CountingWaker::new());
3130

32-
let w1: Waker = task::waker(some_w.clone());
33-
assert_eq!(2, Arc::strong_count(&some_w));
34-
w1.wake_by_ref();
35-
assert_eq!(1, some_w.wakes());
31+
let w1: Waker = task::waker(some_w.clone());
32+
assert_eq!(2, Arc::strong_count(&some_w));
33+
w1.wake_by_ref();
34+
assert_eq!(1, some_w.wakes());
3635

37-
let w2 = w1.clone();
38-
assert_eq!(3, Arc::strong_count(&some_w));
36+
let w2 = w1.clone();
37+
assert_eq!(3, Arc::strong_count(&some_w));
3938

40-
w2.wake_by_ref();
41-
assert_eq!(2, some_w.wakes());
39+
w2.wake_by_ref();
40+
assert_eq!(2, some_w.wakes());
4241

43-
drop(w2);
44-
assert_eq!(2, Arc::strong_count(&some_w));
45-
drop(w1);
46-
assert_eq!(1, Arc::strong_count(&some_w));
47-
}
42+
drop(w2);
43+
assert_eq!(2, Arc::strong_count(&some_w));
44+
drop(w1);
45+
assert_eq!(1, Arc::strong_count(&some_w));
46+
}
4847

49-
#[test]
50-
fn ref_wake_same() {
51-
let some_w = Arc::new(CountingWaker::new());
48+
#[test]
49+
fn ref_wake_same() {
50+
let some_w = Arc::new(CountingWaker::new());
5251

53-
let w1: Waker = task::waker(some_w.clone());
54-
let w2 = task::waker_ref(&some_w);
55-
let w3 = w2.clone();
52+
let w1: Waker = task::waker(some_w.clone());
53+
let w2 = task::waker_ref(&some_w);
54+
let w3 = w2.clone();
5655

57-
assert!(w1.will_wake(&w2));
58-
assert!(w2.will_wake(&w3));
59-
}
56+
assert!(w1.will_wake(&w2));
57+
assert!(w2.will_wake(&w3));
6058
}
6159

6260
#[test]
6361
fn proper_refcount_on_wake_panic() {
64-
use futures::task::{self, ArcWake, Waker};
65-
use std::sync::Arc;
66-
6762
struct PanicWaker;
6863

6964
impl ArcWake for PanicWaker {
@@ -75,7 +70,13 @@ fn proper_refcount_on_wake_panic() {
7570
let some_w = Arc::new(PanicWaker);
7671

7772
let w1: Waker = task::waker(some_w.clone());
78-
assert_eq!("WAKE UP", *std::panic::catch_unwind(|| w1.wake_by_ref()).unwrap_err().downcast::<&str>().unwrap());
73+
assert_eq!(
74+
"WAKE UP",
75+
*std::panic::catch_unwind(|| w1.wake_by_ref())
76+
.unwrap_err()
77+
.downcast::<&str>()
78+
.unwrap()
79+
);
7980
assert_eq!(2, Arc::strong_count(&some_w)); // some_w + w1
8081
drop(w1);
8182
assert_eq!(1, Arc::strong_count(&some_w)); // some_w

0 commit comments

Comments
 (0)