fix CI deprecation warning, fix failing test case

RasaHQ · Aug 21, 2024 · d346b12 · d346b12
1 parent e3856a2
commit d346b12
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/.github/workflows/continous-integration.yml b/.github/workflows/continous-integration.yml
@@ -100,7 +100,7 @@ jobs:
           path: models/${{steps.upload_model.outputs.model}}.tar.gz
 
   run_e2e_tests:
-    name: Run e2e Tests
+    name: Run e2e tests
     runs-on: ubuntu-22.04
     needs: [train-model]
 

diff --git a/config/config.yml b/config/config.yml
@@ -28,7 +28,7 @@ pipeline:
 - name: NLUCommandAdapter
 - name: SingleStepLLMCommandGenerator
   llm:
-    model_name: gpt-4
+    model: gpt-4
     request_timeout: 7
     temperature: 0.0
     top_p: 0.0

diff --git a/e2e_tests_with_assertions/passing/happy_path/user_replace_card_with_linked_flow.yml b/e2e_tests_with_assertions/passing/happy_path/user_replace_card_with_linked_flow.yml
@@ -104,7 +104,12 @@ test_cases:
       assertions:
             - slot_was_set:
                 - name: replacement_reason
-                  value: some other reason
+                  # we cannot specify the original user value because the LLM can extract variable values
+                  # however the slot should be reset as part of running the slot rejections, which
+                  # is why we set the value to null
+            - slot_was_set:
+                - name: replacement_reason
+                  value: null
             - bot_uttered:
                 utter_name: utter_categorical_slot_rejection
             - bot_uttered: