refactor: right align build regressions in metrics email

Closes #1938

Signed-off-by: Marcelo Robert Santos <4mrSantos@gmail.com>

Author: MarceloRobert

backend/kernelCI_app/management/commands/templates/metrics_report.txt.j2

@@ -54,30 +54,37 @@ Period: {{ start_datetime }} to {{ end_datetime }}
 
 {% if build_incidents_by_origin -%}
 
-{{ "{:<12}".format("  Origin") -}}
-{{ "{:<18}".format("Regressions") -}}
-{{ "{:<19}".format("Affected") -}}
-Affected builds by top issues
-{{ "{:<12}".format("") -}}
-{{ "{:<18}".format("(known + new)") -}}
-{{ "{:<19}".format("Builds (total)") -}}
-{{ "{:<8}".format("#1") -}}{{ "{:<8}".format("#2") -}}#3
-  ─────────────────────────────────────────────────────────────────────────
+{% set origin_space = 12 -%}
+{% set known_reg_space = 6 -%}
+{% set new_reg_space = 6 -%}
+{% set total_reg_space = 6 -%}
+{% set reg_space = known_reg_space + new_reg_space + total_reg_space -%}
+{% set affected_space = 16 -%}
+{% set ind_issues_space = 10 -%}
+{{ "{:<{width}}".format("  Origin", width=origin_space) -}}
+{{ "{:>{width}}".format("Regressions", width=reg_space) -}}
+{{ "{:>{width}}".format("Affected", width=affected_space) -}}
+{{ "{:>{width}}".format("Affected builds by top issues", width=ind_issues_space*3 + 1) }} {#- +1 to get to a 2-space gap, same for the #1 issue and the space after total incidents #}
+{{ "{:<{width}}".format("", width=origin_space) -}}
+{{ "{:>{width}}".format("(known + new)", width=reg_space) -}}
+{{ "{:>{width}}".format("Builds (total)", width=affected_space) -}}
+{{ "{:>{width}}".format("#1", width=ind_issues_space + 1) -}}{{ "{:>{width}}".format("#2", width=ind_issues_space) -}}{{ "{:>{width}}".format("#3", width=ind_issues_space) }}
+  ───────────────────────────────────────────────────────────────────────────
 {% for origin, data in build_incidents_by_origin.items() | sort -%}
-  {{ "{:<12}".format("  " + origin) -}}
-  {{ "{:<4}".format(fmt(data['n_existing_issues'])) -}} +{{" "}}
-  {{- "{:<4}".format(fmt(data['n_new_issues'])) -}} ={{" "}}
-  {{- "{:<6}".format(fmt(data['n_total_issues'])) -}}
-  {{ "{:<19}".format(fmt(data['total_incidents'])) -}}
+  {{ "{:<{width}}".format("  " + origin, width=origin_space) -}}
+  {{ "{:>{width}}".format(fmt(data['n_existing_issues']) + " +", width=known_reg_space) }}
+  {{- "{:>{width}}".format(fmt(data['n_new_issues']) + " =", width=new_reg_space) }}
+  {{- "{:>{width}}".format(fmt(data['n_total_issues']), width=total_reg_space) -}}
+  {{ "{:>{width}}".format(fmt(data['total_incidents']), width=affected_space) -}} {{ " " -}}
   {% for issue_key, data in top_issues_by_origin.get(origin, {}).items() -%}
-    {{ "{:<8}".format(fmt(data['total_incidents'])) -}}
+    {{ "{:>{width}}".format(fmt(data['total_incidents']), width=ind_issues_space) -}}
   {% endfor %}
-{% endfor %}  ─────────────────────────────────────────────────────────────────────────
-{{ "{:<12}".format("  Total") -}}
-{{ "{:<4}".format(fmt(build_incidents_by_origin.values() | map(attribute='n_existing_issues') | sum)) -}} +{{" "}}
-{{- "{:<4}".format(fmt(build_incidents_by_origin.values() | map(attribute='n_new_issues') | sum)) -}} ={{" "}}
-{{- "{:<6}".format(fmt(build_incidents_by_origin.values() | map(attribute='n_total_issues') | sum)) -}}
-{{ fmt(build_incidents_by_origin.values() | map(attribute='total_incidents') | sum) }}
+{% endfor %}  ───────────────────────────────────────────────────────────────────────────
+{{ "{:<{width}}".format("  Total", width=origin_space) -}}
+{{ "{:>{width}}".format(fmt(build_incidents_by_origin.values() | map(attribute='n_existing_issues') | sum) + " +", width=known_reg_space) }}
+{{- "{:>{width}}".format(fmt(build_incidents_by_origin.values() | map(attribute='n_new_issues') | sum) + " =", width=new_reg_space) }}
+{{- "{:>{width}}".format(fmt(build_incidents_by_origin.values() | map(attribute='n_total_issues') | sum), width=total_reg_space) -}}
+{{ "{:>{width}}".format(fmt(build_incidents_by_origin.values() | map(attribute='total_incidents') | sum), width=affected_space) }}
 {%- else %}  No build regressions to show in this period. {%- endif %}
 
 

backend/kernelCI_app/tests/unitTests/commands/fixtures/metrics_notifications_example.txt

@@ -16,13 +16,13 @@ KernelCI Metrics Summary
   -----------------
   A "regression" is defined as a reported problem affecting 0 or multiple builds.
 
-  Origin    Regressions       Affected           Affected builds by top issues
-            (known + new)     Builds (total)     #1      #2      #3
-  ─────────────────────────────────────────────────────────────────────────
-  maestro   1   + 1   = 2     70                 50      20      
-  redhat    1   + 0   = 1     5                  5       
-  ─────────────────────────────────────────────────────────────────────────
-  Total     2   + 1   = 3     75
+  Origin           Regressions        Affected  Affected builds by top issues
+                 (known + new)  Builds (total)         #1        #2        #3
+  ───────────────────────────────────────────────────────────────────────────
+  maestro      1 +   1 =     2              70         50        20
+  redhat       1 +   0 =     1               5          5
+  ───────────────────────────────────────────────────────────────────────────
+  Total        2 +   1 =     3              75
 
 
   TOP REGRESSIONS PER ORIGIN