hide_rule_turn should default to False when ActionExecuted is d…

…eserialised. (#9491)

* hide_rule_turn should default to False when ActionExecuted is deserialised.

* add changelog

* black

changelog/9490.bugfix.md

@@ -0,0 +1 @@
+Fixes bug where `hide_rule_turn` was defaulting to `None` when ActionExecuted was deserialised.

rasa/shared/core/events.py

@@ -1562,7 +1562,7 @@ def _from_story_string(cls, parameters: Dict[Text, Any]) -> Optional[List[Event]
                 parameters.get("timestamp"),
                 parameters.get("metadata"),
                 parameters.get("action_text"),
-                parameters.get("hide_rule_turn"),
+                parameters.get("hide_rule_turn", False),
             )
         ]
 

tests/shared/core/test_events.py

@@ -158,6 +158,20 @@ def test_json_parse_user():
     )
 
 
+def test_json_parse_action_executed_with_no_hide_rule():
+    evt = {
+        "event": "action",
+        "name": "action_listen",
+        "policy": None,
+        "confidence": None,
+        "timestamp": None,
+    }
+    deserialised: ActionExecuted = Event.from_parameters(evt)
+    expected = ActionExecuted("action_listen",)
+    assert deserialised == expected
+    assert deserialised.hide_rule_turn == expected.hide_rule_turn
+
+
 def test_json_parse_bot():
     evt = {"event": "bot", "text": "Hey there!", "data": {}}
     assert Event.from_parameters(evt) == BotUttered("Hey there!", {})