add test for conversation id generation

Author: wochinge

rasa/core/training/interactive.py

@@ -1317,7 +1317,7 @@ async def _plot_trackers(
     output_file: Optional[Text],
     endpoint: EndpointConfig,
     unconfirmed: Optional[List[Event]] = None,
-):
+) -> None:
     """Create a plot of the trackers of the passed conversation ids.
 
     This assumes that the last conversation id is the conversation we are currently
@@ -1328,7 +1328,7 @@ async def _plot_trackers(
     if not output_file or not conversation_ids:
         # if there is no output file provided, we are going to skip plotting
         # same happens if there are no conversation ids
-        return None
+        return
 
     event_sequences = await _fetch_events(conversation_ids, endpoint)
 

tests/cli/test_rasa_interactive.py

@@ -177,3 +177,13 @@ def test_pass_conversation_id_to_interactive_learning(monkeypatch: MonkeyPatch):
     do_interactive_learning(args, Mock())
 
     _serve_application.assert_called_once_with(ANY, ANY, True, expected_conversation_id)
+
+
+def test_generate_conversation_id_for_interactive_learning(monkeypatch: MonkeyPatch):
+    parser = argparse.ArgumentParser()
+    sub_parser = parser.add_subparsers()
+    interactive.add_subparser(sub_parser, [])
+
+    args = parser.parse_args(["interactive"])
+
+    assert args.conversation_id