fix(e2e): addressed comments

Author: jrobertboos

tests/e2e/features/skills.feature

@@ -215,7 +215,6 @@ Feature: Agent skills tests
     """
     When I wait for the response to be completed
     Then The status code of the response is 200
-      And The response is the last streamed fragment
       And The body of the "tool_calls" field of the response is the following    
       """
       [
@@ -284,7 +283,6 @@ Feature: Agent skills tests
     """
     When I wait for the response to be completed
     Then The status code of the response is 200
-      And The response is the last streamed fragment
       And The body of the "tool_calls" field of the response is the following    
       """
       [
@@ -358,7 +356,6 @@ Feature: Agent skills tests
     """
     When I wait for the response to be completed
     Then The status code of the response is 200
-      And The response is the last streamed fragment
       And The body of the "tool_calls" field of the response is the following    
       """
       [
@@ -428,7 +425,6 @@ Feature: Agent skills tests
     """
     When I wait for the response to be completed
     Then The status code of the response is 200
-      And The response is the last streamed fragment
      And The body of the "tool_calls" field of the response is the following    
       """
       [
@@ -494,7 +490,6 @@ Feature: Agent skills tests
     """
     When I wait for the response to be completed
     Then The status code of the response is 200
-      And The response is the last streamed fragment
       And The body of the "tool_calls" field of the response is the following    
       """
       [
@@ -623,7 +618,6 @@ Feature: Agent skills tests
     """
     When I wait for the response to be completed
     Then The status code of the response is 200
-      And The response is the last streamed fragment
       And The body of the "tool_calls" field of the response is the following    
       """
       [
@@ -716,7 +710,6 @@ Feature: Agent skills tests
     """
     When I wait for the response to be completed
     Then The status code of the response is 200
-     And The response is the last streamed fragment
       And The body of the "tool_calls" field of the response is the following    
       """
       [

tests/e2e/features/steps/common_http.py

@@ -328,8 +328,7 @@ def check_response_field_body(context: Context, field: str) -> None:
 
     actual_value = response_body[field]
 
-    if not context.text:
-        return
+    assert context.text, "Expected value for response field body is missing"
 
     expected_value = json.loads(context.text)
     validate_json_partially(actual_value, expected_value)

tests/e2e/features/steps/llm_query_response.py

@@ -94,6 +94,7 @@ def wait_for_complete_response(context: Context) -> None:
     context.response_data = _parse_streaming_response(context.response.text)
     context.response.raise_for_status()
     assert context.response_data["finished"] is True
+    context.use_streaming_response_data = True
 
 
 @step('I use "{endpoint}" to ask question')
@@ -404,18 +405,3 @@ def _parse_streaming_response(response_text: str) -> dict:
         "finished": finished,
         "stream_error": stream_error,
     }
-
-
-@then("The response is the last streamed fragment")
-def response_is_last_streamed_fragment(context: Context) -> None:
-    """Assert streaming finished and flag context for field checks.
-
-    Sets ``context.use_streaming_response_data`` so subsequent steps
-    read from ``context.response_data`` instead of the raw HTTP JSON.
-    """
-    assert hasattr(context, "response_data"), "Streaming response has not been parsed"
-    assert (
-        context.response_data.get("finished") is True
-    ), "Streaming response not finished"
-    context.use_streaming_response_data = True
-    print(context.response_data)