For Issue Tracker integration use Product/Version from execution.build

similar to previous commits use Product/Version values based on the
Build a specific TestExecution status was recorded against instead
of using values coming from TestRun->TestPlan which can change in the
future.

NOTE: execution.build is initialized with run.build and then may change
its value if we're recording results against multiple builds inside the
same TestRun.

(For example mark some executions as PASS, others as FAIL;
then update TR to a newer Build; retest and mark the FAIL results as
PASS)

Author: atodorov

tcms/issuetracker/base.py

@@ -90,14 +90,17 @@ def _report_comment(self, execution, user=None):  # pylint: disable=no-self-use
 {reporter}
 
 **Product:**
-{execution.run.plan.product.name}
+{execution.build.version.product.name}
 
-**Component(s):**
-{self.get_case_components(execution.case)}
+**Version:**
+{execution.build.version.value}
 
-**Version-Release number** (if applicable):
+**Build:**
 {execution.build.name}
 
+**Component(s):**
+{self.get_case_components(execution.case)}
+
 **Steps to reproduce**:
 {txt}
 

tcms/issuetracker/bugzilla_integration.py

@@ -78,9 +78,9 @@ def _report_issue(self, execution, user):
         to a URL with some of the values pre-defined as query parameters!
         """
         args = {
-            "product": execution.run.plan.product.name,
+            "product": execution.build.version.product.name,
             "component": self.get_case_components(execution.case),
-            "version": execution.run.plan.product_version.value,
+            "version": execution.build.version.value,
             "short_desc": f"Test case failure: {execution.case.summary}",
             "comment": self._report_comment(execution, user),
         }

tcms/issuetracker/kiwitcms.py

@@ -69,8 +69,8 @@ def _report_issue(self, execution, user):
         data = {
             "reporter": user,
             "summary": f"Failed test: {execution.case.summary}",
-            "product": execution.run.plan.product,
-            "version": execution.run.plan.product_version,
+            "product": execution.build.version.product,
+            "version": execution.build.version,
             "build": execution.build,
             "text": self._report_comment(execution, user),
             "_execution": execution,

tcms/issuetracker/tests/test_azureboards.py

@@ -122,7 +122,7 @@ def test_report_issue_from_test_execution_1click_works(self):
         for expected_string in [
             f"Filed from execution {self.execution_1.get_full_url()}",
             "Reporter",
-            self.execution_1.run.plan.product.name,
+            self.execution_1.build.version.product.name,
             self.component.name,
             "Steps to reproduce",
             self.execution_1.case.text,

tcms/issuetracker/tests/test_bitbucket.py

@@ -111,7 +111,7 @@ def test_report_issue_from_test_execution_1click_works(self):
         for expected_string in [
             f"Filed from execution {self.execution_1.get_full_url()}",
             "Reporter",
-            self.execution_1.run.plan.product.name,
+            self.execution_1.build.version.product.name,
             self.component.name,
             "Steps to reproduce",
             self.execution_1.case.text,

tcms/issuetracker/tests/test_bugzilla.py

@@ -105,6 +105,9 @@ def test_report_issue_from_test_execution_falls_back_to_query_params(self):
         test_case.save()  # will generate history object
 
         test_run = TestRunFactory(plan=test_plan)
+        test_run.build.version = version
+        test_run.build.save()
+
         execution2 = TestExecutionFactory(
             run=test_run, case=test_case, build=test_run.build
         )
@@ -144,6 +147,9 @@ def test_report_issue_from_test_execution_1click_works(self):
         test_case.save()  # will generate history object
 
         test_run = TestRunFactory(plan=test_plan)
+        test_run.build.version = version
+        test_run.build.save()
+
         execution2 = TestExecutionFactory(
             run=test_run, case=test_case, build=test_run.build
         )

tcms/issuetracker/tests/test_github.py

@@ -151,7 +151,7 @@ def test_report_issue_from_test_execution_1click_works(self):
         for expected_string in [
             f"Filed from execution {self.execution_1.get_full_url()}",
             "Reporter",
-            self.execution_1.run.plan.product.name,
+            self.execution_1.build.version.product.name,
             self.component.name,
             "Steps to reproduce",
             self.execution_1.case.text,

tcms/issuetracker/tests/test_gitlab_com.py

@@ -136,7 +136,7 @@ def test_report_issue_from_test_execution_1click_works(self):
         for expected_string in [
             f"Filed from execution {self.execution_1.get_full_url()}",
             "Reporter",
-            self.execution_1.run.plan.product.name,
+            self.execution_1.build.version.product.name,
             self.component.name,
             "Steps to reproduce",
             self.execution_1.case.text,

tcms/issuetracker/tests/test_gitlab_ee.py

@@ -175,7 +175,7 @@ def test_report_issue_from_test_execution_1click_works(self):
         for expected_string in [
             f"Filed from execution {self.execution_1.get_full_url()}",
             "Reporter",
-            self.execution_1.run.plan.product.name,
+            self.execution_1.build.version.product.name,
             self.component.name,
             "Steps to reproduce",
             self.execution_1.case.text,

tcms/issuetracker/tests/test_jira.py

@@ -34,8 +34,8 @@ def _fixture_setup(self):
         self.execution_1.run.save()
 
         # this is the name of the Project in JIRA. Key is "KT"
-        self.execution_1.run.plan.product.name = "Kiwi TCMS"
-        self.execution_1.run.plan.product.save()
+        self.execution_1.build.version.product.name = "Kiwi TCMS"
+        self.execution_1.build.version.product.save()
 
         self.component = ComponentFactory(
             name="JIRA integration", product=self.execution_1.run.plan.product
@@ -129,7 +129,7 @@ def test_report_issue_from_test_execution_1click_works(self):
         for expected_string in [
             f"Filed from execution {self.execution_1.get_full_url()}",
             "Reporter",
-            self.execution_1.run.plan.product.name,
+            self.execution_1.build.version.product.name,
             self.component.name,
             "Steps to reproduce",
             self.execution_1.case.text,