fix(agent): skip empty switch conditions

agent/component/switch.py

@@ -88,7 +88,7 @@ def _invoke(self, **kwargs):
                     self.set_output("_next", cond["to"])
                     return
 
-            if all(res):
+            if res and all(res):
                 self.set_output("next", [self._canvas.get_component_name(cpn_id) for cpn_id in cond["to"]])
                 self.set_output("_next", cond["to"])
                 return

test/unit_test/agent/component/test_switch.py

@@ -0,0 +1,59 @@
+from agent.component.switch import Switch, SwitchParam
+
+
+class _Canvas:
+    def __init__(self, variables=None):
+        self.variables = variables or {}
+
+    def is_canceled(self):
+        return False
+
+    def get_variable_value(self, cpn_id):
+        return self.variables[cpn_id]
+
+    def get_component_name(self, cpn_id):
+        return cpn_id
+
+
+def _switch(param, variables=None):
+    cpn = Switch.__new__(Switch)
+    cpn._canvas = _Canvas(variables)
+    cpn._id = "switch"
+    cpn._param = param
+    return cpn
+
+
+def test_switch_empty_condition_falls_through_to_else():
+    param = SwitchParam()
+    param.conditions = [
+        {
+            "logical_operator": "and",
+            "items": [{"cpn_id": "", "operator": "=", "value": "yes"}],
+            "to": ["case_target"],
+        }
+    ]
+    param.end_cpn_ids = ["else_target"]
+
+    cpn = _switch(param)
+    cpn._invoke()
+
+    assert cpn.output("_next") == ["else_target"]
+    assert cpn.output("next") == ["else_target"]
+
+
+def test_switch_non_empty_and_condition_still_matches():
+    param = SwitchParam()
+    param.conditions = [
+        {
+            "logical_operator": "and",
+            "items": [{"cpn_id": "answer", "operator": "=", "value": "yes"}],
+            "to": ["case_target"],
+        }
+    ]
+    param.end_cpn_ids = ["else_target"]
+
+    cpn = _switch(param, {"answer": "yes"})
+    cpn._invoke()
+
+    assert cpn.output("_next") == ["case_target"]
+    assert cpn.output("next") == ["case_target"]