Skip to content

Commit

Permalink
Add an #[ignore] to the instant_duration_since_panic test.
Browse files Browse the repository at this point in the history
As of rust-lang/rust#89926, this test behaves differently. We support
rust versions before and after this change, so both behaviors are
possible, so just disable this test for now.
  • Loading branch information
snev68 authored and sunfishcode committed Feb 28, 2022
1 parent b5245f9 commit efe6505
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tests/time.rs
Original file line number Diff line number Diff line change
Expand Up @@ -78,8 +78,14 @@ fn instant_math_is_associative() {
assert_eq!((now + offset) - now, (now - now) + offset);
}

// Ignore this test for now. As of rust-lang/rust#89926, Rust no longer panics
// on this test.
//
// TODO: Once our MSRV no longer contains the old behavior, we should update
// to the latest version of this test.
#[test]
#[should_panic]
#[ignore]
fn instant_duration_since_panic() {
let clock = MonotonicClock::new(ambient_authority());
let a = clock.now();
Expand Down

0 comments on commit efe6505

Please sign in to comment.