Skip to content

Commit

Permalink
uv/tests: fix windows-only dependency filter
Browse files Browse the repository at this point in the history
Previously we were using `[+-~]`, but this includes the full range of
characters from `+` to `~`. Incidentally, this does include `-`. We
instead rewrite this as `[-+~]`, which probably matches the intent.
  • Loading branch information
BurntSushi committed Sep 3, 2024
1 parent 7516b09 commit 04d3510
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions crates/uv/tests/common/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1089,10 +1089,10 @@ pub fn run_and_format_with_status<T: AsRef<str>>(
if let Some(windows_filters) = windows_filters {
// The optional leading +/-/~ is for install logs, the optional next line is for lockfiles
let windows_only_deps = [
("( [+-~] )?colorama==\\d+(\\.[\\d+])+( \\\\\n --hash=.*)?\n( # via .*\n)?"),
("( [+-~] )?colorama==\\d+(\\.[\\d+])+(\\s+# via .*)?\n"),
("( [+-~] )?tzdata==\\d+(\\.[\\d+])+( \\\\\n --hash=.*)?\n( # via .*\n)?"),
("( [+-~] )?tzdata==\\d+(\\.[\\d+])+(\\s+# via .*)?\n"),
("( [-+~] )?colorama==\\d+(\\.[\\d+])+( \\\\\n --hash=.*)?\n( # via .*\n)?"),
("( [-+~] )?colorama==\\d+(\\.[\\d+])+(\\s+# via .*)?\n"),
("( [-+~] )?tzdata==\\d+(\\.[\\d+])+( \\\\\n --hash=.*)?\n( # via .*\n)?"),
("( [-+~] )?tzdata==\\d+(\\.[\\d+])+(\\s+# via .*)?\n"),
];
let mut removed_packages = 0;
for windows_only_dep in windows_only_deps {
Expand Down

0 comments on commit 04d3510

Please sign in to comment.