Add strict ini option to enable all other strictness options

The `--strict` option is undeprecated and enables `strict`.

Fix #13823.

Author: bluetech

changelog/13823.feature.rst

@@ -0,0 +1,9 @@
+Added a :confval:`strict` configuration option to enable all strictness-related options.
+
+When set to ``True``, the :confval:`strict` option currently enables :confval:`strict_config`,
+:confval:`strict_markers`, :confval:`strict_xfail`, and :confval:`strict_parametrization_ids`.
+
+The individual strictness options can be explicitly set to override the global :confval:`strict` setting.
+
+If new strictness options are added in the future, they will also be automatically enabled by :confval:`strict`.
+Therefore, we only recommend setting ``strict=True`` if you're using a locked version of pytest.

doc/en/deprecations.rst

@@ -589,18 +589,20 @@ removed in pytest 8 (deprecated since pytest 2.4.0):
 - ``parser.addoption(..., type="int/string/float/complex")`` - use ``type=int`` etc. instead.
 
 
-The ``--strict`` command-line option
-~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+The ``--strict`` command-line option (reintroduced)
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 .. deprecated:: 6.2
-.. versionremoved:: 8.0
+.. versionchanged:: 9.0
 
 The ``--strict`` command-line option has been deprecated in favor of ``--strict-markers``, which
 better conveys what the option does.
 
-We have plans to maybe in the future to reintroduce ``--strict`` and make it an encompassing
-flag for all strictness related options (``--strict-markers`` and ``--strict-config``
-at the moment, more might be introduced in the future).
+In version 8.1, we accidentally un-deprecated ``--strict``.
+
+In version 9.0, we changed ``--strict`` to make it set the new :confval:`strict`
+configuration option. It now enables all strictness related options (including
+:confval:`strict_markers`).
 
 
 .. _cmdline-preparse-deprecated:

doc/en/reference/reference.rst

@@ -2066,6 +2066,25 @@ passed multiple times. The expected format is ``name=value``. For example::
     "auto" can be used to explicitly use the global verbosity level.
 
 
+.. confval:: strict
+
+    If set to ``True``, enables all strictness options: :confval:`strict_config`, :confval:`strict_markers`,
+    :confval:`strict_xfail`, and :confval:`strict_parametrization_ids`.
+
+    If you explicitly set an individual strictness option, it takes precedence over ``strict``.
+
+    .. note::
+        If new strictness options are added to pytest in the future, they will also be enabled by ``strict``.
+        We therefore only recommend using this option when using a locked version of pytest.
+
+    .. code-block:: ini
+
+        [pytest]
+        strict = True
+
+    .. versionadded:: 9.0
+
+
 .. confval:: strict_xfail
 
     If set to ``True``, tests marked with ``@pytest.mark.xfail`` that actually succeed will by default fail the
@@ -2078,6 +2097,8 @@ passed multiple times. The expected format is ``name=value``. For example::
         [pytest]
         strict_xfail = True
 
+    You can also enable this option via the :confval:`strict` option.
+
     .. versionchanged:: 9.0
         Renamed from ``xfail_strict`` to ``strict_xfail``.
         ``xfail_strict`` is accepted as an alias for ``strict_xfail``.
@@ -2092,6 +2113,8 @@ passed multiple times. The expected format is ``name=value``. For example::
         [pytest]
         strict_config = True
 
+    You can also enable this option via the :confval:`strict` option.
+
 
 .. confval:: strict_markers
 
@@ -2102,6 +2125,8 @@ passed multiple times. The expected format is ``name=value``. For example::
         [pytest]
         strict_markers = True
 
+    You can also enable this option via the :confval:`strict` option.
+
 
 .. confval:: strict_parametrization_ids
 
@@ -2115,6 +2140,8 @@ passed multiple times. The expected format is ``name=value``. For example::
           [pytest]
           strict_parametrization_ids = True
 
+    You can also enable this option via the :confval:`strict` option.
+
     For example,
 
     .. code-block:: python

pyproject.toml

@@ -378,10 +378,7 @@ norecursedirs = [
     "build",
     "dist",
 ]
-strict_xfail = true
-strict_parametrization_ids = true
-strict_markers = true
-strict_config = true
+strict = true
 filterwarnings = [
     "error",
     "default:Using or importing the ABCs:DeprecationWarning:unittest2.*",

src/_pytest/config/__init__.py

@@ -1515,7 +1515,11 @@ def _validate_plugins(self) -> None:
             )
 
     def _warn_or_fail_if_strict(self, message: str) -> None:
-        strict_config = self.getini("strict_config")
+        if self.hasini("strict_config"):
+            strict_config = self.getini("strict_config")
+        else:
+            strict_config = self.getini("strict")
+
         if strict_config:
             raise UsageError(message)
 

src/_pytest/main.py

@@ -90,8 +90,10 @@ def pytest_addoption(parser: Parser) -> None:
     )
     group.addoption(
         "--strict",
-        action="store_true",
-        help="(Deprecated) alias to --strict-markers",
+        action=OverrideIniAction,
+        ini_option="strict",
+        ini_value="true",
+        help="Enables the strict option",
     )
     parser.addini(
         "strict_config",
@@ -107,6 +109,13 @@ def pytest_addoption(parser: Parser) -> None:
         type="bool",
         default=False,
     )
+    parser.addini(
+        "strict",
+        "Enables all strictness options, currently: "
+        "strict_config, strict_markers, strict_xfail, strict_parametrization_ids",
+        type="bool",
+        default=False,
+    )
 
     group = parser.getgroup("pytest-warnings")
     group.addoption(

src/_pytest/mark/structures.py

@@ -585,9 +585,10 @@ def __getattr__(self, name: str) -> MarkDecorator:
                     __tracebackhide__ = True
                     fail(f"Unknown '{name}' mark, did you mean 'parametrize'?")
 
-                strict_markers = (
-                    self._config.getini("strict_markers") or self._config.option.strict
-                )
+                if self._config.hasini("strict_markers"):
+                    strict_markers = self._config.getini("strict_markers")
+                else:
+                    strict_markers = self._config.getini("strict")
                 if strict_markers:
                     fail(
                         f"{name!r} not found in `markers` configuration option",

src/_pytest/python.py

@@ -964,7 +964,10 @@ def make_unique_parameterset_ids(self) -> list[str | _HiddenParam]:
 
     def _strict_parametrization_ids_enabled(self) -> bool:
         if self.config:
-            return bool(self.config.getini("strict_parametrization_ids"))
+            if self.config.hasini("strict_parametrization_ids"):
+                return bool(self.config.getini("strict_parametrization_ids"))
+            else:
+                return bool(self.config.getini("strict"))
         return False
 
     def _resolve_ids(self) -> Iterable[str | _HiddenParam]:

src/_pytest/skipping.py

@@ -214,7 +214,12 @@ def evaluate_xfail_marks(item: Item) -> Xfail | None:
     """Evaluate xfail marks on item, returning Xfail if triggered."""
     for mark in item.iter_markers(name="xfail"):
         run = mark.kwargs.get("run", True)
-        strict = mark.kwargs.get("strict", item.config.getini("strict_xfail"))
+        if "strict" in mark.kwargs:
+            strict = mark.kwargs["strict"]
+        elif item.config.hasini("strict_xfail"):
+            strict = item.config.getini("strict_xfail")
+        else:
+            strict = item.config.getini("strict")
         raises = mark.kwargs.get("raises", None)
         if "condition" not in mark.kwargs:
             conditions = mark.args

testing/test_collection.py

@@ -2726,15 +2726,17 @@ def test_1(): pass
         ),
     ],
 )
+@pytest.mark.parametrize("option_name", ["strict_parametrization_ids", "strict"])
 def test_strict_parametrization_ids(
     pytester: Pytester,
     x_y: Sequence[tuple[int, int]],
     expected_duplicates: Sequence[str],
+    option_name: str,
 ) -> None:
     pytester.makeini(
-        """
+        f"""
         [pytest]
-        strict_parametrization_ids = true
+        {option_name} = true
         """
     )
     pytester.makepyfile(