Skip to content

Commit 591f081

Browse files
committed
Update tests to work with new alias version structure
1 parent 74bc54d commit 591f081

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

tests/test_dependency_mgmt.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,7 @@ def test_register_components_and_aliases(
261261
cn: DependencySpec(f"https://fake.repo.url/{cn}.git", "master", "")
262262
for cn in component_dirs
263263
}
264+
patch_read.return_value["fooer"] = patch_read.return_value["foo"]
264265

265266
dependency_mgmt.register_components(config)
266267

@@ -320,6 +321,7 @@ def test_register_dangling_aliases(
320321
cn: DependencySpec(f"https://fake.repo.url/{cn}.git", "master", "")
321322
for cn in component_dirs
322323
}
324+
patch_read.return_value["bazzer"] = patch_read.return_value["baz"]
323325

324326
dependency_mgmt.register_components(config)
325327

@@ -473,10 +475,10 @@ def test_validate_component_library_name(tmp_path: Path, libname: str, expected:
473475
)
474476
def test_verify_component_version_overrides(cluster_params: dict, expected: str):
475477
if expected == "":
476-
dependency_mgmt.verify_version_overrides(cluster_params)
478+
dependency_mgmt.verify_version_overrides(cluster_params, {})
477479
else:
478480
with pytest.raises(click.ClickException) as e:
479-
dependency_mgmt.verify_version_overrides(cluster_params)
481+
dependency_mgmt.verify_version_overrides(cluster_params, {})
480482

481483
assert expected in str(e)
482484

tests/test_dependency_mgmt_version_parsing.py

+5-3
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,9 @@
2323
@patch.object(version_parsing, "kapitan_inventory")
2424
def test_read_components(patch_inventory, config: Config):
2525
components = _setup_mock_inventory(patch_inventory)
26-
cspecs = version_parsing._read_components(config, ["test-component"])
26+
cspecs = version_parsing._read_components(
27+
config, {"test-component": "test-component"}
28+
)
2729

2830
# check that exactly 'test-component' is discovered
2931
assert {"test-component"} == set(cspecs.keys())
@@ -34,7 +36,7 @@ def test_read_components(patch_inventory, config: Config):
3436
@patch.object(version_parsing, "kapitan_inventory")
3537
def test_read_components_multiple(patch_inventory, config: Config):
3638
components = _setup_mock_inventory(patch_inventory)
37-
cspecs = version_parsing._read_components(config, components.keys())
39+
cspecs = version_parsing._read_components(config, {k: k for k in components.keys()})
3840
# check that exactly 'test-component' is discovered
3941
assert set(components.keys()) == set(cspecs.keys())
4042
assert all(components[cn]["url"] == cspecs[cn].url for cn in components.keys())
@@ -80,7 +82,7 @@ def test_read_components_exc(
8082
}
8183

8284
with pytest.raises(click.ClickException) as exc_info:
83-
_ = version_parsing._read_components(config, ckeys)
85+
_ = version_parsing._read_components(config, {k: k for k in ckeys})
8486

8587
assert exc_info.value.args[0] == exctext
8688

0 commit comments

Comments
 (0)