Unnamed repository; edit this file 'description' to name the repository.
hir-def tests: sort results before comparing, since FxHashSet iteration order isn't guaranteed
(And, in fact, it failed on i686)
Amos Wenger 2022-07-24
parent 74998e4 · commit ff31785
-rw-r--r--crates/hir-def/src/import_map.rs53
1 files changed, 28 insertions, 25 deletions
diff --git a/crates/hir-def/src/import_map.rs b/crates/hir-def/src/import_map.rs
index 000fe5ac7b..252d669324 100644
--- a/crates/hir-def/src/import_map.rs
+++ b/crates/hir-def/src/import_map.rs
@@ -516,6 +516,9 @@ mod tests {
mark
))
})
+ // HashSet iteration order isn't defined - it's different on
+ // x86_64 and i686 at the very least
+ .sorted()
.collect::<String>();
expect.assert_eq(&actual)
}
@@ -831,10 +834,10 @@ mod tests {
Query::new("fmt".to_string()).search_mode(SearchMode::Fuzzy),
expect![[r#"
dep::fmt (t)
- dep::fmt::Display::format_method (a)
dep::fmt::Display (t)
dep::fmt::Display::FMT_CONST (a)
dep::fmt::Display::format_function (a)
+ dep::fmt::Display::format_method (a)
"#]],
);
}
@@ -860,10 +863,10 @@ mod tests {
"main",
Query::new("fmt".to_string()).search_mode(SearchMode::Fuzzy).assoc_items_only(),
expect![[r#"
- dep::fmt::Display::format_method (a)
- dep::fmt::Display::FMT_CONST (a)
- dep::fmt::Display::format_function (a)
- "#]],
+ dep::fmt::Display::FMT_CONST (a)
+ dep::fmt::Display::format_function (a)
+ dep::fmt::Display::format_method (a)
+ "#]],
);
check_search(
@@ -920,13 +923,13 @@ mod tests {
"main",
Query::new("fmt".to_string()).search_mode(SearchMode::Fuzzy),
expect![[r#"
- dep::fmt (t)
- dep::format (f)
- dep::Fmt (v)
dep::Fmt (m)
dep::Fmt (t)
- dep::fmt::Display::fmt (a)
+ dep::Fmt (v)
+ dep::fmt (t)
dep::fmt::Display (t)
+ dep::fmt::Display::fmt (a)
+ dep::format (f)
"#]],
);
@@ -935,10 +938,10 @@ mod tests {
"main",
Query::new("fmt".to_string()).search_mode(SearchMode::Equals),
expect![[r#"
- dep::fmt (t)
- dep::Fmt (v)
dep::Fmt (m)
dep::Fmt (t)
+ dep::Fmt (v)
+ dep::fmt (t)
dep::fmt::Display::fmt (a)
"#]],
);
@@ -948,12 +951,12 @@ mod tests {
"main",
Query::new("fmt".to_string()).search_mode(SearchMode::Contains),
expect![[r#"
- dep::fmt (t)
- dep::Fmt (v)
dep::Fmt (m)
dep::Fmt (t)
- dep::fmt::Display::fmt (a)
+ dep::Fmt (v)
+ dep::fmt (t)
dep::fmt::Display (t)
+ dep::fmt::Display::fmt (a)
"#]],
);
}
@@ -989,12 +992,12 @@ mod tests {
"main",
Query::new("fmt".to_string()),
expect![[r#"
- dep::fmt (t)
- dep::Fmt (v)
dep::Fmt (m)
dep::Fmt (t)
- dep::fmt::Display::fmt (a)
+ dep::Fmt (v)
+ dep::fmt (t)
dep::fmt::Display (t)
+ dep::fmt::Display::fmt (a)
"#]],
);
@@ -1003,10 +1006,10 @@ mod tests {
"main",
Query::new("fmt".to_string()).name_only(),
expect![[r#"
- dep::fmt (t)
- dep::Fmt (v)
dep::Fmt (m)
dep::Fmt (t)
+ dep::Fmt (v)
+ dep::fmt (t)
dep::fmt::Display::fmt (a)
"#]],
);
@@ -1027,10 +1030,10 @@ mod tests {
"main",
Query::new("FMT".to_string()),
expect![[r#"
- dep::fmt (t)
+ dep::FMT (t)
dep::FMT (v)
+ dep::fmt (t)
dep::fmt (v)
- dep::FMT (t)
"#]],
);
@@ -1068,10 +1071,10 @@ mod tests {
"main",
Query::new("".to_string()).limit(2),
expect![[r#"
- dep::fmt (t)
+ dep::Fmt (m)
dep::Fmt (t)
dep::Fmt (v)
- dep::Fmt (m)
+ dep::fmt (t)
"#]],
);
}
@@ -1091,10 +1094,10 @@ mod tests {
"main",
Query::new("FMT".to_string()),
expect![[r#"
- dep::fmt (t)
+ dep::FMT (t)
dep::FMT (v)
+ dep::fmt (t)
dep::fmt (v)
- dep::FMT (t)
"#]],
);