commit aae928185bb0ffa4bab6faca3fd111e13c1a0259
author Srikrishna Iyer <sriyer@google.com> Wed Jul 01 00:24:46 2020 -0700
committer sriyerg <46467186+sriyerg@users.noreply.github.com> Wed Jul 01 11:05:26 2020 -0700
tree 8078d87dbcb3c76ed7bcac136cd9d547cebdd1f2
parent 5cae0cf1fac783e0d0df8c8597bf65322a696a56

[dvsim] Fix for #2686 - missing else

Testplan milestones (V1, V2, V3) are also regression targets (one can
run all V2 tests for instance by passing the switch `-i V2`). It could
be possible that the tests added to the testplan to one of these
milestones could actually be missing (yet to be developed) in the
simulation config hjson.

When that is true, invoking dvsim results in the error messages reported
in #2686. The fix for this is a missing else clause.

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

util/dvsim/Modes.py

diff --git a/util/dvsim/Modes.py b/util/dvsim/Modes.py
index 6998d53..ac6ef6d 100644
--- a/util/dvsim/Modes.py
+++ b/util/dvsim/Modes.py
@@ -536,6 +536,7 @@
                         log.error(
                             "Test \"%s\" added to regression \"%s\" not found!",
                             test, regression_obj.name)
+                        continue
                     tests_objs.append(test_obj)
                 regression_obj.tests = tests_objs