Use ruff lint.extend-select to keep ruff's default rules

Ruff 0.16 enables a much larger default rule set. 'lint.select' was
replacing it entirely; 'lint.extend-select' keeps the defaults and adds
our own selection on top.

Fix the new violations:
- RUF022: sort __all__
- RUF046: drop redundant int() around math.floor()
- RUF059: use _ for unused unpacked variables
- PLW0127: remove self-assignment of scenario_name
- TRY002: raise pytest.UsageError instead of a bare Exception for
  gherkin-terminal-reporter incompatibilities

Author: Pierre-Sassoulas

CHANGES.rst

@@ -16,6 +16,7 @@ Changed
 +++++++
 * Relaxed `gherkin-official` dependency requirement to `>=29.0.0` to allow for newer versions of the `gherkin-official` package.
 * Excluded `gherkin-official` `31.0.0` and `32.0.0`, which crash with ``StopIteration`` when parsing empty descriptions (fixed upstream in `32.0.1`).
+* The ``--gherkin-terminal-reporter`` incompatibility errors (with another terminal reporter or with ``xdist``) are now raised as ``pytest.UsageError`` instead of a bare ``Exception``.
 
 Deprecated
 ++++++++++

pyproject.toml

@@ -76,7 +76,7 @@ build-backend = "uv_build"
 
 [tool.ruff]
 line-length = 120
-lint.select = [
+lint.extend-select = [
     "B",     # flake8-bugbear
     "BLE",   # flake8-blind-except
     "C4",    # flake8-comprehensions

src/pytest_bdd/__init__.py

@@ -5,4 +5,4 @@
 from pytest_bdd.scenario import scenario, scenarios
 from pytest_bdd.steps import given, step, then, when
 
-__all__ = ["given", "when", "step", "then", "scenario", "scenarios"]
+__all__ = ["given", "scenario", "scenarios", "step", "then", "when"]

src/pytest_bdd/cucumber_json.py

@@ -124,7 +124,7 @@ def _get_result(self, step: StepReportDict, report: TestReport, error_message: b
             status = "skipped"
         else:
             raise ValueError(f"Unknown test outcome {report.outcome}")
-        res: ResultElementDict = {"status": status, "duration": int(math.floor((10**9) * step["duration"]))}  # nanosec
+        res: ResultElementDict = {"status": status, "duration": math.floor((10**9) * step["duration"])}  # nanosec
         if res_message is not None:
             res["error_message"] = res_message
         return res

src/pytest_bdd/generation.py

@@ -187,7 +187,7 @@ def _show_missing_code_main(config: Config, session: Session) -> None:
         session.exitstatus = 100
         return
 
-    features, scenarios, steps = parse_feature_files(config.option.features)
+    _, scenarios, steps = parse_feature_files(config.option.features)
 
     for item in session.items:
         if not isinstance(item, Function):

src/pytest_bdd/gherkin_terminal_reporter.py

@@ -2,6 +2,7 @@
 
 import typing
 
+import pytest
 from _pytest.terminal import TerminalReporter
 
 from .reporting import test_report_context_registry
@@ -28,7 +29,7 @@ def configure(config: Config) -> None:
         # Get the standard terminal reporter plugin and replace it with our
         current_reporter = config.pluginmanager.getplugin("terminalreporter")
         if current_reporter.__class__ != TerminalReporter:
-            raise Exception(
+            raise pytest.UsageError(
                 "gherkin-terminal-reporter is not compatible with any other terminal reporter."
                 "You can use only one terminal reporter."
                 f"Currently '{current_reporter.__class__}' is used."
@@ -39,7 +40,7 @@ def configure(config: Config) -> None:
         config.pluginmanager.unregister(current_reporter)
         config.pluginmanager.register(gherkin_reporter, "terminalreporter")
         if config.pluginmanager.getplugin("dsession"):
-            raise Exception("gherkin-terminal-reporter is not compatible with 'xdist' plugin.")
+            raise pytest.UsageError("gherkin-terminal-reporter is not compatible with 'xdist' plugin.")
 
 
 class GherkinTerminalReporter(TerminalReporter):  # type: ignore[misc]

src/pytest_bdd/scenario.py

@@ -372,7 +372,6 @@ def scenario(
     :param features_base_dir: Optional base dir location for locating feature files. If not set, it will try and resolve using property set in .ini file, then the caller_module_path.
     """
     __tracebackhide__ = True
-    scenario_name = scenario_name
     caller_module_path = get_caller_module_path()
 
     # Get the feature

tests/scripts/test_generate.py

@@ -35,7 +35,7 @@ def test_generate(pytester, monkeypatch, capsys):
 
     monkeypatch.setattr(sys, "argv", ["", "generate", str(feature)])
     main()
-    out, err = capsys.readouterr()
+    out, _ = capsys.readouterr()
     assert out == textwrap.dedent(
         '''\
     """Code generation feature tests."""

tests/scripts/test_main.py

@@ -16,7 +16,7 @@ def test_main(monkeypatch, capsys):
     monkeypatch.setattr(sys, "argv", ["pytest-bdd"])
     monkeypatch.setattr(sys, "exit", lambda x: x)
     main()
-    out, err = capsys.readouterr()
+    _, err = capsys.readouterr()
     assert "usage: pytest-bdd [-h]" in err
     assert "pytest-bdd: error:" in err
 

tests/scripts/test_migrate.py

@@ -28,7 +28,7 @@ def test_migrate(monkeypatch, capsys, pytester):
 
     monkeypatch.setattr(sys, "argv", ["", "migrate", str(tests)])
     main()
-    out, err = capsys.readouterr()
+    out, _ = capsys.readouterr()
     out = "\n".join(sorted(out.splitlines()))
     expected = textwrap.dedent(
         """