Skip to content

Commit 4d74033

Browse files
committed
Clarify assertion messages involving \\localhost
This abbreviates the significance of `\\localhost` more specifically, as win-absolute-unc-host rather than win-absolue-unc. It also points out what seems unusual about joining `\` and `\\localhost` to get `\localhost` with just one backslash. This message should be further revised to state why this behavior happens or otherwise describe it clearly, once known. (Or if this turns out to be a bug in the standard library, then that can be stated, with an issue link either in the code or commit message.) This relates to: - https://github.com/Byron/gitoxide/pull/1374/files/e955770c0b5c06a6c8518a06df4aa0cc3b506f16#r1721079809
1 parent a34c0db commit 4d74033

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

gix-fs/tests/stack/mod.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,12 +120,12 @@ fn path_join_handling() {
120120
assert_eq!(
121121
p("/").join("\\\\localhost"),
122122
p("\\localhost"),
123-
"unix-absolute + win-absolute-unc = win-absolute-unc"
123+
"unix-absolute + win-absolute-unc-host = strangely, single-backslashed host"
124124
);
125125
assert_eq!(
126126
p("relative").join("\\\\localhost"),
127127
p("\\\\localhost"),
128-
"relative + win-absolute-unc = win-absolute-unc"
128+
"relative + win-absolute-unc-host = win-absolute-unc-host"
129129
);
130130
}
131131

0 commit comments

Comments
 (0)