test(recorded): handle refresh edge cases

Author: Pouyanpi

tests/recorded/cassette.py

@@ -180,7 +180,7 @@ def cassette_with_parsed_bodies(cassette: dict[str, Any]) -> dict[str, Any]:
     without changing stream semantics.
     """
     cassette = deepcopy(cassette)
-    for interaction in cassette.get("interactions", []):
+    for interaction in cassette.get("interactions") or []:
         request = interaction.get("request", {})
         request_body = request.get("body")
         request_data = _json_body(request_body)
@@ -209,7 +209,7 @@ def cassette_with_parsed_bodies(cassette: dict[str, Any]) -> dict[str, Any]:
 def cassette_with_rehydrated_bodies(cassette: dict[str, Any]) -> dict[str, Any]:
     """Convert readable cassette bodies back to raw strings for VCR replay."""
     cassette = deepcopy(cassette)
-    for interaction in cassette.get("interactions", []):
+    for interaction in cassette.get("interactions") or []:
         request = interaction.get("request", {})
         if "parsed_body" in request:
             request["body"] = _json_body_text(request["parsed_body"])

tests/recorded/test_cassette_sanitization.py

@@ -31,7 +31,7 @@
     recorded_body_matcher,
 )
 from tests.recorded.sanitization import FILTERED_HEADERS
-from tests.recorded.utils import api_key_for_record_mode
+from tests.recorded.utils import api_key_for_record_mode, set_api_key_for_record_mode
 
 pytestmark = [pytest.mark.recorded]
 
@@ -124,6 +124,13 @@ def test_recorded_cassette_serializer_keeps_json_bodies_readable():
     assert ReadableYamlSerializer.deserialize(text)["interactions"][0]["response"]["body"]["string"].startswith("{")
 
 
+def test_recorded_cassette_serializer_handles_null_interactions():
+    cassette = {"version": 1, "interactions": None}
+
+    assert yaml.safe_load(ReadableYamlSerializer.serialize(cassette)) == cassette
+    assert ReadableYamlSerializer.deserialize("version: 1\ninteractions:\n") == cassette
+
+
 def test_recorded_cassette_serializer_redacts_access_tokens_from_parsed_bodies():
     request = Request(
         method="POST",
@@ -513,6 +520,13 @@ def test_recorded_refresh_uses_api_key_without_live_mode_gate(monkeypatch):
     assert api_key_for_record_mode("OPENAI_API_KEY", "dummy-key", "rewrite") == "real-key"
 
 
+def test_recorded_refresh_fixture_returns_selected_api_key(monkeypatch):
+    monkeypatch.setenv("OPENAI_API_KEY", "real-key")
+
+    assert set_api_key_for_record_mode(monkeypatch, "OPENAI_API_KEY", "dummy-key", "rewrite") == "real-key"
+    assert set_api_key_for_record_mode(monkeypatch, "OPENAI_API_KEY", "dummy-key", "none") == "dummy-key"
+
+
 def test_recorded_body_matcher_compares_sanitized_json_bodies():
     cassette_request = Request(
         method="POST",

tests/recorded/utils.py

@@ -37,4 +37,4 @@ def set_api_key_for_record_mode(
 ) -> str:
     value = api_key_for_record_mode(env_name, dummy_value, record_mode)
     monkeypatch.setenv(env_name, value)
-    return dummy_value
+    return value