Conditionalize more tests that require PyPI (#13699)

<!--
Thank you for contributing to uv! To help us out with reviewing, please
consider the following:

- Does this pull request include a summary of the change? (See below.)
- Does this pull request include a descriptive title?
- Does this pull request include references to any relevant issues?
-->

## Summary

<!-- What's the purpose of the change? What does it do, and why? -->
Use the existing `pypi` feature to conditionalize a number of tests that
attempt to access https://pypi.org and/or
https://files.pythonhosted.org. See
#8970 (comment).

There is no reason to believe that these are *all* of the tests that
need to be conditionalized on the `pypi` feature, but this should be a
solid step in the right direction.

## Test Plan

<!-- How was it tested? -->

This allows me to build and run the integration tests in [Fedora’s `uv`
package](https://src.fedoraproject.org/rpms/uv) without having to
manually skip tests that try to access PyPI. I confirmed that this
appears to accomplish that goal.

Otherwise, this should be tested by building and running the tests as
usual. As mentioned in
#8970 (comment), a
more complete solution would include CI tests that confirm these
features are working as intended. I’m not in a position to offer that.

Author: musicinmybrain

crates/uv/tests/it/branching_urls.rs

@@ -14,6 +14,7 @@ use crate::common::{TestContext, make_project, uv_snapshot};
 /// ]
 /// ```
 #[test]
+#[cfg(feature = "pypi")]
 fn branching_urls_disjoint() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -47,6 +48,7 @@ fn branching_urls_disjoint() -> Result<()> {
 /// ]
 /// ```
 #[test]
+#[cfg(feature = "pypi")]
 fn branching_urls_overlapping() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -83,6 +85,7 @@ fn branching_urls_overlapping() -> Result<()> {
 /// a -> b -> b2 -> https://../iniconfig-2.0.0-py3-none-any.whl
 /// ```
 #[test]
+#[cfg(feature = "pypi")]
 fn root_package_splits_but_transitive_conflict() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -151,6 +154,7 @@ fn root_package_splits_but_transitive_conflict() -> Result<()> {
 /// a -> b -> b2 ; python_version >= '3.12' -> https://../iniconfig-2.0.0-py3-none-any.whl
 /// ```
 #[test]
+#[cfg(feature = "pypi")]
 fn root_package_splits_transitive_too() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -356,6 +360,7 @@ fn root_package_splits_transitive_too() -> Result<()> {
 /// a -> b2 ; python_version >= '3.12' -> iniconfig==2.0.0
 /// ```
 #[test]
+#[cfg(feature = "pypi")]
 fn root_package_splits_other_dependencies_too() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -539,6 +544,7 @@ fn root_package_splits_other_dependencies_too() -> Result<()> {
 /// ]
 /// ```
 #[test]
+#[cfg(feature = "pypi")]
 fn branching_between_registry_and_direct_url() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -624,7 +630,7 @@ fn branching_between_registry_and_direct_url() -> Result<()> {
 /// ]
 /// ```
 #[test]
-#[cfg(feature = "git")]
+#[cfg(all(feature = "git", feature = "pypi"))]
 fn branching_urls_of_different_sources_disjoint() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -708,7 +714,7 @@ fn branching_urls_of_different_sources_disjoint() -> Result<()> {
 /// ]
 /// ```
 #[test]
-#[cfg(feature = "git")]
+#[cfg(all(feature = "git", feature = "pypi"))]
 fn branching_urls_of_different_sources_conflict() -> Result<()> {
     let context = TestContext::new("3.12");
 

crates/uv/tests/it/build_backend.rs

@@ -24,6 +24,7 @@ const BUILT_BY_UV_TEST_SCRIPT: &str = indoc! {r#"
 ///
 /// We can't test end-to-end here including the PEP 517 bridge code since we don't have a uv wheel.
 #[test]
+#[cfg(feature = "pypi")]
 fn built_by_uv_direct_wheel() -> Result<()> {
     let context = TestContext::new("3.12");
     let built_by_uv = Path::new("../../scripts/packages/built-by-uv");
@@ -83,6 +84,7 @@ fn built_by_uv_direct_wheel() -> Result<()> {
 /// We can't test end-to-end here including the PEP 517 bridge code since we don't have a uv wheel,
 /// so we call the build backend directly.
 #[test]
+#[cfg(feature = "pypi")]
 fn built_by_uv_direct() -> Result<()> {
     let context = TestContext::new("3.12");
     let built_by_uv = Path::new("../../scripts/packages/built-by-uv");
@@ -160,6 +162,7 @@ fn built_by_uv_direct() -> Result<()> {
 /// We can't test end-to-end here including the PEP 517 bridge code since we don't have a uv wheel,
 /// so we call the build backend directly.
 #[test]
+#[cfg(feature = "pypi")]
 fn built_by_uv_editable() -> Result<()> {
     let context = TestContext::new("3.12");
     let built_by_uv = Path::new("../../scripts/packages/built-by-uv");

crates/uv/tests/it/main.rs

@@ -41,6 +41,7 @@ mod lock_scenarios;
 
 mod version;
 
+#[cfg(all(feature = "python", feature = "pypi"))]
 mod pip_check;
 
 #[cfg(all(feature = "python", feature = "pypi"))]

crates/uv/tests/it/pip_list.rs

@@ -56,6 +56,7 @@ fn list_empty_json() {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_single_no_editable() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -96,6 +97,7 @@ fn list_single_no_editable() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_outdated_columns() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -136,6 +138,7 @@ fn list_outdated_columns() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_outdated_json() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -232,6 +235,7 @@ fn list_outdated_git() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_outdated_index() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -275,6 +279,7 @@ fn list_outdated_index() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_editable() {
     let context = TestContext::new("3.12");
 
@@ -320,6 +325,7 @@ fn list_editable() {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_editable_only() {
     let context = TestContext::new("3.12");
 
@@ -394,6 +400,7 @@ fn list_editable_only() {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_exclude() {
     let context = TestContext::new("3.12");
 
@@ -475,6 +482,7 @@ fn list_exclude() {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 #[cfg(not(windows))]
 fn list_format_json() {
     let context = TestContext::new("3.12");
@@ -541,6 +549,7 @@ fn list_format_json() {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_format_freeze() {
     let context = TestContext::new("3.12");
 
@@ -706,6 +715,7 @@ Version: 0.1-bulbasaur
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn list_ignores_quiet_flag_format_freeze() {
     let context = TestContext::new("3.12");
 

crates/uv/tests/it/pip_show.rs

@@ -26,6 +26,7 @@ fn show_empty() {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_requires_multiple() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -75,6 +76,7 @@ fn show_requires_multiple() -> Result<()> {
 /// Asserts that the Python version marker in the metadata is correctly evaluated.
 /// `click` v8.1.7 requires `importlib-metadata`, but only when `python_version < "3.8"`.
 #[test]
+#[cfg(feature = "pypi")]
 fn show_python_version_marker() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -124,6 +126,7 @@ fn show_python_version_marker() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_found_single_package() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -168,6 +171,7 @@ fn show_found_single_package() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_found_multiple_packages() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -224,6 +228,7 @@ fn show_found_multiple_packages() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_found_one_out_of_three() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -276,6 +281,7 @@ fn show_found_one_out_of_three() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_found_one_out_of_two_quiet() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -323,6 +329,7 @@ fn show_found_one_out_of_two_quiet() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_empty_quiet() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -369,6 +376,7 @@ fn show_empty_quiet() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_editable() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -405,6 +413,7 @@ fn show_editable() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_required_by_multiple() -> Result<()> {
     let context = TestContext::new("3.12");
 
@@ -460,6 +469,7 @@ fn show_required_by_multiple() -> Result<()> {
 }
 
 #[test]
+#[cfg(feature = "pypi")]
 fn show_files() {
     let context = TestContext::new("3.12");