Skip to content

Commit 8947469

Browse files
[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 f1ef910 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.
1 parent 00b6d03 commit 8947469

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

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

+3-8
Original file line numberDiff line numberDiff line change
@@ -33,24 +33,19 @@ def do_test(self, term_width, pattern_list):
3333
# PExpect uses many timeouts internally and doesn't play well
3434
# under ASAN on a loaded machine..
3535
@skipIfAsan
36-
@skipUnlessDarwin
3736
@skipIfEditlineSupportMissing
3837
def test_trimmed_progress_message(self):
39-
self.do_test(
40-
19, ["Locating externa...", "Loading Apple DW...", "Parsing symbol t..."]
41-
)
38+
self.do_test(19, ["Locating e...", "Parsing sy..."])
4239

4340
# PExpect uses many timeouts internally and doesn't play well
4441
# under ASAN on a loaded machine..
4542
@skipIfAsan
46-
@skipUnlessDarwin
4743
@skipIfEditlineSupportMissing
4844
def test_long_progress_message(self):
4945
self.do_test(
5046
80,
5147
[
52-
"Locating external symbol file for a.out...",
53-
"Loading Apple DWARF index for a.out...",
54-
"Parsing symbol table for dyld...",
48+
"Locating external symbol file",
49+
"Parsing symbol table",
5550
],
5651
)

0 commit comments

Comments
 (0)