Skip to content

Commit 859853f

Browse files
committed
refactor(harness): Move sorting into main discovery logic
1 parent f8fe233 commit 859853f

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

crates/libtest2-harness/src/harness.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,12 @@ impl Harness<StateParsed> {
111111
self.state.notifier.as_mut(),
112112
)?;
113113

114+
cases.sort_unstable_by_key(|case| {
115+
let priority = case_priority(case.as_ref(), &self.state.opts);
116+
let name = case.name().to_owned();
117+
(priority, name)
118+
});
119+
114120
self.state.notifier.notify(
115121
notify::event::DiscoverComplete {
116122
elapsed_s: Some(notify::Elapsed(self.state.start.elapsed())),
@@ -276,12 +282,6 @@ fn discover(
276282
let mut retain_cases = retain_cases.into_iter();
277283
cases.retain(|_| retain_cases.next().unwrap());
278284

279-
cases.sort_unstable_by_key(|case| {
280-
let priority = case_priority(case.as_ref(), opts);
281-
let name = case.name().to_owned();
282-
(priority, name)
283-
});
284-
285285
Ok(())
286286
}
287287

0 commit comments

Comments
 (0)