Skip to content

Commit 0418c47

Browse files
committed
tests: abstract result printing a bit more
Signed-off-by: Trevor Gamblin <[email protected]>
1 parent 683da80 commit 0418c47

File tree

2 files changed

+20
-6
lines changed

2 files changed

+20
-6
lines changed

src/patchtest2/tests/core.py

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,26 @@ def __init__(self, target_repo, series):
5050
]
5151
)
5252

53-
def print_mbox_results(self, tag):
54-
for testresult in self.mbox_results[tag]:
55-
print(testresult)
53+
self.results = dict(
54+
[
55+
(
56+
'mbox',
57+
self.mbox_results,
58+
),
59+
]
60+
)
61+
62+
def _print_result(self, category, tag):
63+
for value in self.results[category][tag]:
64+
print(value)
65+
66+
def _print_results(self, category):
67+
for tag in self.results[category].keys():
68+
self._print_result(category, tag)
5669

70+
def print_results(self):
71+
for category in self.results.keys():
72+
self._print_results(category)
5773

5874
# test_for_pattern()
5975
# @pattern: a pyparsing regex object

src/patchtest2/tests/patchtest.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,4 @@ def run():
1010
series = PatchSeries(args.patch_path)
1111
results = PatchtestResults(target_repo, series)
1212

13-
results.print_mbox_results("signed_off_by")
14-
results.print_mbox_results("shortlog_format")
15-
results.print_mbox_results("commit_message_presence")
13+
results.print_results()

0 commit comments

Comments
 (0)