Merge branch 'main' into 219-setup-uv-action

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 =

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`)

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:

tests/unit/test_utils.py

@@ -44,7 +44,7 @@ def test_raise_for_non_2d_array(self) -> None:
 
     @pytest.mark.parametrize(
         ("densities_type", "rows_type"),
-        product((id_func, tuple, list), (id_func, tuple, list, np.asarray)),
+        list(product((id_func, tuple, list), (id_func, tuple, list, np.asarray))),
     )
     def test_expected_output(
         self,