Use a dedicated all-tests features instead of all-features

Features can change behaviour, e.g. the pyo3 feature setting linker options and #941 setting the stack size. Instead of running tests with all features, we list the features required for tests.

The old command
```
cargo nextest run --all --all-features --status-level skip --failure-output immediate-final --no-fail-fast -j 12
```
and the new command
```
cargo nextest run --all --features all-tests --status-level skip --failure-output immediate-final --no-fail-fast -j 12
```
run the same tests.

This includes a fix for the pep508_rs `evaluate_extras_and_python_version` doc test, which is only run by `cargo test` but not by `cargo nextest`.

This is required for #941.

.github/workflows/ci.yaml

@@ -75,4 +75,4 @@ jobs:
  with:
  save-if: ${{ github.ref == 'refs/heads/main' }}
  - name: "Tests"
- run: cargo nextest run --all --all-features --status-level skip --failure-output immediate-final --no-fail-fast -j 12
+ run: cargo nextest run --all --features all-tests --status-level skip --failure-output immediate-final --no-fail-fast -j 12

crates/install-wheel-rs/src/wheel.rs

@@ -362,7 +362,7 @@ pub(crate) fn write_script_entrypoints(
  record,
  )?;
  // We need to make the launcher executable
- #[cfg(target_family = "unix")]
+ #[cfg(unix)]
  {
  use std::os::unix::fs::PermissionsExt;
  fs::set_permissions(

crates/pep508-rs/src/marker.rs

@@ -733,15 +733,16 @@ impl MarkerExpression {
  /// # use pep440_rs::Version;
  ///
  /// # fn main() -> Result<(), Pep508Error> {
+ /// use puffin_normalize::ExtraName;
  /// let marker_tree = MarkerTree::from_str(r#"("linux" in sys_platform) and extra == 'day'"#)?;
  /// let versions: Vec<Version> = (8..12).map(|minor| Version::new([3, minor])).collect();
- /// assert!(marker_tree.evaluate_extras_and_python_version(&["day".to_string()].into(), &versions));
- /// assert!(!marker_tree.evaluate_extras_and_python_version(&["night".to_string()].into(), &versions));
+ /// assert!(marker_tree.evaluate_extras_and_python_version(&[ExtraName::from_str("day").unwrap()].into(), &versions));
+ /// assert!(!marker_tree.evaluate_extras_and_python_version(&[ExtraName::from_str("night").unwrap()].into(), &versions));
  ///
  /// let marker_tree = MarkerTree::from_str(r#"extra == 'day' and python_version < '3.11' and '3.10' <= python_version"#)?;
- /// assert!(!marker_tree.evaluate_extras_and_python_version(&["day".to_string()].into(), &vec![Version::new([3, 9])]));
- /// assert!(marker_tree.evaluate_extras_and_python_version(&["day".to_string()].into(), &vec![Version::new([3, 10])]));
- /// assert!(!marker_tree.evaluate_extras_and_python_version(&["day".to_string()].into(), &vec![Version::new([3, 11])]));
+ /// assert!(!marker_tree.evaluate_extras_and_python_version(&[ExtraName::from_str("day").unwrap()].into(), &vec![Version::new([3, 9])]));
+ /// assert!(marker_tree.evaluate_extras_and_python_version(&[ExtraName::from_str("day").unwrap()].into(), &vec![Version::new([3, 10])]));
+ /// assert!(!marker_tree.evaluate_extras_and_python_version(&[ExtraName::from_str("day").unwrap()].into(), &vec![Version::new([3, 11])]));
  /// # Ok(())
  /// # }
  /// ```

crates/puffin-cli/Cargo.toml

@@ -92,3 +92,6 @@ pypi = []
 git = []
 # Introduces a dependency on Maturin.
 maturin = []
+
+# Activate all feature gated tests
+all-tests = ["python", "pypi", "git", "maturin"]

crates/puffin-resolver/Cargo.toml

@@ -71,3 +71,6 @@ insta = { version = "1.34.0" }
 [features]
 # Introduces a dependency on PyPI.
 pypi = []
+
+# Activate all feature gated tests
+all-tests = ["pypi"]