Skip to content

Commit 44260bf

Browse files
committed
fix(test_sessions::test_add_tags): TypeError: list indices must be integers or slices, not str
Signed-off-by: Teo <[email protected]>
1 parent 0fc50b7 commit 44260bf

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

tests/unit/test_session.py

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -315,6 +315,7 @@ def test_two_sessions(self, mock_req):
315315
assert len(request_json["session"]["tags"]) == 0
316316

317317
def test_add_tags(self, mock_req):
318+
"""Test adding tags to multiple sessions"""
318319
# Arrange
319320
session_1_tags = ["session-1"]
320321
session_2_tags = ["session-2"]
@@ -333,13 +334,19 @@ def test_add_tags(self, mock_req):
333334
session_2.end_session(end_state)
334335
time.sleep(0.15)
335336

336-
# Assert 3 requests, 1 for session init, 1 for event, 1 for end session
337-
req1 = mock_req.request_history[-1].json()
338-
req2 = mock_req.request_history[-2].json()
337+
# Find update session requests
338+
update_requests = [r for r in mock_req.request_history if "/v2/update_session" in r.url]
339+
assert len(update_requests) >= 2
339340

340-
session_1_req = req1 if req1["session"]["session_id"] == session_1.session_id else req2
341-
session_2_req = req2 if req2["session"]["session_id"] == session_2.session_id else req1
341+
# Get the last two update requests
342+
req1 = update_requests[-1].json()
343+
req2 = update_requests[-2].json()
342344

345+
# Match requests to sessions
346+
session_1_req = req1 if req1["session"]["session_id"] == str(session_1.session_id) else req2
347+
session_2_req = req2 if req2["session"]["session_id"] == str(session_2.session_id) else req1
348+
349+
# Assert
343350
assert session_1_req["session"]["end_state"] == end_state
344351
assert session_2_req["session"]["end_state"] == end_state
345352

0 commit comments

Comments
 (0)