Skip to content

Commit 0e0423f

Browse files
committed
Remove suite progress printout
It's not possible to compute this reliably with the V2 interface. The previous method only calculated the top level suites, not all suites, which was more confusing than it was helpful.
1 parent 28d5b4a commit 0e0423f

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

robot_trace/RobotTrace.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,6 @@ def flush(self, decrement_depth: bool = True):
175175

176176
class TestStatistics:
177177
def __init__(self):
178-
self.top_level_suite_count: int | None = None
179178
self.top_level_test_count: int | None = None
180179
self.started_suites = 0
181180
self.started_tests = 0
@@ -188,8 +187,6 @@ def __init__(self):
188187

189188
def start_suite(self, _, attributes):
190189
self.started_suites += 1
191-
if self.top_level_suite_count is None:
192-
self.top_level_suite_count = len(attributes["suites"]) or 1
193190
if self.top_level_test_count is None:
194191
self.top_level_test_count = attributes["totaltests"]
195192

@@ -209,7 +206,7 @@ def end_test(self, _, attributes):
209206
self.skipped_tests += 1
210207

211208
def format_suite_progress(self) -> str:
212-
return f"{self.started_suites:2d}/{self.top_level_suite_count:2d}"
209+
return f"{self.started_suites:2d}"
213210

214211
def format_test_progress(self) -> str:
215212
return f"{self.started_tests:2d}/{self.top_level_test_count:2d}"

tests/unit/test_cliprogress.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,6 @@ def test_start_suite_counts(self):
105105
attributes = {"suites": [1, 2, 3], "totaltests": 10}
106106

107107
stats.start_suite("My Suite", attributes)
108-
self.assertEqual(stats.top_level_suite_count, 3)
109108
self.assertEqual(stats.top_level_test_count, 10)
110109

111110
def test_start_test_increment(self):

0 commit comments

Comments
 (0)