Merge pull request #97 from JoseKilo/fix/pytest-warning-calling-fixture-as-function

Fix: Avoid warning on latest Pytest versions

Author: Tinche

pytest_asyncio/plugin.py

@@ -174,25 +174,29 @@ def event_loop(request):
     loop.close()
 
 
-@pytest.fixture
-def unused_tcp_port():
+def _unused_tcp_port():
     """Find an unused localhost TCP port from 1024-65535 and return it."""
     with contextlib.closing(socket.socket()) as sock:
         sock.bind(('127.0.0.1', 0))
         return sock.getsockname()[1]
 
 
+@pytest.fixture
+def unused_tcp_port():
+    return _unused_tcp_port()
+
+
 @pytest.fixture
 def unused_tcp_port_factory():
     """A factory function, producing different unused TCP ports."""
     produced = set()
 
     def factory():
         """Return an unused port."""
-        port = unused_tcp_port()
+        port = _unused_tcp_port()
 
         while port in produced:
-            port = unused_tcp_port()
+            port = _unused_tcp_port()
 
         produced.add(port)
 

setup.cfg

@@ -7,6 +7,7 @@ show_missing = true
 [tool:pytest]
 addopts = -rsx --tb=short
 testpaths = tests
+filterwarnings = error
 
 [metadata]
 # ensure LICENSE is included in wheel metadata

tests/test_simple.py

@@ -104,7 +104,7 @@ def mock_unused_tcp_port():
         else:
             return 10000 + counter
 
-    monkeypatch.setattr(pytest_asyncio.plugin, 'unused_tcp_port',
+    monkeypatch.setattr(pytest_asyncio.plugin, '_unused_tcp_port',
                         mock_unused_tcp_port)
 
     assert unused_tcp_port_factory() == 10000