Skip to content

Commit

Permalink
Revert panic_safe test changes
Browse files Browse the repository at this point in the history
The changes made only a limited improvement for the current small
miri coverage and in general test coverage of the sort implementations.
But they exploded test times from ~13s to ~240s, which is not deemed
worth it.
  • Loading branch information
Voultapher committed Jun 17, 2024
1 parent 78eaad5 commit 384c205
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion alloc/src/slice/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,8 @@ fn panic_safe() {

let mut rng = test_rng();

let lens = if cfg!(miri) { (1..10).chain(30..36) } else { (1..20).chain(70..MAX_LEN) };
// Miri is too slow (but still need to `chain` to make the types match)
let lens = if cfg!(miri) { (1..10).chain(0..0) } else { (1..20).chain(70..MAX_LEN) };
let moduli: &[u32] = if cfg!(miri) { &[5] } else { &[5, 20, 50] };

for len in lens {
Expand Down

0 comments on commit 384c205

Please sign in to comment.