Test-reporting bugfix (#711)

src/report_tests.py

@@ -17,6 +17,9 @@ def append(self, status: str, log_file: str, comment: str = '') -> None:
         link = HTML.link(Path(log_file).stem, self.artifact_path + log_file)
         self.body.append([status, f'{link} {comment}'])
 
+    def clear(self) -> None:
+        self.body = []
+
     def __str__(self) -> str:
         return HTML.table(self.head, self.body)
 
@@ -51,9 +54,10 @@ def __str__(self) -> str:
          'timed out after 5 seconds'),
     ]
 
-    comment = Markdown.header(machine_name(), level=3)
+    report = Markdown.header(machine_name(), level=3)
     any_tests_ran_on_machine = False
     for test_type in ['unit', 'integration', 'system']:
+        table.clear()
         all_tests_of_type_passed = True
         any_tests_of_type = False
         for file_name in sorted(
@@ -79,13 +83,13 @@ def __str__(self) -> str:
                 icon = ':green_circle:'
             else:
                 icon = ':red_circle:'
-            comment += HTML.collapsible(f'{icon} {test_type} test', table)
+            report += HTML.collapsible(f'{icon} {test_type} test', table)
 
         if test_type == 'system' and not os.path.isfile(
                 f'log/{machine_name()}/system_test/serial_test_run'):
-            comment += '\n\n**WARNING**: Serial tests did not run for system tests'
+            report += '\n\n**WARNING**: Serial tests did not run for system tests'
 
     if not any_tests_ran_on_machine:
-        comment += f'No tests executed.'
+        report += f'No tests executed.'
 
-    repo.comment(args.issue_id, comment)
+    repo.comment(args.issue_id, report)