Skip to content

Commit

Permalink
Auto merge of #11976 - ehuss:fix-not_found_permutations, r=epage
Browse files Browse the repository at this point in the history
Fix flaky not_found_permutations test.

This fixes the `registry::not_found_permutations` test which would randomly fail since the order of http requests was not deterministic. The resolver can issue queries in parallel which can process requests out-of-order.

Fixes #11975
  • Loading branch information
bors committed Apr 14, 2023
2 parents d2e7cfb + 6fa758e commit 7fb89f0
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions tests/testsuite/registry.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3183,13 +3183,14 @@ required by package `foo v0.0.1 ([ROOT]/foo)`
",
)
.run();
let misses = misses.lock().unwrap();
let mut misses = misses.lock().unwrap();
misses.sort();
assert_eq!(
&*misses,
&[
"/index/a-/b-/a-b-c",
"/index/a_/b-/a_b-c",
"/index/a-/b_/a-b_c",
"/index/a_/b-/a_b-c",
"/index/a_/b_/a_b_c"
]
);
Expand Down

0 comments on commit 7fb89f0

Please sign in to comment.