Better format report summary when there is no base set to compare with

Author: dflook

corpus_test/generate_report.py

@@ -350,8 +350,7 @@ def report(results_dir: str, minifier_ref: str, minifier_sha: str, base_ref: str
     :param base_sha: The git sha of the base version of python-minifier we are comparing against
     """
 
-    yield f'''
-# Python Minifier Test Report
+    yield f'''# Python Minifier Test Report
 
 Git Ref: {minifier_ref}
 Git Sha: {minifier_sha}
@@ -375,24 +374,36 @@ def report(results_dir: str, minifier_ref: str, minifier_sha: str, base_ref: str
         try:
             base_summary = result_summary(results_dir, python_version, base_sha)
         except FileNotFoundError:
-            base_summary = ResultSet(python_version, base_ref)
-
-        mean_time_change = summary.mean_time - base_summary.mean_time
-
-        yield (
-                f'| {python_version} ' +
-                f'| {summary.valid_count} ' +
-                f'| {summary.mean_time:.3f} ({mean_time_change:+.3f}) ' +
-                f'| {format_size_change_detail(summary, base_summary)} ' +
-                f'| {format_difference(summary.larger_than_original(), base_summary.larger_than_original())} ' +
-                f'| {format_difference(summary.recursion_error(), base_summary.recursion_error())} ' +
-                f'| {format_difference(summary.unstable_minification(), base_summary.unstable_minification())} ' +
-                f'| {format_difference(summary.exception(), base_summary.exception())} |'
-        )
+            base_summary = None
+
+        if base_summary is None:
+            yield (
+                    f'| {python_version} ' +
+                    f'| {summary.valid_count} ' +
+                    f'| {summary.mean_time:.3f} ' +
+                    f'| {summary.mean_percent_of_original:.3f}% ' +
+                    f'| {len(list(summary.larger_than_original()))} ' +
+                    f'| {len(list(summary.recursion_error()))} ' +
+                    f'| {len(list(summary.unstable_minification()))} ' +
+                    f'| {len(list(summary.exception()))} |'
+            )
+        else:
+            mean_time_change = summary.mean_time - base_summary.mean_time
+
+            yield (
+                    f'| {python_version} ' +
+                    f'| {summary.valid_count} ' +
+                    f'| {summary.mean_time:.3f} ({mean_time_change:+.3f}) ' +
+                    f'| {format_size_change_detail(summary, base_summary)} ' +
+                    f'| {format_difference(summary.larger_than_original(), base_summary.larger_than_original())} ' +
+                    f'| {format_difference(summary.recursion_error(), base_summary.recursion_error())} ' +
+                    f'| {format_difference(summary.unstable_minification(), base_summary.unstable_minification())} ' +
+                    f'| {format_difference(summary.exception(), base_summary.exception())} |'
+            )
 
     if ENHANCED_REPORT:
         yield from report_larger_than_original(results_dir, ['3.14'], minifier_sha)
-        yield from report_larger_than_base(results_dir, ['3.14'], minifier_sha, base_sha)
+        yield from report_larger_than_base(results_dir, ['3.13'], minifier_sha, base_sha)
         yield from report_slowest(results_dir, ['3.14'], minifier_sha)
         yield from report_unstable(results_dir, ['2.7', '3.3', '3.4', '3.5', '3.6', '3.7', '3.8', '3.9', '3.10', '3.11', '3.12', '3.13', '3.14'], minifier_sha)
         yield from report_exceptions(results_dir, ['2.7', '3.3', '3.4', '3.5', '3.6', '3.7', '3.8', '3.9', '3.10', '3.11', '3.12', '3.13', '3.14'], minifier_sha)