Skip to content

Commit 4911264

Browse files
committed
fix channel tests
Signed-off-by: Yoshua Wuyts <[email protected]>
1 parent 7248e08 commit 4911264

File tree

1 file changed

+20
-20
lines changed

1 file changed

+20
-20
lines changed

tests/channel.rs

+20-20
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@ fn smoke() {
1818
let (s, r) = channel(1);
1919

2020
s.send(7).await;
21-
assert_eq!(r.recv().await, Some(7));
21+
assert_eq!(r.recv().await.unwrap(), 7);
2222

2323
s.send(8).await;
24-
assert_eq!(r.recv().await, Some(8));
24+
assert_eq!(r.recv().await.unwrap(), 8);
2525

2626
drop(s);
27-
assert_eq!(r.recv().await, None);
27+
assert!(r.recv().await.is_err());
2828
});
2929

3030
task::block_on(async {
@@ -74,7 +74,7 @@ fn len_empty_full() {
7474
assert_eq!(r.is_empty(), false);
7575
assert_eq!(r.is_full(), true);
7676

77-
r.recv().await;
77+
let _ = r.recv().await;
7878

7979
assert_eq!(s.len(), 1);
8080
assert_eq!(s.is_empty(), false);
@@ -91,12 +91,12 @@ fn recv() {
9191
let (s, r) = channel(100);
9292

9393
task::spawn(async move {
94-
assert_eq!(r.recv().await, Some(7));
94+
assert_eq!(r.recv().await.unwrap(), 7);
9595
task::sleep(ms(1000)).await;
96-
assert_eq!(r.recv().await, Some(8));
96+
assert_eq!(r.recv().await.unwrap(), 8);
9797
task::sleep(ms(1000)).await;
98-
assert_eq!(r.recv().await, Some(9));
99-
assert_eq!(r.recv().await, None);
98+
assert_eq!(r.recv().await.unwrap(), 9);
99+
assert!(r.recv().await.is_err());
100100
});
101101

102102
task::sleep(ms(1500)).await;
@@ -122,9 +122,9 @@ fn send() {
122122
});
123123

124124
task::sleep(ms(1500)).await;
125-
assert_eq!(r.recv().await, Some(7));
126-
assert_eq!(r.recv().await, Some(8));
127-
assert_eq!(r.recv().await, Some(9));
125+
assert_eq!(r.recv().await.unwrap(), 7);
126+
assert_eq!(r.recv().await.unwrap(), 8);
127+
assert_eq!(r.recv().await.unwrap(), 9);
128128
})
129129
}
130130

@@ -139,10 +139,10 @@ fn recv_after_disconnect() {
139139

140140
drop(s);
141141

142-
assert_eq!(r.recv().await, Some(1));
143-
assert_eq!(r.recv().await, Some(2));
144-
assert_eq!(r.recv().await, Some(3));
145-
assert_eq!(r.recv().await, None);
142+
assert_eq!(r.recv().await.unwrap(), 1);
143+
assert_eq!(r.recv().await.unwrap(), 2);
144+
assert_eq!(r.recv().await.unwrap(), 3);
145+
assert!(r.recv().await.is_err());
146146
})
147147
}
148148

@@ -164,7 +164,7 @@ fn len() {
164164
}
165165

166166
for i in 0..50 {
167-
r.recv().await;
167+
let _ = r.recv().await;
168168
assert_eq!(r.len(), 50 - i - 1);
169169
}
170170
}
@@ -188,7 +188,7 @@ fn len() {
188188
let r = r.clone();
189189
async move {
190190
for i in 0..COUNT {
191-
assert_eq!(r.recv().await, Some(i));
191+
assert_eq!(r.recv().await.unwrap(), i);
192192
let len = r.len();
193193
assert!(len <= CAP);
194194
}
@@ -214,7 +214,7 @@ fn disconnect_wakes_receiver() {
214214
let (s, r) = channel::<()>(1);
215215

216216
let child = task::spawn(async move {
217-
assert_eq!(r.recv().await, None);
217+
assert!(r.recv().await.is_err());
218218
});
219219

220220
task::sleep(ms(1000)).await;
@@ -233,9 +233,9 @@ fn spsc() {
233233

234234
let child = task::spawn(async move {
235235
for i in 0..COUNT {
236-
assert_eq!(r.recv().await, Some(i));
236+
assert_eq!(r.recv().await.unwrap(), i);
237237
}
238-
assert_eq!(r.recv().await, None);
238+
assert!(r.recv().await.is_err());
239239
});
240240

241241
for i in 0..COUNT {

0 commit comments

Comments
 (0)