mirrors/uv
1
0
Fork 0
mirror of https://github.com/astral-sh/uv.git synced 2025-10-03 15:24:36 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Add missing --exclude-newer to executable tests (#1201)

Browse source 
A new version of `platformdirs` came out, which broke these.
This commit is contained in:
Charlie Marsh 2024-01-30 17:26:11 -08:00 • committed by GitHub
parent b88b9e1f3d
commit 262f29b558
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 36 additions and 32 deletions
Download patch file Download diff file Expand all files Collapse all files

68
crates/puffin/tests/pip_install.rs
View file

@ -976,10 +976,11 @@ fn install_executable() -> Result<()> {
assert_cmd_snapshot!(Command::new(get_cargo_bin(BIN_NAME)) assert_cmd_snapshot!(Command::new(get_cargo_bin(BIN_NAME))
.arg("pip") .arg("pip")
.arg("install") .arg("install")
.arg("pylint==3.0.3") .arg("pylint==3.0.0")
.arg("--cache-dir") .arg("--cache-dir")
.arg(cache_dir.path()) .arg(cache_dir.path())
.arg("--exclude-newer")
.arg(EXCLUDE_NEWER)
.env("VIRTUAL_ENV", venv.as_os_str()) .env("VIRTUAL_ENV", venv.as_os_str())
.current_dir(&temp_dir), @r###" .current_dir(&temp_dir), @r###"
success: true success: true
@ -990,12 +991,12 @@ fn install_executable() -> Result<()> {
Resolved 7 packages in [TIME] Resolved 7 packages in [TIME]
Downloaded 7 packages in [TIME] Downloaded 7 packages in [TIME]
Installed 7 packages in [TIME] Installed 7 packages in [TIME]
+ astroid==3.0.2 + astroid==3.0.1
+ dill==0.3.8 + dill==0.3.7
+ isort==5.13.2 + isort==5.12.0
+ mccabe==0.7.0 + mccabe==0.7.0
+ platformdirs==4.1.0 + platformdirs==4.0.0
+ pylint==3.0.3 + pylint==3.0.0
+ tomlkit==0.12.3 + tomlkit==0.12.3
"###); "###);
}); });
@ -1016,19 +1017,20 @@ fn install_executable_copy() -> Result<()> {
let venv = create_venv_py312(&temp_dir, &cache_dir); let venv = create_venv_py312(&temp_dir, &cache_dir);
insta::with_settings!({ insta::with_settings!({
filters => INSTA_FILTERS.to_vec() filters => INSTA_FILTERS.to_vec()
}, { }, {
assert_cmd_snapshot!(Command::new(get_cargo_bin(BIN_NAME)) assert_cmd_snapshot!(Command::new(get_cargo_bin(BIN_NAME))
.arg("pip") .arg("pip")
.arg("install") .arg("install")
.arg("pylint==3.0.3") .arg("pylint==3.0.0")
.arg("--link-mode") .arg("--link-mode")
.arg("copy") .arg("copy")
.arg("--cache-dir")
.arg("--cache-dir") .arg(cache_dir.path())
.arg(cache_dir.path()) .arg("--exclude-newer")
.env("VIRTUAL_ENV", venv.as_os_str()) .arg(EXCLUDE_NEWER)
.current_dir(&temp_dir), @r###" .env("VIRTUAL_ENV", venv.as_os_str())
.current_dir(&temp_dir), @r###"
success: true success: true
exit_code: 0 exit_code: 0
----- stdout ----- ----- stdout -----
@ -1037,15 +1039,15 @@ fn install_executable_copy() -> Result<()> {
Resolved 7 packages in [TIME] Resolved 7 packages in [TIME]
Downloaded 7 packages in [TIME] Downloaded 7 packages in [TIME]
Installed 7 packages in [TIME] Installed 7 packages in [TIME]
+ astroid==3.0.2 + astroid==3.0.1
+ dill==0.3.8 + dill==0.3.7
+ isort==5.13.2 + isort==5.12.0
+ mccabe==0.7.0 + mccabe==0.7.0
+ platformdirs==4.1.0 + platformdirs==4.0.0
+ pylint==3.0.3 + pylint==3.0.0
+ tomlkit==0.12.3 + tomlkit==0.12.3
"###); "###);
}); });
// Verify that `pylint` is executable. // Verify that `pylint` is executable.
let executable = venv.join("bin/pylint"); let executable = venv.join("bin/pylint");
@ -1068,11 +1070,13 @@ fn install_executable_hardlink() -> Result<()> {
assert_cmd_snapshot!(Command::new(get_cargo_bin(BIN_NAME)) assert_cmd_snapshot!(Command::new(get_cargo_bin(BIN_NAME))
.arg("pip") .arg("pip")
.arg("install") .arg("install")
.arg("pylint==3.0.3") .arg("pylint==3.0.0")
.arg("--link-mode") .arg("--link-mode")
.arg("hardlink") .arg("hardlink")
.arg("--cache-dir") .arg("--cache-dir")
.arg(cache_dir.path()) .arg(cache_dir.path())
.arg("--exclude-newer")
.arg(EXCLUDE_NEWER)
.env("VIRTUAL_ENV", venv.as_os_str()) .env("VIRTUAL_ENV", venv.as_os_str())
.current_dir(&temp_dir), @r###" .current_dir(&temp_dir), @r###"
success: true success: true
@ -1083,12 +1087,12 @@ fn install_executable_hardlink() -> Result<()> {
Resolved 7 packages in [TIME] Resolved 7 packages in [TIME]
Downloaded 7 packages in [TIME] Downloaded 7 packages in [TIME]
Installed 7 packages in [TIME] Installed 7 packages in [TIME]
+ astroid==3.0.2 + astroid==3.0.1
+ dill==0.3.8 + dill==0.3.7
+ isort==5.13.2 + isort==5.12.0
+ mccabe==0.7.0 + mccabe==0.7.0
+ platformdirs==4.1.0 + platformdirs==4.0.0
+ pylint==3.0.3 + pylint==3.0.0
+ tomlkit==0.12.3 + tomlkit==0.12.3
"###); "###);
}); });