Skip to content

Commit ab739e5

Browse files
committed
merge fix
2 parents 73eb55e + b4d379e commit ab739e5

1 file changed

Lines changed: 8 additions & 27 deletions

File tree

src/backend/tests/unit/api/v1/test_mcp_projects.py

Lines changed: 8 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1041,12 +1041,9 @@ async def test_should_report_available_true_when_app_directory_exists_but_config
10411041

10421042
for entry in results:
10431043
assert entry["available"] is True, (
1044-
f"{entry['name']} should be available (directory exists) "
1045-
f"even when config file is missing"
1046-
)
1047-
assert entry["installed"] is False, (
1048-
f"{entry['name']} should not be installed (config file doesn't exist)"
1044+
f"{entry['name']} should be available (directory exists) even when config file is missing"
10491045
)
1046+
assert entry["installed"] is False, f"{entry['name']} should not be installed (config file doesn't exist)"
10501047

10511048

10521049
async def test_should_report_installed_true_when_config_file_contains_matching_url(
@@ -1086,9 +1083,7 @@ async def test_should_report_installed_true_when_config_file_contains_matching_u
10861083

10871084
for entry in results:
10881085
assert entry["available"] is True, f"{entry['name']} should be available"
1089-
assert entry["installed"] is True, (
1090-
f"{entry['name']} should be installed (config has matching URL)"
1091-
)
1086+
assert entry["installed"] is True, f"{entry['name']} should be installed (config has matching URL)"
10921087

10931088

10941089
async def test_should_report_installed_false_when_config_file_has_no_matching_url(
@@ -1107,13 +1102,7 @@ async def test_should_report_installed_false_when_config_file_has_no_matching_ur
11071102
client_paths = _prepare_installed_check_env(monkeypatch, tmp_path)
11081103

11091104
for path in client_paths.values():
1110-
config = {
1111-
"mcpServers": {
1112-
"other-server": {
1113-
"args": ["https://other-server.example.com/sse"]
1114-
}
1115-
}
1116-
}
1105+
config = {"mcpServers": {"other-server": {"args": ["https://other-server.example.com/sse"]}}}
11171106
path.write_text(json.dumps(config))
11181107

11191108
response = await client.get(
@@ -1126,9 +1115,7 @@ async def test_should_report_installed_false_when_config_file_has_no_matching_ur
11261115

11271116
for entry in results:
11281117
assert entry["available"] is True, f"{entry['name']} should be available"
1129-
assert entry["installed"] is False, (
1130-
f"{entry['name']} should not be installed (URL doesn't match)"
1131-
)
1118+
assert entry["installed"] is False, f"{entry['name']} should not be installed (URL doesn't match)"
11321119

11331120

11341121
async def test_should_report_available_false_when_app_directory_does_not_exist(
@@ -1175,9 +1162,7 @@ async def fake_sse(project_id):
11751162
results = response.json()
11761163

11771164
for entry in results:
1178-
assert entry["available"] is False, (
1179-
f"{entry['name']} should not be available (directory doesn't exist)"
1180-
)
1165+
assert entry["available"] is False, f"{entry['name']} should not be available (directory doesn't exist)"
11811166
assert entry["installed"] is False
11821167

11831168

@@ -1208,9 +1193,5 @@ async def test_should_report_available_true_when_config_file_has_corrupt_json(
12081193
results = response.json()
12091194

12101195
for entry in results:
1211-
assert entry["available"] is True, (
1212-
f"{entry['name']} should be available (directory exists)"
1213-
)
1214-
assert entry["installed"] is False, (
1215-
f"{entry['name']} should not be installed (JSON is corrupt)"
1216-
)
1196+
assert entry["available"] is True, f"{entry['name']} should be available (directory exists)"
1197+
assert entry["installed"] is False, f"{entry['name']} should not be installed (JSON is corrupt)"

0 commit comments

Comments
 (0)