templates: Change overview for build

kernelci · May 19, 2022 · 9e100b0 · 9e100b0
1 parent 5385055
commit 9e100b0
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/kcidb/templates/build_description.txt.j2 b/kcidb/templates/build_description.txt.j2
@@ -13,8 +13,7 @@ OVERVIEW
 
          Build: {{ misc_macros.valid_badge(build.valid) }}
 {% if build.tests %}
-         Tests: {{ test_macros.waived_status_badge(build.tests_root.waived,
-                                                   build.tests_root.status) }}
+         Tests: {{ test_macros.test_status(build.tests_root.waived_status_nodes) }}
 {% endif %}
 
 BUILD

diff --git a/kcidb/templates/test.j2 b/kcidb/templates/test.j2
@@ -37,20 +37,24 @@
         reject("none") | join(" ") | default(test.id, true) -}}
 {% endmacro %}
 
+{% macro test_status(waived_status_nodes) %}
+    {% for waived, status_nodes in waived_status_nodes.items() %}
+        {% for status, nodes in status_nodes.items() %}
+            {% if nodes %}
+                {{- "  " + waived_status_emoji(waived, status) +
+                    " " + (nodes | length | string) -}}
+            {% endif %}
+        {% endfor %}
+    {% endfor %}
+{% endmacro %}
+
 {% macro container_summary(container, max_list_len) %}
     {% if container.tests %}
         {{- "\nTESTS" }}
         {% set waived_status_nodes = container.tests_root.waived_status_nodes %}
         {{- "\n    Status" }}
             {{- "\n      " -}}
-            {% for waived, status_nodes in waived_status_nodes.items() %}
-                {% for status, nodes in status_nodes.items() %}
-                    {% if nodes %}
-                        {{- "  " + waived_status_emoji(waived, status) +
-                            " " + (nodes | length | string) -}}
-                    {% endif %}
-                {% endfor %}
-            {% endfor %}
+            {{ test_status(waived_status_nodes) }}
             {{- "" }}
         {% set errored_nodes = waived_status_nodes[false]["ERROR"] %}
         {% if errored_nodes %}