commit 32aae3f2ca750f79b663bbe2cc6e42e3c97d3172
author Srikrishna Iyer <sriyer@google.com> Thu Mar 19 17:34:00 2020 -0700
committer sriyerg <46467186+sriyerg@users.noreply.github.com> Tue Mar 24 13:20:47 2020 -0700
tree 721505c17a838e9066e9024a43f33d331171c8a5
parent 4488d241da8db12ba459119c1fb9f1a3cd146513

[dvsim] Replaced dicts with OrderedDicts

- Fixes #1478

Signed-off-by: Srikrishna Iyer <sriyer@google.com>

util/dvsim/SimCfg.py

diff --git a/util/dvsim/SimCfg.py b/util/dvsim/SimCfg.py
index e0d04a5..c70c075 100644
--- a/util/dvsim/SimCfg.py
+++ b/util/dvsim/SimCfg.py
@@ -7,6 +7,7 @@
 
 import logging as log
 import sys
+from collections import OrderedDict
 
 from testplanner import class_defs, testplan_utils
 
@@ -94,7 +95,7 @@
         self.deploy = []
         self.cov_merge_deploy = None
         self.cov_report_deploy = None
-        self.results_summary = {}
+        self.results_summary = OrderedDict()
 
         # If is_master_cfg is set, then each cfg will have its own cov_deploy.
         # Maintain an array of those in cov_deploys.
@@ -504,6 +505,8 @@
                 results_str += self.cov_report_deploy.cov_results
                 self.results_summary[
                     "Coverage"] = self.cov_report_deploy.cov_total
+            else:
+                self.results_summary["Coverage"] = "--"
 
             # append link of detail result to block name
             self.results_summary["Name"] = self._get_results_page_link(
@@ -534,7 +537,8 @@
             row = []
             for title in item.results_summary:
                 row.append(item.results_summary[title])
-            if row != []: table.append(row)
+            if row == []: continue
+            table.append(row)
         self.results_summary_md = "## " + self.results_title + " (Summary)\n"
         self.results_summary_md += "### " + self.timestamp_long + "\n"
         self.results_summary_md += tabulate(table,