ED/EDMarketConnector
1
0
Fork 0
mirror of https://github.com/EDCD/EDMarketConnector.git synced 2025-04-14 08:17:13 +03:00
Code Issues Projects Releases Wiki Activity

make tests not explode on windows

Browse Source
This commit is contained in:
A_D 2022-01-26 20:36:29 +02:00
parent 7bcf3a6f4d
commit 8c2a0ae95a
No known key found for this signature in database
GPG Key ID: 4BE9EB7DF45076C4
2 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
config/linux.py
View File

@ -3,7 +3,7 @@ import os
import pathlib import pathlib
import sys import sys
from configparser import ConfigParser from configparser import ConfigParser
from typing import List, Optional, Union from typing import TYPE_CHECKING, List, Optional, Union
from config import AbstractConfig, appname, logger from config import AbstractConfig, appname, logger

21
tests/config.py/test_config.py
View File

@ -1,4 +1,12 @@
"""Test the config system.""" """
Test the config system.
Note: These tests to arbitrary reads and writes to an existing config, including
key deletions. Said modifications are to keys that are generated internally.
Most of these tests are parity tests with the "old" config, and likely one day can be
entirely removed.
"""
from __future__ import annotations from __future__ import annotations
import contextlib import contextlib
@ -19,7 +27,7 @@ print(sys.path)
from _old_config import old_config # noqa: E402 from _old_config import old_config # noqa: E402
from config import LinuxConfig, config # noqa: E402 from config import config # noqa: E402
def _fuzz_list(length: int) -> List[str]: def _fuzz_list(length: int) -> List[str]:
@ -77,6 +85,11 @@ class TestNewConfig:
def __update_linuxconfig(self) -> None: def __update_linuxconfig(self) -> None:
"""On linux config uses ConfigParser, which doesn't update from disk changes. Force the update here.""" """On linux config uses ConfigParser, which doesn't update from disk changes. Force the update here."""
if sys.platform != 'linux':
return
from config.linux import LinuxConfig
if isinstance(config, LinuxConfig) and config.config is not None: if isinstance(config, LinuxConfig) and config.config is not None:
config.config.read(config.filename) config.config.read(config.filename)
@ -163,6 +176,10 @@ class TestOldNewConfig:
def __update_linuxconfig(self) -> None: def __update_linuxconfig(self) -> None:
"""On linux config uses ConfigParser, which doesn't update from disk changes. Force the update here.""" """On linux config uses ConfigParser, which doesn't update from disk changes. Force the update here."""
if sys.platform != 'linux':
return
from config.linux import LinuxConfig
if isinstance(config, LinuxConfig) and config.config is not None: if isinstance(config, LinuxConfig) and config.config is not None:
config.config.read(config.filename) config.config.read(config.filename)