Fixed e2e test inconsistencies

Author: asimurka

tests/e2e/features/conversation_cache_v2.feature

@@ -212,6 +212,7 @@ Feature: Conversation Cache V2 API tests
   @NoCacheConfig
   Scenario: Check conversations/{conversation_id} fails when cache not configured
     Given REST API service prefix is /v2
+    And An invalid conversation cache path is configured
     And I set the Authorization header to Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6Ikpva
      When I access REST API endpoint "conversations" using HTTP GET method
      Then The status code of the response is 500
@@ -280,8 +281,11 @@ Feature: Conversation Cache V2 API tests
     Given REST API service prefix is /v2
     And I set the Authorization header to Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6Ikpva
      When I use REST API conversation endpoint with conversation_id "12345678-abcd-0000-0123-456789abcdef" using HTTP DELETE method
-     Then The status code of the response is 404
-     And The body of the response contains Conversation not found
+     Then The status code of the response is 200
+     And The body of the response, ignoring the "conversation_id" field, is the following
+      """
+      {"success": true, "response": "Conversation cannot be deleted"}
+      """
 
   @skip-in-library-mode
   Scenario: V2 conversations DELETE endpoint works even when llama-stack is down

tests/e2e/features/environment.py

@@ -95,11 +95,6 @@ def before_scenario(context: Context, scenario: Scenario) -> None:
         context.scenario_config = (
             f"tests/e2e/configuration/{mode_dir}/lightspeed-stack-no-cache.yaml"
         )
-        # Switch config and restart immediately
-        switch_config(
-            context.scenario_config
-        )  # Copies to default lightspeed-stack.yaml
-        restart_container("lightspeed-stack")
 
 
 def after_scenario(context: Context, scenario: Scenario) -> None:

tests/e2e/features/steps/conversation.py

@@ -4,7 +4,7 @@
 from behave import step, when, then  # pyright: ignore[reportAttributeAccessIssue]
 from behave.runner import Context
 import requests
-from tests.e2e.utils.utils import replace_placeholders
+from tests.e2e.utils.utils import replace_placeholders, restart_container, switch_config
 
 # default timeout for HTTP operations
 DEFAULT_TIMEOUT = 10
@@ -341,3 +341,10 @@ def check_conversation_model_provider(
         assert (
             actual_provider == expected_provider
         ), f"Turn {idx} expected provider '{expected_provider}', got '{actual_provider}'"
+
+
+@given("An invalid conversation cache path is configured")  # type: ignore
+def configure_invalid_conversation_cache_path(context: Context) -> None:
+    """Set an invalid conversation cache path and restart the container."""
+    switch_config(context.scenario_config)
+    restart_container("lightspeed-stack")