feat: update test_results_parser version and start computing names

database/models/reports.py

@@ -289,6 +289,11 @@ def id(self):
     # for example: the same test being run on windows vs. mac
     flags_hash = Column(types.String(256), nullable=False)
 
+    framework = Column(types.String(100), nullable=True)
+
+    computed_name = Column(types.Text, nullable=True)
+    filename = Column(types.Text, nullable=True)
+
     __table_args__ = (
         UniqueConstraint(
             "repoid",

requirements.in

@@ -1,6 +1,6 @@
 https://github.com/codecov/opentelem-python/archive/refs/tags/v0.0.4a1.tar.gz#egg=codecovopentelem
 https://github.com/codecov/shared/archive/88117b96a4b420d88549b8df2649c3eb9c61c2a5.tar.gz#egg=shared
-https://github.com/codecov/test-results-parser/archive/1507de2241601d678e514c08b38426e48bb6d47d.tar.gz#egg=test-results-parser
+https://github.com/codecov/test-results-parser/archive/ef39a0888acd62d02a316a852a15d755c74e78c6.tar.gz#egg=test-results-parser
 https://github.com/codecov/timestring/archive/d37ceacc5954dff3b5bd2f887936a98a668dda42.tar.gz#egg=timestring
 asgiref>=3.7.2
 analytics-python==1.3.0b1

requirements.txt

@@ -2,7 +2,7 @@
 # This file is autogenerated by pip-compile with Python 3.12
 # by the following command:
 #
-#    pip-compile
+#    pip-compile requirements.in
 #
 amqp==5.2.0
     # via kombu
@@ -391,7 +391,7 @@ statsd==3.3.0
     #   shared
 stripe==9.6.0
     # via -r requirements.in
-test-results-parser @ https://github.com/codecov/test-results-parser/archive/1507de2241601d678e514c08b38426e48bb6d47d.tar.gz
+test-results-parser @ https://github.com/codecov/test-results-parser/archive/ef39a0888acd62d02a316a852a15d755c74e78c6.tar.gz
     # via -r requirements.in
 text-unidecode==1.3
     # via faker

services/test_results.py

@@ -112,8 +112,7 @@ def generate_test_id(repoid, testsuite, name, flags_hash):
 @dataclass
 class TestResultsNotificationFailure:
     failure_message: str
-    testsuite: str
-    testname: str
+    display_name: str
     envs: List[str]
     test_id: str
     duration_seconds: float
@@ -205,7 +204,7 @@ def generate_view_test_analytics_line(commit: Commit) -> str:
 def messagify_failure(
     failure: TestResultsNotificationFailure,
 ) -> str:
-    test_name = wrap_in_code(failure.testname.replace("\x1f", " "))
+    test_name = wrap_in_code(failure.display_name.replace("\x1f", " "))
     formatted_duration = display_duration(failure.duration_seconds)
     stack_trace_summary = f"Stack Traces | {formatted_duration}s run time"
     stack_trace = wrap_in_details(
@@ -219,7 +218,7 @@ def messagify_flake(
     flaky_failure: TestResultsNotificationFailure,
     flake_info: FlakeInfo,
 ) -> str:
-    test_name = wrap_in_code(flaky_failure.testname.replace("\x1f", " "))
+    test_name = wrap_in_code(flaky_failure.display_name.replace("\x1f", " "))
     formatted_duration = display_duration(flaky_failure.duration_seconds)
     flake_rate = flake_info.failed / flake_info.count * 100
     flake_rate_section = f"**Flake rate in main:** {flake_rate:.2f}% (Passed {flake_info.count - flake_info.failed} times, Failed {flake_info.failed} times)"
@@ -256,7 +255,7 @@ def build_message(self) -> str:
                 lambda x: x.test_id not in self.payload.flaky_tests,
                 self.payload.failures,
             ),
-            key=lambda x: (x.duration_seconds, x.testname),
+            key=lambda x: (x.duration_seconds, x.display_name),
         )
 
         if failures:

services/tests/test_test_results.py

@@ -74,7 +74,6 @@ def test_generate_failure_info():
     test_id = generate_test_id(1, "testsuite", "testname", flags_hash)
     fail = TestResultsNotificationFailure(
         "hello world",
-        "testsuite",
         "testname",
         [],
         test_id,
@@ -100,7 +99,6 @@ def test_build_message():
     test_id = generate_test_id(1, "testsuite", "testname", flags_hash)
     fail = TestResultsNotificationFailure(
         "hello world",
-        "testsuite",
         "testname",
         [],
         test_id,
@@ -147,7 +145,6 @@ def test_build_message_with_flake():
     test_id = generate_test_id(1, "testsuite", "testname", flags_hash)
     fail = TestResultsNotificationFailure(
         "hello world",
-        "testsuite",
         "testname",
         [],
         test_id,

tasks/test_results_finisher.py

@@ -232,8 +232,9 @@ def process_impl_within_lock(
 
                 failures.append(
                     TestResultsNotificationFailure(
-                        testsuite=test_instance.test.testsuite,
-                        testname=test_instance.test.name,
+                        display_name=test_instance.test.computed_name
+                        if test_instance.test.computed_name is not None
+                        else test_instance.test.name,
                         failure_message=failure_message,
                         test_id=test_instance.test_id,
                         envs=flag_names,