update to pytest 9.0, remove pytest-subtests (#5271)

* update to pytest 9.0, remove pytest-subtests

* update docs

Author: valentinsulzer

docs/source/user_guide/installation/index.rst

@@ -158,9 +158,8 @@ Dependency
 `pre-commit <https://pre-commit.com/index.html>`__                               \-                 dev                For managing and maintaining multi-language pre-commit hooks.
 `ruff <https://beta.ruff.rs/docs/>`__                                            \-                 dev                For code formatting.
 `nox <https://nox.thea.codes/en/stable/>`__                                      \-                 dev                For running testing sessions in multiple environments.
-`pytest-subtests <https://github.com/pytest-dev/pytest-subtests/>`__             \-                 dev                For subtests pytest fixture.
 `pytest-cov <https://pytest-cov.readthedocs.io/en/stable/>`__                    \-                 dev                For calculating test coverage.
-`pytest <https://docs.pytest.org/en/stable/>`__                                  6.0.0              dev                For running the test suites.
+`pytest <https://docs.pytest.org/en/stable/>`__                                  9.0.0              dev                For running the test suites (includes subtests support).
 `pytest-doctestplus <https://github.com/scientific-python/pytest-doctestplus>`__ \-                 dev                For running doctests.
 `pytest-xdist <https://pytest-xdist.readthedocs.io/en/latest/>`__                \-                 dev                For running tests in parallel across distributed workers.
 `pytest-mock <https://pytest-mock.readthedocs.io/en/latest/index.html>`__        \-                 dev                Provides a mocker fixture.

pyproject.toml

@@ -1,19 +1,16 @@
 [build-system]
-requires = [
-    "hatchling>=1.13.0",
-    "hatch-vcs>=0.3.0",
-]
+requires = ["hatchling>=1.13.0", "hatch-vcs>=0.3.0"]
 build-backend = "hatchling.build"
 
 [project]
 name = "pybamm"
 dynamic = ["version"]
 license = { file = "LICENSE.txt" }
 description = "Python Battery Mathematical Modelling"
-authors = [{name = "The PyBaMM Team", email = "[email protected]"}]
-maintainers = [{name = "The PyBaMM Team", email = "[email protected]"}]
+authors = [{ name = "The PyBaMM Team", email = "[email protected]" }]
+maintainers = [{ name = "The PyBaMM Team", email = "[email protected]" }]
 requires-python = ">=3.10, <3.13"
-readme = {file = "README.md", content-type = "text/markdown"}
+readme = { file = "README.md", content-type = "text/markdown" }
 classifiers = [
     "Development Status :: 5 - Production/Stable",
     "Intended Audience :: Developers",
@@ -70,26 +67,18 @@ docs = [
     "sphinx-docsearch",
 ]
 # For example notebooks
-examples = [
-    "jupyter",
-]
+examples = ["jupyter"]
 plot = [
     # Note: matplotlib is loaded for debug plots, but to ensure PyBaMM runs
     # on systems without an attached display, it should never be imported
     # outside of plot() methods.
     "matplotlib>=3.6.0",
 ]
-cite = [
-    "pybtex>=0.25.0",
-]
+cite = ["pybtex>=0.25.0"]
 # Battery Parameter eXchange format
-bpx = [
-    "bpx>=0.5.0,<0.6.0",
-]
+bpx = ["bpx>=0.5.0,<0.6.0"]
 # Low-overhead progress bars
-tqdm = [
-    "tqdm",
-]
+tqdm = ["tqdm"]
 dev = [
     # For working with pre-commit hooks
     "pre-commit",
@@ -102,20 +91,17 @@ dev = [
     # For doctest
     "pytest-doctestplus",
     # pytest and its plugins
-    "pytest",
+    "pytest>=9.0",
     "pytest-xdist",
     "pytest-mock",
-    "pytest-subtests",
     # For testing Jupyter notebooks
     "nbmake",
     # To access the metadata for python packages
     "importlib-metadata; python_version < '3.10'",
     # For property based testing
     "hypothesis",
 ]
-jax = [
-    "jax>=0.4.36,<0.7.0",
-]
+jax = ["jax>=0.4.36,<0.7.0"]
 # Contains all optional dependencies, except for jax, and dev dependencies
 all = [
     "scikit-fem>=8.1.0",
@@ -155,10 +141,7 @@ Yang2017 = "pybamm.models.full_battery_models.lithium_ion.Yang2017:Yang2017"
 [tool.hatch]
 version.source = "vcs"
 version.fallback-version = "0.0.0"
-build.targets.sdist.include = [
-  "src/pybamm",
-  "CITATION.cff",
-]
+build.targets.sdist.include = ["src/pybamm", "CITATION.cff"]
 build.hooks.vcs.version-file = "src/pybamm/_version.py"
 
 [tool.repo-review]
@@ -178,8 +161,8 @@ extend-exclude = ["__init__.py"]
 
 [tool.ruff.lint]
 extend-select = [
-    "B",             # flake8-bugbear
-    "I",             # isort
+    "B", # flake8-bugbear
+    "I", # isort
     # "ARG",         # flake8-unused-arguments
     # "C4",          # flake8-comprehensions
     # "ICN",         # flake8-import-conventions
@@ -190,31 +173,31 @@ extend-select = [
     # "PT",          # flake8-pytest-style
     # "PTH",         # flake8-use-pathlib
     # "RET",         # flake8-return
-    "RUF",           # Ruff-specific
+    "RUF", # Ruff-specific
     # "SIM",         # flake8-simplify
     # "T20",         # flake8-print
-    "UP",            # pyupgrade
-    "YTT",           # flake8-2020
-    "TID252",        # relative-imports
-    "S101",          # to identify use of assert statement
-    "PT027",         # remove unittest style assertion
-    "PT009",         # Use pytest.raises instead of unittest-style
+    "UP",     # pyupgrade
+    "YTT",    # flake8-2020
+    "TID252", # relative-imports
+    "S101",   # to identify use of assert statement
+    "PT027",  # remove unittest style assertion
+    "PT009",  # Use pytest.raises instead of unittest-style
 ]
 ignore = [
-    "E741",        # Ambiguous variable name
-    "RUF012",      # Mutable class attributes should be annotated with `typing.ClassVar`
-    "SIM108",      # Use ternary operator
-    "ARG001",      # Unused function argument:
-    "ARG002",      # Unused method arguments
-    "PLR2004",     # Magic value used in comparison
-    "PLR0915",     # Too many statements
-    "PLR0913",     # Too many arguments
-    "PLR0912",     # Too many branches
-    "RET504",      # Unnecessary assignment
-    "RET505",      # Unnecessary `else`
-    "RET506",      # Unnecessary `elif`
-    "B018",        # Found useless expression
-    "RUF002",      # Docstring contains ambiguous
+    "E741",    # Ambiguous variable name
+    "RUF012",  # Mutable class attributes should be annotated with `typing.ClassVar`
+    "SIM108",  # Use ternary operator
+    "ARG001",  # Unused function argument:
+    "ARG002",  # Unused method arguments
+    "PLR2004", # Magic value used in comparison
+    "PLR0915", # Too many statements
+    "PLR0913", # Too many arguments
+    "PLR0912", # Too many branches
+    "RET504",  # Unnecessary assignment
+    "RET505",  # Unnecessary `else`
+    "RET506",  # Unnecessary `elif`
+    "B018",    # Found useless expression
+    "RUF002",  # Docstring contains ambiguous
 ]
 
 [tool.ruff.lint.per-file-ignores]
@@ -225,21 +208,10 @@ ignore = [
 "docs/source/examples/notebooks/models/lithium-plating.ipynb" = ["F821"]
 
 [tool.pytest.ini_options]
-minversion = "8"
-required_plugins = [
-    "pytest-xdist",
-    "pytest-mock",
-    "pytest-subtests",
-]
-addopts = [
-    "-nauto",
-    "-vra",
-    "--strict-config",
-    "--strict-markers",
-]
-testpaths = [
-    "tests",
-]
+minversion = "9.0"
+required_plugins = ["pytest-xdist", "pytest-mock"]
+addopts = ["-nauto", "-vra", "--strict-config", "--strict-markers"]
+testpaths = ["tests"]
 console_output_style = "progress"
 xfail_strict = true
 filterwarnings = [
@@ -254,7 +226,7 @@ filterwarnings = [
 ]
 
 # Logging configuration
-log_cli = "true"
+log_cli = true
 log_cli_level = "INFO"
 log_format = "%(asctime)s - %(name)s - %(levelname)s - %(message)s"
 log_date_format = "%Y-%m-%d %H:%M:%S"
@@ -273,7 +245,7 @@ disable_error_code = ["call-overload", "operator"]
 
 [[tool.mypy.overrides]]
 module = [
-        "pybamm.models.base_model.*",
-        "pybamm.models.full_battery_models.base_battery_model.*"
-        ]
+    "pybamm.models.base_model.*",
+    "pybamm.models.full_battery_models.base_battery_model.*",
+]
 disable_error_code = "attr-defined"