Revert "Flow branching based on action output - [ENG 372] (#12669)"

This reverts commit c25ed1d.
RasaHQ · Aug 8, 2023 · 2b659e7 · 2b659e7
1 parent 4bae431
commit 2b659e7
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 11 deletions.
diff --git a/data/test_trackers/tracker_moodbot.json b/data/test_trackers/tracker_moodbot.json
@@ -39,7 +39,6 @@
     "rasa_corrected_slots": null,
     "rasa_previous_flow": null,
     "requested_slot": null,
-    "return_value": null,
     "session_started_metadata": null
   },
   "latest_input_channel": null,

diff --git a/rasa/shared/core/constants.py b/rasa/shared/core/constants.py
@@ -83,14 +83,12 @@
 PREVIOUS_FLOW_SLOT = "rasa_previous_flow"
 CANCELLED_FLOW_SLOT = "rasa_cancelled_flow"
 CORRECTED_SLOTS_SLOT = "rasa_corrected_slots"
-RETURN_VALUE_SLOT = "return_value"
 
 FLOW_SLOT_NAMES = [
     FLOW_STACK_SLOT,
     PREVIOUS_FLOW_SLOT,
     CANCELLED_FLOW_SLOT,
     CORRECTED_SLOTS_SLOT,
-    RETURN_VALUE_SLOT,
 ]
 
 # slots for knowledge base

diff --git a/tests/core/test_actions.py b/tests/core/test_actions.py
@@ -95,7 +95,6 @@
     CANCELLED_FLOW_SLOT,
     PREVIOUS_FLOW_SLOT,
     CORRECTED_SLOTS_SLOT,
-    RETURN_VALUE_SLOT,
 )
 from rasa.shared.core.trackers import DialogueStateTracker
 from rasa.shared.exceptions import RasaException
@@ -248,7 +247,6 @@ async def test_remote_action_runs(
                     CANCELLED_FLOW_SLOT: None,
                     CORRECTED_SLOTS_SLOT: None,
                     PREVIOUS_FLOW_SLOT: None,
-                    RETURN_VALUE_SLOT: None,
                 },
                 "events": [],
                 "latest_input_channel": None,
@@ -315,7 +313,6 @@ async def test_remote_action_logs_events(
                     CANCELLED_FLOW_SLOT: None,
                     CORRECTED_SLOTS_SLOT: None,
                     PREVIOUS_FLOW_SLOT: None,
-                    RETURN_VALUE_SLOT: None,
                 },
                 "events": [],
                 "latest_input_channel": None,

diff --git a/tests/shared/core/test_domain.py b/tests/shared/core/test_domain.py
@@ -918,7 +918,6 @@ def test_domain_from_multiple_files():
         "rasa_corrected_slots",
         "rasa_previous_flow",
         "requested_slot",
-        "return_value",
         "session_started_metadata",
     ]
 

diff --git a/tests/shared/importers/test_rasa.py b/tests/shared/importers/test_rasa.py
@@ -16,7 +16,6 @@
     PREVIOUS_FLOW_SLOT,
     CANCELLED_FLOW_SLOT,
     CORRECTED_SLOTS_SLOT,
-    RETURN_VALUE_SLOT,
 )
 from rasa.shared.core.domain import Domain
 from rasa.shared.core.slots import AnySlot
@@ -38,7 +37,6 @@ def test_rasa_file_importer(project: Text):
         AnySlot(PREVIOUS_FLOW_SLOT, mappings=[{}]),
         AnySlot(CANCELLED_FLOW_SLOT, mappings=[{}]),
         AnySlot(CORRECTED_SLOTS_SLOT, mappings=[{}]),
-        AnySlot(RETURN_VALUE_SLOT, mappings=[{}]),
         AnySlot(SESSION_START_METADATA_SLOT, mappings=[{}]),
     ]
     assert domain.entities == []

diff --git a/tests/test_server.py b/tests/test_server.py
@@ -59,7 +59,6 @@
     CANCELLED_FLOW_SLOT,
     CORRECTED_SLOTS_SLOT,
     PREVIOUS_FLOW_SLOT,
-    RETURN_VALUE_SLOT,
 )
 from rasa.shared.core.domain import Domain, SessionConfig
 from rasa.shared.core.events import (
@@ -1125,7 +1124,6 @@ async def test_requesting_non_existent_tracker(rasa_app: SanicASGITestClient):
         CANCELLED_FLOW_SLOT: None,
         CORRECTED_SLOTS_SLOT: None,
         PREVIOUS_FLOW_SLOT: None,
-        RETURN_VALUE_SLOT: None,
     }
     assert content["sender_id"] == "madeupid"
     assert content["events"] == [