diff --git a/pytest_asyncio/plugin.py b/pytest_asyncio/plugin.py index 1cf4b0b7..584efbf4 100644 --- a/pytest_asyncio/plugin.py +++ b/pytest_asyncio/plugin.py @@ -174,14 +174,18 @@ 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.""" @@ -189,10 +193,10 @@ def unused_tcp_port_factory(): 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) diff --git a/setup.cfg b/setup.cfg index 9a120472..4a0e3005 100644 --- a/setup.cfg +++ b/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 diff --git a/tests/test_simple.py b/tests/test_simple.py index 9cf378a4..f9e3992f 100644 --- a/tests/test_simple.py +++ b/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