Skip to content

Commit 9367d18

Browse files
committed
tests: config generaton allow multiple needles
1 parent 5257e3b commit 9367d18

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

tests/config/test_load.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939

4040
class ConfigLoadingFileTestCase(ConfigFileTestCase):
4141
def test_load_fails_if_server_name_missing(self) -> None:
42-
self.generate_config_and_remove_lines_containing("server_name")
42+
self.generate_config_and_remove_lines_containing(["server_name"])
4343
with self.assertRaises(ConfigError):
4444
HomeServerConfig.load_config("", ["-c", self.config_file])
4545
with self.assertRaises(ConfigError):
@@ -76,7 +76,7 @@ def test_generates_and_loads_macaroon_secret_key(self) -> None:
7676
)
7777

7878
def test_load_succeeds_if_macaroon_secret_key_missing(self) -> None:
79-
self.generate_config_and_remove_lines_containing("macaroon")
79+
self.generate_config_and_remove_lines_containing(["macaroon"])
8080
config1 = HomeServerConfig.load_config("", ["-c", self.config_file])
8181
config2 = HomeServerConfig.load_config("", ["-c", self.config_file])
8282
config3 = HomeServerConfig.load_or_generate_config("", ["-c", self.config_file])
@@ -111,7 +111,7 @@ def test_disable_registration(self) -> None:
111111
self.assertTrue(config3.registration.enable_registration)
112112

113113
def test_stats_enabled(self) -> None:
114-
self.generate_config_and_remove_lines_containing("enable_metrics")
114+
self.generate_config_and_remove_lines_containing(["enable_metrics"])
115115
self.add_lines_to_config(["enable_metrics: true"])
116116

117117
# The default Metrics Flags are off by default.
@@ -164,7 +164,7 @@ def test_secret_files_missing(self, config_str: str) -> None:
164164
def test_secret_files_existing(
165165
self, config_line: str, get_secret: Callable[[RootConfig], str]
166166
) -> None:
167-
self.generate_config_and_remove_lines_containing("registration_shared_secret")
167+
self.generate_config_and_remove_lines_containing(["registration_shared_secret"])
168168
with tempfile.NamedTemporaryFile(buffering=0) as secret_file:
169169
secret_file.write(b"53C237")
170170

tests/config/utils.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,12 +51,13 @@ def generate_config(self) -> None:
5151
],
5252
)
5353

54-
def generate_config_and_remove_lines_containing(self, needle: str) -> None:
54+
def generate_config_and_remove_lines_containing(self, needles: list[str]) -> None:
5555
self.generate_config()
5656

5757
with open(self.config_file) as f:
5858
contents = f.readlines()
59-
contents = [line for line in contents if needle not in line]
59+
for needle in needles:
60+
contents = [line for line in contents if needle not in line]
6061
with open(self.config_file, "w") as f:
6162
f.write("".join(contents))
6263

0 commit comments

Comments
 (0)