Skip to content

Commit d5b94d4

Browse files
committed
test(canary): replace assert len(mock_req.request_history) == N checks with specific request filtering
Signed-off-by: Teo <[email protected]>
1 parent 93482dd commit d5b94d4

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

tests/unit/test_canary.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,13 @@ def test_agent_ops_record(self, mock_req):
2222
agentops.record(ActionEvent(event_type))
2323
time.sleep(2)
2424

25-
# 3 requests: check_for_updates, create_session, create_events
26-
assert len(mock_req.request_history) == 3
25+
# Find event requests
26+
event_requests = [r for r in mock_req.request_history if "/v2/create_events" in r.url]
27+
assert len(event_requests) > 0
28+
last_event_request = event_requests[-1]
2729

28-
request_json = mock_req.last_request.json()
29-
assert mock_req.last_request.headers["X-Agentops-Api-Key"] == self.api_key
30+
assert last_event_request.headers["X-Agentops-Api-Key"] == self.api_key
31+
request_json = last_event_request.json()
3032
assert request_json["events"][0]["event_type"] == event_type
3133

3234
agentops.end_session("Success")

0 commit comments

Comments
 (0)