Skip to content

Commit

Permalink
sync: fix Semaphore::MAX_PERMITS test (#5582)
Browse files Browse the repository at this point in the history
  • Loading branch information
Darksonn committed Mar 25, 2023
1 parent 92d33b7 commit 822af18
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions tokio/src/sync/tests/semaphore_batch.rs
@@ -1,6 +1,8 @@
use crate::sync::batch_semaphore::Semaphore;
use tokio_test::*;

const MAX_PERMITS: usize = crate::sync::Semaphore::MAX_PERMITS;

#[cfg(tokio_wasm_not_wasi)]
use wasm_bindgen_test::wasm_bindgen_test as test;

Expand Down Expand Up @@ -168,12 +170,16 @@ fn poll_acquire_one_zero_permits() {
assert_ready_ok!(acquire.poll());
}

#[test]
fn max_permits_doesnt_panic() {
Semaphore::new(MAX_PERMITS);
}

#[test]
#[should_panic]
#[cfg(not(tokio_wasm))] // wasm currently doesn't support unwinding
fn validates_max_permits() {
use std::usize;
Semaphore::new((usize::MAX >> 2) + 1);
Semaphore::new(MAX_PERMITS + 1);
}

#[test]
Expand Down

0 comments on commit 822af18

Please sign in to comment.