[Lldb-commits] [lldb] 8947469 - [lldb][Progress] Fix test for trimmed progress reports (#78357)

[via lldb-commits]

Author: Chelsea Cassanova
Date: 2024-01-16T14:35:39-08:00
New Revision: 8947469ec1ad6d35b2feec0acc43d0d191514f0b

URL: https://github.com/llvm/llvm-project/commit/8947469ec1ad6d35b2feec0acc43d0d191514f0b
DIFF: https://github.com/llvm/llvm-project/commit/8947469ec1ad6d35b2feec0acc43d0d191514f0b.diff

LOG: [lldb][Progress] Fix test for trimmed progress reports (#78357)

The test TestTrimmedProgressReporting tests that progress reports are
being sent by listening for events with the titles of specific progress
reports. Commit f1ef910b removed the report for Apple DWARF indices
which was one of the reports being listened for in this test, so that
report is removed here as well.

That commit also now creates all progress reports with details so
reports string are prepended with the details count. This changes the
length of the trimmed progress report title string that's checked for
here so this commit changes the string to match as well.

This test was skipped on non-Apple platforms, but since the progress
report for Apple DWARF indices has been removed this commit removes that
decorator.

Added: 
    

Modified: 
    lldb/test/API/functionalities/progress_reporting/TestTrimmedProgressReporting.py

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/progress_reporting/TestTrimmedProgressReporting.py b/lldb/test/API/functionalities/progress_reporting/TestTrimmedProgressReporting.py
index c4ff30a75a7aec8..357999b6f561933 100644
--- a/lldb/test/API/functionalities/progress_reporting/TestTrimmedProgressReporting.py
+++ b/lldb/test/API/functionalities/progress_reporting/TestTrimmedProgressReporting.py
@@ -33,24 +33,19 @@ def do_test(self, term_width, pattern_list):
     # PExpect uses many timeouts internally and doesn't play well
     # under ASAN on a loaded machine..
     @skipIfAsan
-    @skipUnlessDarwin
     @skipIfEditlineSupportMissing
     def test_trimmed_progress_message(self):
-        self.do_test(
-            19, ["Locating externa...", "Loading Apple DW...", "Parsing symbol t..."]
-        )
+        self.do_test(19, ["Locating e...", "Parsing sy..."])
 
     # PExpect uses many timeouts internally and doesn't play well
     # under ASAN on a loaded machine..
     @skipIfAsan
-    @skipUnlessDarwin
     @skipIfEditlineSupportMissing
     def test_long_progress_message(self):
         self.do_test(
             80,
             [
-                "Locating external symbol file for a.out...",
-                "Loading Apple DWARF index for a.out...",
-                "Parsing symbol table for dyld...",
+                "Locating external symbol file",
+                "Parsing symbol table",
             ],
         )