feat(update): for a given blueprint name only update the one with the higher index in the list

Author: fyhertz

src/ops2deb/updater.py

@@ -210,17 +210,23 @@ async def _find_latest_versions(
 
 def find_latest_releases(
     configuration: Configuration,
-    blueprints: list[Blueprint],
     fetcher: Fetcher,
     skip_names: list[str] | None,
     only_names: list[str] | None,
 ) -> Tuple[list[LatestRelease], list[Ops2debError]]:
-    blueprints = [b for b in blueprints if b.fetch is not None]
+    blueprints = [b for b in configuration.blueprints if b.fetch is not None]
     if skip_names:
         blueprints = [b for b in blueprints if b.name not in skip_names]
     if only_names:
         blueprints = [b for b in blueprints if b.name in only_names]
 
+    # when multiple blueprints have the same name, only look for new releases for the
+    # last one in the list
+    blueprints_by_name: dict[str, Blueprint] = {}
+    for blueprint in blueprints:
+        blueprints_by_name[blueprint.name] = blueprint
+    blueprints = list(blueprints_by_name.values())
+
     # find new releases for the selected list of blueprints
     releases, errors = asyncio.run(_find_latest_versions(blueprints))
 
@@ -303,9 +309,8 @@ def update(
     max_versions: int = 1,
 ) -> None:
     logger.title("Looking for new releases...")
-    blueprints = list(configuration.blueprints)
     releases, errors = find_latest_releases(
-        configuration, blueprints, fetcher, skip_names, only_names
+        configuration, fetcher, skip_names, only_names
     )
 
     summary: list[str] = []

tests/test_ops2deb.py

@@ -890,6 +890,32 @@ def test_update__should_not_produce_configuration_files_that_dont_pass_format_co
     assert result_format.exit_code == 0
 
 
+def test_update___only_looks_for_updates_for_the_last_blueprint_when_multiple_blueprints_have_the_same_name(  # noqa: E501
+    call_ops2deb, configuration_paths
+):
+    # Given
+    configuration = """\
+    - name: great-app
+      version: 1.0.0
+      summary: great package
+      fetch: http://testserver/{{version}}/great-app.tar.gz
+
+    - name: great-app
+      version: 1.0.1
+      summary: super package
+      fetch: http://testserver/{{version}}/great-app.tar.gz
+    """
+
+    # When
+    result = call_ops2deb("update", "--only", "great-app", configurations=[configuration])
+
+    # Then
+    raw_blueprints = load_configuration_file(configuration_paths[0]).raw_blueprints
+    assert result.exit_code == 0
+    assert raw_blueprints[0]["version"] == "1.0.0"
+    assert raw_blueprints[1]["version"] == "1.1.1"
+
+
 def test__format_should_be_idempotent(call_ops2deb, configuration_paths):
     # Given
     configuration_0 = """