fixup: attempt to stabilise tests around noproc

Author: fizyk

README.rst

@@ -101,6 +101,11 @@ You can create additional fixtures using factories:
 .. note::
 
     Each process fixture can be configured independently through factory arguments.
+    When using chained fixtures (``postgresql_noproc(depends_on=...)``), the
+    factory injects the dependency name into the runtime signature so
+    pytest/xdist can order setup/teardown correctly. This may not show up in
+    ``--setup-show`` output, and some older pytest versions do not reflect the
+    dynamic signature when displaying fixtures.
 
 Pre-populating the database for tests
 -------------------------------------

pytest_postgresql/factories/client.py

@@ -17,8 +17,7 @@
 # along with pytest-postgresql.  If not, see <http://www.gnu.org/licenses/>.
 """Fixture factory for postgresql client."""
 
-import inspect
-from typing import Any, Callable, Iterator
+from typing import Callable, Iterator
 
 import psycopg
 import pytest
@@ -46,7 +45,7 @@ def postgresql(
     """
 
     @pytest.fixture
-    def postgresql_factory(request: FixtureRequest, **kwargs: Any) -> Iterator[Connection]:
+    def postgresql_factory(request: FixtureRequest) -> Iterator[Connection]:
         """Fixture factory for PostgreSQL.
 
         :param request: fixture request object
@@ -85,11 +84,4 @@ def postgresql_factory(request: FixtureRequest, **kwargs: Any) -> Iterator[Conne
             yield db_connection
             db_connection.close()
 
-    # Expose process fixture dependency to pytest for correct teardown ordering.
-    postgresql_factory.__signature__ = inspect.Signature(  # type: ignore[attr-defined]
-        parameters=[
-            inspect.Parameter("request", inspect.Parameter.POSITIONAL_OR_KEYWORD),
-            inspect.Parameter(process_fixture_name, inspect.Parameter.POSITIONAL_OR_KEYWORD),
-        ]
-    )
     return postgresql_factory

pytest_postgresql/factories/noprocess.py

@@ -17,6 +17,7 @@
 # along with pytest-postgresql.  If not, see <http://www.gnu.org/licenses/>.
 """Fixture factory for existing postgresql server."""
 
+import inspect
 import os
 from pathlib import Path
 from typing import Callable, Iterator
@@ -121,4 +122,12 @@ def postgresql_noproc_fixture(request: FixtureRequest) -> Iterator[NoopExecutor]
                 janitor.load(load_element)
             yield noop_exec
 
+    if depends_on and depends_on.isidentifier():
+        postgresql_noproc_fixture.__signature__ = inspect.Signature(  # type: ignore[attr-defined]
+            parameters=[
+                inspect.Parameter("request", inspect.Parameter.POSITIONAL_OR_KEYWORD),
+                inspect.Parameter(depends_on, inspect.Parameter.POSITIONAL_OR_KEYWORD),
+            ]
+        )
+
     return postgresql_noproc_fixture