Merge branch 'main' into consolidate-bot-prs-379-383

Author: tpvasconcelos

.coveragerc

@@ -17,7 +17,7 @@ src =
 [report]
 precision = 1
 show_missing = true
-skip_covered = false
+skip_covered = true
 fail_under = 50
 
 # Some of the patterns below were taken or adapted
@@ -36,6 +36,7 @@ exclude_lines =
     : \.\.\.(\s*#.*)?$
     ^ +\.\.\.$
     -> ['"]?NoReturn['"]?:
+    ^\s*assert_never\b
     # non-runnable code
     if __name__ == ['"]__main__['"]:$
 partial_branches =

.github/actions/setup-python/action.yml

@@ -6,12 +6,6 @@ inputs:
   python-version:
     description: "Python version (e.g. '3.7', '3.8.13', '3.11.0-alpha.1', '3.x')."
     required: true
-  install-seed-packages:
-    description: >
-      Set to 'true' to install and/or upgrade seed packages (pip, setuptools, and wheel)
-      before installing other packages.
-    required: false
-    default: "false"
   requirements:
     description: "Optional list of arguments passed to `uv pip install` (e.g. 'numpy==1.19.5', '-r requirements.txt')."
     required: false
@@ -23,37 +17,19 @@ runs:
       with:
         python-version: "${{ inputs.python-version }}"
 
+    # Install uv and enable its built-in cache. The cache key takes the
+    # platform, the uv version, and a hash of the files below into account.
+    # See https://docs.astral.sh/uv/guides/integration/github/ for details.
+    # NOTE: setup-uv publishes immutable releases only (no
+    #       floating major tags), so we pin the full version
     - name: Install uv
-      shell: bash
-      run: pip install --upgrade uv
-
-    # >>>>>>>> Caching uv packages >>>>>>>>
-
-    - name: Get uv cache dir
-      id: uv-cache
-      shell: bash
-      run: echo "dir=$(uv cache dir)" >> $GITHUB_OUTPUT
-
-    - name: Get the current date in format YYYYWW (year and week-of-year)
-      id: get-date
-      shell: bash
-      run: echo "date=$(/bin/date -u "+%Y%U")" >> $GITHUB_OUTPUT
-
-    - name: uv cache
-      uses: actions/cache@v4
+      uses: astral-sh/setup-uv@v9.0.0
       with:
-        path: ${{ steps.uv-cache.outputs.dir }}
-        # The cache's TTL is 7 days (one week)
-        key: uv-${{ runner.os }}-${{ inputs.python-version }}-${{ steps.get-date.outputs.date }}
-        # Fall back to the previous week's cache
-        restore-keys: uv-${{ runner.os }}-${{ inputs.python-version }}-
-
-    # <<<<<<<< Caching uv packages <<<<<<<<
-
-    - name: Install and/or upgrade seed packages
-      if: ${{ inputs.install-seed-packages == 'true' }}
-      shell: bash
-      run: uv pip install --verbose --system --upgrade pip setuptools wheel
+        enable-cache: true
+        cache-suffix: "py${{ inputs.python-version }}"
+        cache-dependency-glob: |
+          pyproject.toml
+          requirements/*.txt
 
     - name: Install requirements
       if: ${{ inputs.requirements != '' }}

AGENTS.md

@@ -209,7 +209,8 @@ def _coerce_to_densities(...):
 ## CI/CD Pipeline
 
 GitHub Actions runs tests on Python 3.10–3.14 across Ubuntu, macOS, and Windows.
-Codecov minimums are 98% overall and 100% diff coverage for new code.
+Codecov minimums are 100% overall and 100% diff coverage for new code.
+Intentionally-untested lines must carry an explicit `# pragma: no cover` comment.
 
 ## Notes for AI Assistants
 

codecov.yml

@@ -0,0 +1,15 @@
+# Codecov configuration
+# ref: https://docs.codecov.com/docs/codecov-yaml
+#
+# Note: CI only uploads the combined coverage report
+# (see the coverage-combine env in tox.ini), which omits
+# "cicd_utils/*" and "tests/*", so these targets apply
+# to the src/ridgeplot package only.
+coverage:
+  status:
+    project:
+      default:
+        target: 100%
+    patch:
+      default:
+        target: 100%

docs/reference/changelog.md

@@ -7,6 +7,9 @@ Unreleased changes
 
 ### CI/CD
 
+- Review the coverage configuration in light of `covdefaults`, adopting its `assert_never` exclusion and `skip_covered` report setting, and raising all package coverage gates to 100% ({gh-pr}`390`)
+- Adopt pytest's strict mode, following the recommendations from pytest's "Good Integration Practices" guide ({gh-pr}`387`)
+- Make the e2e path sanity check independent of the checkout directory's name, so tests can run from git worktrees ({gh-pr}`391`)
 - Bump actions/download-artifact from 7 to 8 ({gh-pr}`368`)
 - Bump actions/upload-artifact from 6 to 7 ({gh-pr}`369`)
 - Bump sigstore/gh-action-sigstore-python from 3.2.0 to 3.3.0 ({gh-pr}`370`)
@@ -19,6 +22,7 @@ Unreleased changes
 - Bump actions/checkout from 6 to 7 ({gh-pr}`383`)
 - pre-commit autoupdate ({gh-pr}`379`)
 - Fix the test suite's compatibility with the latest pytest release ({gh-pr}`384`)
+- Use the official `astral-sh/setup-uv` action to install and cache `uv` in CI ({gh-pr}`386`)
 
 ---
 

pytest.ini

@@ -1,19 +1,17 @@
 [pytest]
 addopts =
     # Tests summary report
-    # ref: https://docs.pytest.org/en/8.3.x/how-to/output.html#producing-a-detailed-summary-report
+    # ref: https://docs.pytest.org/en/stable/how-to/output.html#producing-a-detailed-summary-report
     -ra
     # (V)Verbose
     -vv
     # Show local variables in tracebacks
     --showlocals
-    # Raise error on unknown markers
-    --strict-markers
     # Import mode
-    # ref: https://docs.pytest.org/en/8.3.x/explanation/pythonpath.html#import-modes
+    # ref: https://docs.pytest.org/en/stable/explanation/pythonpath.html#import-modes
     --import-mode=importlib
     # Minimal duration in seconds for inclusion in slowest list
-    # ref: https://docs.pytest.org/en/8.3.x/how-to/usage.html#profiling-test-execution-duration
+    # ref: https://docs.pytest.org/en/stable/how-to/usage.html#profiling-test-execution-duration
     --durations-min=0.5
     # Fail if a test tries to open a network connection
     # ref: https://github.com/miketheman/pytest-socket
@@ -25,22 +23,25 @@ addopts =
     --no-cov-on-fail
     --cov-report=term-missing
 
+# Enable all of pytest's strictness options (requires pytest>=9).
+# As of pytest 9.1, this enables: strict_config, strict_markers,
+# strict_parametrization_ids, and strict_xfail. Any new strictness
+# options added in future pytest releases will also be picked up here.
+# ref: https://docs.pytest.org/en/stable/explanation/goodpractices.html#using-pytest-s-strict-mode
+# ref: https://docs.pytest.org/en/stable/reference/reference.html#confval-strict
+strict = true
+
 # Default list of directories that pytest will search for tests in.
 # This should be overridden in tox.ini to run specific test suites.
-# https://docs.pytest.org/en/8.3.x/reference/reference.html#confval-testpaths
+# https://docs.pytest.org/en/stable/reference/reference.html#confval-testpaths
 testpaths = tests
 
 # List of directories that should be added to the python search path
-# https://docs.pytest.org/en/8.3.x/reference/reference.html#confval-pythonpath
+# https://docs.pytest.org/en/stable/reference/reference.html#confval-pythonpath
 pythonpath = cicd_utils
 
-# Tests marked with @pytest.mark.xfail that actually
-# succeed will by default fail the test suite.
-# https://docs.pytest.org/en/8.3.x/reference/reference.html#confval-xfail_strict
-xfail_strict = true
-
 # Warning filters pattern: action:message:category:module:line
-# ref: https://docs.pytest.org/en/8.3.x/how-to/capture-warnings.html
+# ref: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
 # ref: https://docs.python.org/3/library/warnings.html#describing-warning-filters
 # ref: https://docs.python.org/3/using/cmdline.html#cmdoption-W
 filterwarnings =

requirements/tests.txt

@@ -1,5 +1,6 @@
 # pytest and plugins
-pytest
+# (pytest>=9 is needed for the `strict` config option set in pytest.ini)
+pytest>=9
 pytest-icdiff
 pytest-socket
 

tests/e2e/test_examples.py

@@ -13,7 +13,10 @@
 
 
 def test_paths_exist() -> None:
-    assert PATH_ROOT.name == "ridgeplot"
+    # Sanity check that PATH_ROOT points at the repo root, without
+    # depending on the name of the checkout directory (which differs
+    # in git worktrees and renamed clones)
+    assert (PATH_ROOT / "pyproject.toml").is_file()
     assert PATH_ARTIFACTS.is_dir()
     assert PATH_CHARTS.is_dir()
 

tests/unit/test_hist.py

@@ -39,7 +39,11 @@ def test_bin_trace_samples_nbins(nbins: int) -> None:
     assert len(density_trace) == nbins
 
 
-@pytest.mark.parametrize("non_finite_value", [np.inf, np.nan, float("inf"), float("nan")])
+@pytest.mark.parametrize(
+    "non_finite_value",
+    [np.inf, np.nan, float("inf"), float("nan")],
+    ids=["np-inf", "np-nan", "float-inf", "float-nan"],
+)
 def test_bin_trace_samples_fails_for_non_finite_values(non_finite_value: float) -> None:
     err_msg = "The samples array should not contain any infs or NaNs."
     with pytest.raises(ValueError, match=err_msg):
@@ -64,7 +68,11 @@ def test_bin_trace_samples_weights_not_same_length() -> None:
         bin_trace_samples(trace_samples=SAMPLES_IN, nbins=NBINS, weights=[1, 1, 1])
 
 
-@pytest.mark.parametrize("non_finite_value", [np.inf, np.nan, float("inf"), float("nan")])
+@pytest.mark.parametrize(
+    "non_finite_value",
+    [np.inf, np.nan, float("inf"), float("nan")],
+    ids=["np-inf", "np-nan", "float-inf", "float-nan"],
+)
 def test_bin_trace_samples_weights_fails_for_non_finite_values(
     non_finite_value: float,
 ) -> None:

tests/unit/test_kde.py

@@ -49,7 +49,11 @@ def test_estimate_density_trace_points() -> None:
     assert np.argmin(y) == 0
 
 
-@pytest.mark.parametrize("non_finite_value", [np.inf, np.nan, float("inf"), float("nan")])
+@pytest.mark.parametrize(
+    "non_finite_value",
+    [np.inf, np.nan, float("inf"), float("nan")],
+    ids=["np-inf", "np-nan", "float-inf", "float-nan"],
+)
 def test_estimate_density_trace_fails_for_non_finite_values(non_finite_value: float) -> None:
     err_msg = "The samples array should not contain any infs or NaNs."
     with pytest.raises(ValueError, match=err_msg):
@@ -101,7 +105,11 @@ def test_estimate_density_trace_weights_not_same_length() -> None:
         )
 
 
-@pytest.mark.parametrize("non_finite_value", [np.inf, np.nan, float("inf"), float("nan")])
+@pytest.mark.parametrize(
+    "non_finite_value",
+    [np.inf, np.nan, float("inf"), float("nan")],
+    ids=["np-inf", "np-nan", "float-inf", "float-nan"],
+)
 def test_estimate_density_trace_weights_fails_for_non_finite_values(
     non_finite_value: float,
 ) -> None: