Skip to content

Commit 007d398

Browse files
committed
[Gardening] ReportFormatter: tying up loose ends
1 parent b3f7996 commit 007d398

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

benchmark/scripts/compare_perf_tests.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -648,10 +648,8 @@ def format_columns(r, is_strong):
648648
def table(title, results, is_strong=False, is_open=False):
649649
if not results:
650650
return ''
651-
rows = [
652-
row(format_columns(ReportFormatter.values(r), is_strong))
653-
for r in results
654-
]
651+
rows = [row(format_columns(ReportFormatter.values(r), is_strong))
652+
for r in results]
655653
table = (header(title if self.single_table else '',
656654
ReportFormatter.header_for(results[0])) +
657655
''.join(rows))

benchmark/scripts/test_compare_perf_tests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -856,7 +856,7 @@ def test_report_only_changes(self):
856856
self.assertNotIn('AngryPhonebook', html)
857857

858858
def test_single_table_report(self):
859-
"""Single table report has bold inline headers and no sections."""
859+
"""Single table report has inline headers and no elaborate sections."""
860860
self.tc.removed = [] # test handling empty section
861861
rf = ReportFormatter(self.tc, changes_only=True, single_table=True)
862862
markdown = rf.markdown()

0 commit comments

Comments
 (0)