Do not auto update fixtures from return to yield statements

Author: schloerke

ruff.toml

@@ -14,6 +14,7 @@ extend-exclude = [
 extend-ignore = [
   "E501",  # E501: Line too long
   "PT011", # PT011 `pytest.raises(ValueError)` is too broad
+  "PT022", # PT022 [*] No teardown in fixture
   # Conflicting lint rules: https://docs.astral.sh/ruff/formatter/#conflicting-lint-rules
   "E111",   # * indentation-with-invalid-multiple (E111)
   "E114",   # * indentation-with-invalid-multiple-comment (E114)

tests/playwright/conftest.py

@@ -234,7 +234,7 @@ def fixture_func():
         # Pass through `yield` via `next(...)` call
         # (`yield` must be on same line as `next`!)
         app_gen = local_app_fixture_gen(app)
-        return next(app_gen)
+        yield next(app_gen)
 
     return fixture_func
 
@@ -285,9 +285,9 @@ def x_create_doc_example_fixture(example_name: str, scope: ScopeName = "module")
 
 
 @pytest.fixture(scope="module")
-def local_app(request: pytest.FixtureRequest) -> ShinyAppProc:
+def local_app(request: pytest.FixtureRequest) -> Generator[ShinyAppProc, None, None]:
     app_gen = local_app_fixture_gen(PurePath(request.path).parent / "app.py")
-    return next(app_gen)
+    yield next(app_gen)
 
 
 @contextmanager