From d2fc523721154ae6f988c0ed73f2345069cfb067 Mon Sep 17 00:00:00 2001 From: Mark Nottingham Date: Tue, 9 Jun 2026 14:05:35 +1000 Subject: [PATCH] Assert no unexpected interim responses were received The interim-response check only verified that expected interims were present; an empty expected_interim_responses array asserted nothing, so interim-not-cached (kind: required) could not catch a cache that replayed a stored interim. Add a count check that the number of received interims matches the number expected. Closes #168 Co-Authored-By: Claude Opus 4.8 --- test-engine/client/test.mjs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test-engine/client/test.mjs b/test-engine/client/test.mjs index fb08d80..c7010ec 100644 --- a/test-engine/client/test.mjs +++ b/test-engine/client/test.mjs @@ -223,6 +223,9 @@ function checkResponse (test, requests, idx, response, interimResponses) { }) } }) + + assert(isSetup, interimResponses.length === reqConfig.expected_interim_responses.length, + `Received ${interimResponses.length} interim response(s), expected ${reqConfig.expected_interim_responses.length}`) } return response.text().then(makeCheckResponseBody(test, reqConfig, response.status))