Skip to content

Commit 532ee2d

Browse files
committed
Rename cases to clarify "ordinary" and "strange"
That is, to clarify that "strange" means "strange values," and that "ordinary" means "ordinary values" or, really, "no strange values." The vacuously ordinary case is accordingly commented for clarity.
1 parent 323f52f commit 532ee2d

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

gix-path/src/env/git/tests.rs

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ mod locations {
5050
}
5151

5252
#[test]
53-
fn locations_under_program_files_ordinary_current_var_only() {
53+
fn locations_under_program_files_ordinary_values_current_var_only() {
5454
assert_eq!(
5555
locations_from!(
5656
"ProgramFiles" => r"C:\Program Files",
@@ -67,7 +67,7 @@ mod locations {
6767
}
6868

6969
#[test]
70-
fn locations_under_program_files_ordinary_all_vars() {
70+
fn locations_under_program_files_ordinary_values_all_vars() {
7171
assert_eq!(
7272
locations_from!(
7373
"ProgramFiles" => {
@@ -89,12 +89,13 @@ mod locations {
8989
}
9090

9191
#[test]
92-
fn locations_under_program_files_ordinary_no_vars() {
92+
fn locations_under_program_files_ordinary_values_no_vars() {
93+
// Regarding this test name: the significance of "ordinary values" is "no strange values."
9394
assert_eq!(locations_from!(), Vec::<PathBuf>::new());
9495
}
9596

9697
#[test]
97-
fn locations_under_program_files_strange_all_vars_distinct() {
98+
fn locations_under_program_files_strange_values_all_vars_distinct() {
9899
assert_eq!(
99100
locations_from!(
100101
"ProgramFiles" => r"X:\cur\rent",
@@ -121,7 +122,7 @@ mod locations {
121122
}
122123

123124
#[test]
124-
fn locations_under_program_files_strange_64bit_var_only() {
125+
fn locations_under_program_files_strange_values_64bit_var_only() {
125126
assert_eq!(
126127
locations_from!(
127128
"ProgramW6432" => r"Z:\wi\de",
@@ -131,7 +132,7 @@ mod locations {
131132
}
132133

133134
#[test]
134-
fn locations_under_program_files_strange_all_vars_path_cruft() {
135+
fn locations_under_program_files_strange_values_all_vars_path_cruft() {
135136
assert_eq!(
136137
locations_from!(
137138
"ProgramFiles" => r"Z:/wi//de/",
@@ -156,7 +157,7 @@ mod locations {
156157
}
157158

158159
#[test]
159-
fn locations_under_program_files_strange_some_relative() {
160+
fn locations_under_program_files_strange_values_some_relative() {
160161
assert_eq!(
161162
locations_from!(
162163
"ProgramFiles" => r"foo\bar",

0 commit comments

Comments
 (0)