From: Tom Krizek <tkrizek@isc.org>
Date: Mon, 27 Nov 2023 12:28:27 +0000 (+0100)
Subject: Simplify statsport fixture in statschannel test
X-Git-Tag: v9.19.19~14^2~2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=f0481c8600b0752f9d9029b3d20ddb85976fa9da;p=thirdparty%2Fbind9.git

Simplify statsport fixture in statschannel test

The ports fixture provides the required port numbers already and there's
no need to get the port number from environment.
---

diff --git a/bin/tests/system/statschannel/conftest.py b/bin/tests/system/statschannel/conftest.py
index 363dd7ad1bf..c26935b0ab2 100644
--- a/bin/tests/system/statschannel/conftest.py
+++ b/bin/tests/system/statschannel/conftest.py
@@ -9,17 +9,9 @@
 # See the COPYRIGHT file distributed with this work for additional
 # information regarding copyright ownership.
 
-import os
 import pytest
 
 
-@pytest.fixture
-def statsport(request):
-    # pylint: disable=unused-argument
-    env_port = os.getenv("EXTRAPORT1")
-    if env_port is None:
-        env_port = 5301
-    else:
-        env_port = int(env_port)
-
-    return env_port
+@pytest.fixture(scope="module")
+def statsport(ports):
+    return ports["EXTRAPORT1"]