Skip to content

Commit f973bad

Browse files
committed
Fix pip tree tests
1 parent bf3286c commit f973bad

1 file changed

Lines changed: 40 additions & 89 deletions

File tree

crates/uv/tests/pip_tree.rs

Lines changed: 40 additions & 89 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ use crate::common::{get_bin, TestContext};
1010
mod common;
1111

1212
#[test]
13-
fn no_package() {
13+
fn pip_tree_no_package() {
1414
let context = TestContext::new("3.12");
1515

1616
uv_snapshot!(context.filters(), context.pip_tree(), @r###"
@@ -25,7 +25,7 @@ fn no_package() {
2525
}
2626

2727
#[test]
28-
fn prune_last_in_the_subgroup() {
28+
fn pip_tree_prune_last_in_the_subgroup() {
2929
let context = TestContext::new("3.12");
3030

3131
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -68,7 +68,7 @@ fn prune_last_in_the_subgroup() {
6868
}
6969

7070
#[test]
71-
fn single_package() {
71+
fn pip_tree_single_package() {
7272
let context = TestContext::new("3.12");
7373

7474
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -113,7 +113,7 @@ fn single_package() {
113113
// `pandas` requires `numpy` with markers on Python version.
114114
#[test]
115115
#[cfg(not(windows))]
116-
fn python_version_marker() {
116+
fn pip_tree_python_version_marker() {
117117
let context = TestContext::new("3.12");
118118

119119
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -159,7 +159,7 @@ fn python_version_marker() {
159159
}
160160

161161
#[test]
162-
fn nested_dependencies() {
162+
fn pip_tree_nested_dependencies() {
163163
let context = TestContext::new("3.12");
164164

165165
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -206,7 +206,7 @@ fn nested_dependencies() {
206206

207207
// Identical test as `invert` since `--reverse` is simply an alias for `--invert`.
208208
#[test]
209-
fn reverse() {
209+
fn pip_tree_reverse() {
210210
let context = TestContext::new("3.12");
211211

212212
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -254,7 +254,7 @@ fn reverse() {
254254
}
255255

256256
#[test]
257-
fn invert() {
257+
fn pip_tree_invert() {
258258
let context = TestContext::new("3.12");
259259

260260
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -302,7 +302,7 @@ fn invert() {
302302
}
303303

304304
#[test]
305-
fn depth() {
305+
fn pip_tree_depth() {
306306
let context = TestContext::new("3.12");
307307

308308
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -330,16 +330,9 @@ fn depth() {
330330
"###
331331
);
332332

333-
uv_snapshot!(context.filters(), Command::new(get_bin())
334-
.arg("pip")
335-
.arg("tree")
336-
.arg("--cache-dir")
337-
.arg(context.cache_dir.path())
333+
uv_snapshot!(context.filters(), context.pip_tree()
338334
.arg("--depth")
339-
.arg("0")
340-
.env("VIRTUAL_ENV", context.venv.as_os_str())
341-
.env("UV_NO_WRAP", "1")
342-
.current_dir(&context.temp_dir), @r###"
335+
.arg("0"), @r###"
343336
success: true
344337
exit_code: 0
345338
----- stdout -----
@@ -350,16 +343,9 @@ fn depth() {
350343
"###
351344
);
352345

353-
uv_snapshot!(context.filters(), Command::new(get_bin())
354-
.arg("pip")
355-
.arg("tree")
356-
.arg("--cache-dir")
357-
.arg(context.cache_dir.path())
346+
uv_snapshot!(context.filters(), context.pip_tree()
358347
.arg("--depth")
359-
.arg("1")
360-
.env("VIRTUAL_ENV", context.venv.as_os_str())
361-
.env("UV_NO_WRAP", "1")
362-
.current_dir(&context.temp_dir), @r###"
348+
.arg("1"), @r###"
363349
success: true
364350
exit_code: 0
365351
----- stdout -----
@@ -374,16 +360,9 @@ fn depth() {
374360
"###
375361
);
376362

377-
uv_snapshot!(context.filters(), Command::new(get_bin())
378-
.arg("pip")
379-
.arg("tree")
380-
.arg("--cache-dir")
381-
.arg(context.cache_dir.path())
363+
uv_snapshot!(context.filters(), context.pip_tree()
382364
.arg("--depth")
383-
.arg("2")
384-
.env("VIRTUAL_ENV", context.venv.as_os_str())
385-
.env("UV_NO_WRAP", "1")
386-
.current_dir(&context.temp_dir), @r###"
365+
.arg("2"), @r###"
387366
success: true
388367
exit_code: 0
389368
----- stdout -----
@@ -400,7 +379,7 @@ fn depth() {
400379
}
401380

402381
#[test]
403-
fn prune() {
382+
fn pip_tree_prune() {
404383
let context = TestContext::new("3.12");
405384

406385
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -428,16 +407,9 @@ fn prune() {
428407
"###
429408
);
430409

431-
uv_snapshot!(context.filters(), Command::new(get_bin())
432-
.arg("pip")
433-
.arg("tree")
434-
.arg("--cache-dir")
435-
.arg(context.cache_dir.path())
410+
uv_snapshot!(context.filters(), context.pip_tree()
436411
.arg("--prune")
437-
.arg("numpy")
438-
.env("VIRTUAL_ENV", context.venv.as_os_str())
439-
.env("UV_NO_WRAP", "1")
440-
.current_dir(&context.temp_dir), @r###"
412+
.arg("numpy"), @r###"
441413
success: true
442414
exit_code: 0
443415
----- stdout -----
@@ -450,18 +422,11 @@ fn prune() {
450422
"###
451423
);
452424

453-
uv_snapshot!(context.filters(), Command::new(get_bin())
454-
.arg("pip")
455-
.arg("tree")
456-
.arg("--cache-dir")
457-
.arg(context.cache_dir.path())
425+
uv_snapshot!(context.filters(), context.pip_tree()
458426
.arg("--prune")
459427
.arg("numpy")
460428
.arg("--prune")
461-
.arg("joblib")
462-
.env("VIRTUAL_ENV", context.venv.as_os_str())
463-
.env("UV_NO_WRAP", "1")
464-
.current_dir(&context.temp_dir), @r###"
429+
.arg("joblib"), @r###"
465430
success: true
466431
exit_code: 0
467432
----- stdout -----
@@ -473,16 +438,9 @@ fn prune() {
473438
"###
474439
);
475440

476-
uv_snapshot!(context.filters(), Command::new(get_bin())
477-
.arg("pip")
478-
.arg("tree")
479-
.arg("--cache-dir")
480-
.arg(context.cache_dir.path())
441+
uv_snapshot!(context.filters(), context.pip_tree()
481442
.arg("--prune")
482-
.arg("scipy")
483-
.env("VIRTUAL_ENV", context.venv.as_os_str())
484-
.env("UV_NO_WRAP", "1")
485-
.current_dir(&context.temp_dir), @r###"
443+
.arg("scipy"), @r###"
486444
success: true
487445
exit_code: 0
488446
----- stdout -----
@@ -498,7 +456,7 @@ fn prune() {
498456

499457
#[test]
500458
#[cfg(target_os = "macos")]
501-
fn complex_nested_dependencies_inverted() {
459+
fn pip_tree_complex_nested_dependencies_inverted() {
502460
let context = TestContext::new("3.12");
503461

504462
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -620,7 +578,7 @@ fn complex_nested_dependencies_inverted() {
620578

621579
#[test]
622580
#[cfg(target_os = "macos")]
623-
fn complex_nested_dependencies() {
581+
fn pip_tree_complex_nested_dependencies() {
624582
let context = TestContext::new("3.12");
625583

626584
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -723,7 +681,7 @@ fn complex_nested_dependencies() {
723681

724682
#[test]
725683
#[cfg(target_os = "macos")]
726-
fn prune_large_tree() {
684+
fn pip_tree_prune_large_tree() {
727685
let context = TestContext::new("3.12");
728686

729687
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -776,16 +734,9 @@ fn prune_large_tree() {
776734
"###
777735
);
778736

779-
uv_snapshot!(context.filters(), Command::new(get_bin())
780-
.arg("pip")
781-
.arg("tree")
782-
.arg("--cache-dir")
783-
.arg(context.cache_dir.path())
737+
uv_snapshot!(context.filters(), context.pip_tree()
784738
.arg("--prune")
785-
.arg("hatchling")
786-
.env("VIRTUAL_ENV", context.venv.as_os_str())
787-
.env("UV_NO_WRAP", "1")
788-
.current_dir(&context.temp_dir), @r###"
739+
.arg("hatchling"), @r###"
789740
success: true
790741
exit_code: 0
791742
----- stdout -----
@@ -833,7 +784,7 @@ fn prune_large_tree() {
833784
// Additionally, package `uv-cyclic-dependencies-c` is included (depends on `uv-cyclic-dependencies-a`)
834785
// to make this test case more realistic and meaningful.
835786
#[test]
836-
fn cyclic_dependency() {
787+
fn pip_tree_cyclic_dependency() {
837788
let context = TestContext::new("3.12");
838789

839790
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -881,7 +832,7 @@ fn cyclic_dependency() {
881832

882833
// Ensure `pip tree` behaves correctly after a package has been removed.
883834
#[test]
884-
fn removed_dependency() {
835+
fn pip_tree_removed_dependency() {
885836
let context = TestContext::new("3.12");
886837

887838
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -936,7 +887,7 @@ fn removed_dependency() {
936887
}
937888

938889
#[test]
939-
fn multiple_packages() {
890+
fn pip_tree_multiple_packages() {
940891
let context = TestContext::new("3.12");
941892

942893
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -995,7 +946,7 @@ fn multiple_packages() {
995946
// Both `pendulum` and `boto3` depend on `python-dateutil`.
996947
#[test]
997948
#[cfg(not(windows))]
998-
fn multiple_packages_shared_descendant() {
949+
fn pip_tree_multiple_packages_shared_descendant() {
999950
let context = TestContext::new("3.12");
1000951

1001952
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1063,7 +1014,7 @@ fn multiple_packages_shared_descendant() {
10631014
// Test the interaction between `--no-dedupe` and `--invert`.
10641015
#[test]
10651016
#[cfg(not(windows))]
1066-
fn no_dedupe_and_invert() {
1017+
fn pip_tree_no_dedupe_and_invert() {
10671018
let context = TestContext::new("3.12");
10681019

10691020
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1139,7 +1090,7 @@ fn no_dedupe_and_invert() {
11391090
// in the presence of dependency cycles.
11401091
#[test]
11411092
#[cfg(not(windows))]
1142-
fn no_dedupe_and_cycle() {
1093+
fn pip_tree_no_dedupe_and_cycle() {
11431094
let context = TestContext::new("3.12");
11441095

11451096
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1239,7 +1190,7 @@ fn no_dedupe_and_cycle() {
12391190

12401191
#[test]
12411192
#[cfg(not(windows))]
1242-
fn no_dedupe() {
1193+
fn pip_tree_no_dedupe() {
12431194
let context = TestContext::new("3.12");
12441195

12451196
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1312,7 +1263,7 @@ fn no_dedupe() {
13121263

13131264
#[test]
13141265
#[cfg(feature = "git")]
1315-
fn with_editable() {
1266+
fn pip_tree_with_editable() {
13161267
let context = TestContext::new("3.12");
13171268

13181269
// Install the editable package.
@@ -1353,7 +1304,7 @@ fn with_editable() {
13531304

13541305
#[test]
13551306
#[cfg(target_os = "macos")]
1356-
fn package_flag_complex() {
1307+
fn pip_tree_package_flag_complex() {
13571308
let context = TestContext::new("3.12");
13581309

13591310
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1436,7 +1387,7 @@ fn package_flag_complex() {
14361387
}
14371388

14381389
#[test]
1439-
fn package_flag() {
1390+
fn pip_tree_package_flag() {
14401391
let context = TestContext::new("3.12");
14411392

14421393
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1502,7 +1453,7 @@ fn package_flag() {
15021453
}
15031454

15041455
#[test]
1505-
fn show_version_specifiers_simple() {
1456+
fn pip_tree_show_version_specifiers_simple() {
15061457
let context = TestContext::new("3.12");
15071458

15081459
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1546,7 +1497,7 @@ fn show_version_specifiers_simple() {
15461497

15471498
#[test]
15481499
#[cfg(target_os = "macos")]
1549-
fn show_version_specifiers_complex() {
1500+
fn pip_tree_show_version_specifiers_complex() {
15501501
let context = TestContext::new("3.12");
15511502

15521503
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1648,7 +1599,7 @@ fn show_version_specifiers_complex() {
16481599
}
16491600

16501601
#[test]
1651-
fn show_version_specifiers_with_invert() {
1602+
fn pip_tree_show_version_specifiers_with_invert() {
16521603
let context = TestContext::new("3.12");
16531604

16541605
let requirements_txt = context.temp_dir.child("requirements.txt");
@@ -1700,7 +1651,7 @@ fn show_version_specifiers_with_invert() {
17001651
}
17011652

17021653
#[test]
1703-
fn show_version_specifiers_with_package() {
1654+
fn pip_tree_show_version_specifiers_with_package() {
17041655
let context = TestContext::new("3.12");
17051656

17061657
let requirements_txt = context.temp_dir.child("requirements.txt");

0 commit comments

Comments
 (0)