YAshhh29 commented on code in PR #71575:
URL: https://github.com/apache/airflow/pull/71575#discussion_r4048455744
##########
providers/common/ai/tests/unit/common/ai/durable/test_caching_model.py:
##########
@@ -158,6 +158,40 @@ async def
test_legacy_entry_without_fingerprint_treated_as_miss(
assert result is sample_response
mock_model.request.assert_called_once()
+ @pytest.mark.asyncio
+ async def test_unverifiable_current_request_treated_as_miss(
+ self, mock_model, mock_storage, counter, sample_response
+ ):
+ stale = ModelResponse(parts=[TextPart(content="stale")])
+ mock_storage.load_model_response.return_value = (stale, None)
Review Comment:
I've Added `test_unverifiable_request_leaves_a_verifiable_entry_intact` plus
the toolset equivalent:
a real stored fingerprint against a current request that can't be
fingerprinted, asserting the step runs live, doesn't replay, and that `save_*`
isn't called — so the usable entry survives for a later attempt instead of
being clobbered with a `fingerprint=None` one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
