Tests: auto-patch to avoid loading user/site configs

cylc/uiserver/app.py

@@ -145,15 +145,6 @@ class CylcUIServer(ExtensionApp):
     cylc gui --no-browser     # Start the server but don't open the browser
 
     ''')  # type: ignore[assignment]
-    config_file_paths = get_conf_dir_hierarchy(
-        [
-            SITE_CONF_ROOT,  # site configuration
-            USER_CONF_ROOT,  # user configuration
-        ], filename=False
-    )
-    # Next include currently needed for directory making
-    config_file_paths.insert(0, str(Path(uis_pkg).parent))  # packaged config
-    config_file_paths.reverse()
     # TODO: Add a link to the access group table mappings in cylc documentation
     # https://github.com/cylc/cylc-uiserver/issues/466
     AUTH_DESCRIPTION = '''
@@ -348,6 +339,19 @@ class CylcUIServer(ExtensionApp):
         default_value=False,
     )
 
+    @property
+    def config_file_paths(self):
+        ret = get_conf_dir_hierarchy(
+            [
+                SITE_CONF_ROOT,  # site configuration
+                USER_CONF_ROOT,  # user configuration
+            ], filename=False
+        )
+        # Next include currently needed for directory making
+        ret.insert(0, str(Path(uis_pkg).parent))  # packaged config
+        ret.reverse()
+        return ret
+
     @validate('ui_build_dir')
     def _check_ui_build_dir_exists(self, proposed):
         if proposed['value'].exists():

cylc/uiserver/tests/conftest.py

@@ -14,17 +14,14 @@
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 """Test code and fixtures."""
 
-from getpass import getuser
 import inspect
 import logging
 from pathlib import Path
 from shutil import rmtree
-from socket import gethostname
 from tempfile import TemporaryDirectory
 from uuid import uuid4
 
 import pytest
-from tornado.web import HTTPError
 from traitlets.config import Config
 import zmq
 
@@ -47,6 +44,7 @@
 from cylc.flow.parsec.config import ParsecConfig
 from cylc.flow.parsec.validate import cylc_config_validate
 
+
 class AsyncClientFixture(WorkflowRuntimeClient):
     pattern = zmq.REQ
     host = ''
@@ -223,7 +221,7 @@ def mock_authentication_yossarian(monkeypatch):
 def jp_server_config(jp_template_dir):
     """Config to turn the CylcUIServer extension on.
 
-    Auto-loading, add as an argument in the test function to activate.
+    Overrides jupyter server fixture of the same name.
     """
     config = {
         "ServerApp": {
@@ -235,12 +233,9 @@ def jp_server_config(jp_template_dir):
     return config
 
 
-@pytest.fixture
-def patch_conf_files(monkeypatch):
-    """Auto-patches the CylcUIServer to prevent it loading config files.
-
-    Auto-loading, add as an argument in the test function to activate.
-    """
+@pytest.fixture(autouse=True)
+def patch_conf_files(monkeypatch: pytest.MonkeyPatch):
+    """Auto-patches the CylcUIServer to prevent it loading config files."""
     monkeypatch.setattr(
         'cylc.uiserver.app.CylcUIServer.config_file_paths', []
     )
@@ -364,6 +359,7 @@ def workflow_run_dir(request):
     if not request.session.testsfailed:
         rmtree(run_dir)
 
+
 @pytest.fixture
 def mock_glbl_cfg(tmp_path: Path, monkeypatch: pytest.MonkeyPatch):
     """A Pytest fixture for fiddling global config values.

cylc/uiserver/tests/test_auth.py

@@ -23,7 +23,7 @@
 
 @pytest.mark.integration
 @pytest.mark.usefixtures("mock_authentication_yossarian")
-async def test_cylc_handler(patch_conf_files, jp_fetch):
+async def test_cylc_handler(jp_fetch):
     """The Cylc endpoints have been added and work."""
     resp = await jp_fetch(
         'cylc', 'userprofile', method='GET'
@@ -60,7 +60,6 @@ async def test_cylc_handler(patch_conf_files, jp_fetch):
     ]
 )
 async def test_authorised_and_authenticated(
-    patch_conf_files,
     jp_fetch,
     endpoint,
     code,
@@ -95,7 +94,6 @@ async def test_authorised_and_authenticated(
     ]
 )
 async def test_unauthorised(
-    patch_conf_files,
     jp_fetch,
     endpoint,
     code,