Added proper uid to imported testcases (#1100)

Added proper uid to imported testcases, so the ui navigation can work with them properly.

Co-authored-by: Krisztian Notaisz <[email protected]>

doc/newsfragments/2966_changed.imported_reports_uids.rst

@@ -0,0 +1 @@
+Imported unittest reports (gtest, cppunit, junit) are now properly rendering the error output in the report viewer.

testplan/importers/cppunit.py

@@ -9,6 +9,7 @@
 from lxml.builder import E
 from lxml.objectify import Element
 
+from testplan.common.utils.strings import uuid4
 from testplan.importers.base import ThreePhaseFileImporter, T
 from testplan.importers.suitesresults import SuitesResult
 from testplan.report import (
@@ -133,6 +134,7 @@ def _process_data(self, data: Element) -> List[TestGroupReport]:
             suite_report = TestGroupReport(
                 name=suite_name,
                 category=ReportCategories.TESTSUITE,
+                uid=uuid4(),
             )
 
             for testcase in suite.getchildren():
@@ -144,6 +146,7 @@ def _process_data(self, data: Element) -> List[TestGroupReport]:
                 testcase_prefix = testcase_classname.split(".")[-1]
                 testcase_report = TestCaseReport(
                     name="{}::{}".format(testcase_prefix, testcase_name),
+                    uid=uuid4(),
                 )
 
                 if not testcase.getchildren():

testplan/importers/gtest.py

@@ -6,6 +6,7 @@
 from lxml import objectify
 from lxml.objectify import Element
 
+from testplan.common.utils.strings import uuid4
 from testplan.importers.base import T, ThreePhaseFileImporter
 from testplan.importers.suitesresults import SuitesResult
 from testplan.report import (
@@ -51,13 +52,16 @@ def _process_data(self, data: Element) -> List[TestGroupReport]:
             suite_report = TestGroupReport(
                 name=suite_name,
                 category=ReportCategories.TESTSUITE,
+                uid=uuid4(),
             )
             suite_has_run = False
 
             for testcase in suite.getchildren():
 
                 testcase_name = testcase.attrib["name"]
-                testcase_report = TestCaseReport(name=testcase_name)
+                testcase_report = TestCaseReport(
+                    name=testcase_name, uid=uuid4()
+                )
 
                 if not testcase.getchildren():
                     assertion_obj = RawAssertion(

testplan/importers/junit.py

@@ -16,6 +16,7 @@
 )
 from testplan.testing.multitest.entries.assertions import RawAssertion
 from testplan.testing.multitest.entries.schemas.base import registry
+from ..common.utils.strings import uuid4
 
 
 class JUnitImportedResult(SuitesResult):
@@ -53,6 +54,7 @@ def _process_data(self, data: Element) -> List[TestGroupReport]:
             suite_report = TestGroupReport(
                 name=suite_name,
                 category=ReportCategories.TESTSUITE,
+                uid=uuid4(),
             )
 
             for element in suite.getchildren():
@@ -76,7 +78,9 @@ def _process_data(self, data: Element) -> List[TestGroupReport]:
                 else:
                     case_report_name = f"{case_class}::{case_name}"
 
-                case_report = TestCaseReport(name=case_report_name)
+                case_report = TestCaseReport(
+                    name=case_report_name, uid=uuid4()
+                )
 
                 if not element.getchildren():
                     assertion = RawAssertion(

testplan/importers/suitesresults.py

@@ -1,5 +1,6 @@
 from typing import List
 
+from testplan.common.utils.strings import uuid4
 from testplan.importers import ImportedResult
 from testplan.report import TestGroupReport, TestReport, ReportCategories
 
@@ -30,11 +31,14 @@ def as_test_report(self) -> TestReport:
 
         :return: a plan report contains a single test having all the returned suite results
         """
-        report = TestReport(name=self.name, description=self.description)
+        report = TestReport(
+            name=self.name, description=self.description, uid=uuid4()
+        )
         test_report = TestGroupReport(
             name=self.name,
             category=self.REPORT_CATEGORY,
             description=self.description,
+            uid=uuid4(),
         )
 
         for suite_report in self.results():