Include Koji build IDs in TF API response (#3002)

Include Koji build IDs in TF API response

Related to packit/dashboard#512
Assisted-by: Cursor(Claude)

Reviewed-by: gemini-code-assist[bot]
Reviewed-by: Nikola Forró

Author: centosinfra-prod-github-app[bot]

packit_service/service/api/testing_farm.py

@@ -163,6 +163,7 @@ def get(self, id):
             "commit_sha": test_run_model.commit_sha,
             "web_url": test_run_model.web_url,
             "copr_build_ids": [build.id for build in test_run_model.copr_builds],
+            "koji_build_ids": [build.id for build in test_run_model.koji_builds],
             "run_ids": sorted(run.id for run in test_run_model.group_of_targets.runs),
             "submitted_time": optional_timestamp(test_run_model.submitted_time),
         }

tests_openshift/conftest.py

@@ -59,6 +59,7 @@
     sa_session_transaction,
     sync_release_pr_association_table,
     tf_copr_association_table,
+    tf_koji_association_table,
 )
 
 
@@ -212,6 +213,7 @@ def clean_db():
         session.query(ProjectEventModel).delete()
 
         session.query(tf_copr_association_table).delete()
+        session.query(tf_koji_association_table).delete()
         session.query(TFTTestRunTargetModel).delete()
         session.query(TFTTestRunGroupModel).delete()
         session.query(CoprBuildTargetModel).delete()
@@ -1173,6 +1175,32 @@ def a_new_test_run_pr(srpm_build_model_with_new_run_for_pr, a_copr_build_for_pr)
     )
 
 
+@pytest.fixture()
+def a_new_test_run_with_koji_build(pr_project_event_model):
+    """Test run with an associated Koji build (Fedora CI scenario)."""
+    _, run_model = SRPMBuildModel.create_with_new_run(
+        project_event_model=pr_project_event_model,
+    )
+    test_group = TFTTestRunGroupModel.create(run_model, ranch="public")
+    koji_group = KojiBuildGroupModel.create(run_model)
+    koji_build = KojiBuildTargetModel.create(
+        task_id=SampleValues.build_id,
+        web_url=SampleValues.koji_web_url,
+        target=SampleValues.target,
+        status=SampleValues.status_pending,
+        scratch=True,
+        koji_build_group=koji_group,
+    )
+    yield TFTTestRunTargetModel.create(
+        pipeline_id=SampleValues.different_pipeline_id,
+        web_url=SampleValues.testing_farm_url,
+        target=SampleValues.target,
+        status=TestingFarmResult.new,
+        test_run_group=test_group,
+        koji_build_targets=[koji_build],
+    )
+
+
 @pytest.fixture()
 def a_new_test_run_branch_push(
     srpm_build_model_with_new_run_for_branch,

tests_openshift/service/test_api.py

@@ -362,6 +362,33 @@ def test_get_testing_farm_result(client, clean_before_and_after, a_new_test_run_
     assert "release" in response_dict
 
 
+def test_get_testing_farm_result_with_koji_build(
+    client,
+    clean_before_and_after,
+    a_new_test_run_with_koji_build,
+):
+    """Test that koji_build_ids are included in TF result API (Fedora CI scenario)."""
+    response = client.get(
+        url_for(
+            "api.testing-farm_testing_farm_result",
+            id=a_new_test_run_with_koji_build.id,
+        ),
+    )
+    response_dict = response.json
+
+    assert response_dict["pipeline_id"] == a_new_test_run_with_koji_build.pipeline_id
+    assert response_dict["status"] == TestingFarmResult.new
+
+    # Verify koji_build_ids is present and contains the expected build
+    assert "koji_build_ids" in response_dict
+    assert len(response_dict["koji_build_ids"]) == 1
+    assert response_dict["koji_build_ids"][0] == a_new_test_run_with_koji_build.koji_builds[0].id
+
+    # copr_build_ids should be empty for this test run
+    assert "copr_build_ids" in response_dict
+    assert response_dict["copr_build_ids"] == []
+
+
 def test_get_projects_list(client, clean_before_and_after, a_copr_build_for_pr):
     """Test Get Projects"""
     response = client.get(url_for("api.projects_projects_list"))