Fix CI test failures from incomplete dummy config and encoding
8 test files had _DummyConfigManager missing get_active_media_server(), causing failures when pytest ran them before the test file that had it. Whichever file set sys.modules first won, and the incomplete dummy broke later tests. Also fix script.js read_text() missing encoding='utf-8' which failed on non-UTF-8 default locales.
This commit is contained in:
parent
12087f2407
commit
c940363ec2
9 changed files with 25 additions and 1 deletions
|
|
@ -30,6 +30,9 @@ if "config.settings" not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_pkg.settings = settings_mod
|
||||
sys.modules["config"] = config_pkg
|
||||
|
|
|
|||
|
|
@ -28,6 +28,9 @@ if 'config.settings' not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_mod.settings = settings_mod
|
||||
sys.modules['config'] = config_mod
|
||||
|
|
|
|||
|
|
@ -29,6 +29,9 @@ if 'config.settings' not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_mod.settings = settings_mod
|
||||
sys.modules['config'] = config_mod
|
||||
|
|
|
|||
|
|
@ -32,6 +32,9 @@ if "config.settings" not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_pkg.settings = settings_mod
|
||||
sys.modules["config"] = config_pkg
|
||||
|
|
|
|||
|
|
@ -29,6 +29,9 @@ if 'config.settings' not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_mod.settings = settings_mod
|
||||
sys.modules['config'] = config_mod
|
||||
|
|
|
|||
|
|
@ -33,6 +33,9 @@ if "config.settings" not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_pkg.settings = settings_mod
|
||||
sys.modules["config"] = config_pkg
|
||||
|
|
|
|||
|
|
@ -28,6 +28,9 @@ if 'config.settings' not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_mod.settings = settings_mod
|
||||
sys.modules['config'] = config_mod
|
||||
|
|
|
|||
|
|
@ -30,6 +30,9 @@ if "config.settings" not in sys.modules:
|
|||
def get(self, key, default=None):
|
||||
return default
|
||||
|
||||
def get_active_media_server(self):
|
||||
return "plex"
|
||||
|
||||
settings_mod.config_manager = _DummyConfigManager()
|
||||
config_pkg.settings = settings_mod
|
||||
sys.modules["config"] = config_pkg
|
||||
|
|
|
|||
|
|
@ -3,6 +3,6 @@ from pathlib import Path
|
|||
|
||||
def test_websocket_client_prefers_polling_before_websocket():
|
||||
script_path = Path(__file__).resolve().parents[1] / "webui" / "static" / "script.js"
|
||||
script = script_path.read_text()
|
||||
script = script_path.read_text(encoding="utf-8")
|
||||
|
||||
assert "transports: ['polling', 'websocket']" in script
|
||||
|
|
|
|||
Loading…
Reference in a new issue