Skip to content

Commit a5cfb49

Browse files
committed
Fix flaky asc pagination test: verify cursor via pagination instead of requiring grouped count >= 2
1 parent a670103 commit a5cfb49

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

tests/integration/test_activity.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -143,11 +143,16 @@ async def test_pagination_since_desc_uses_min_id(self, nc_mcp: McpTestHelper) ->
143143
@pytest.mark.asyncio
144144
async def test_pagination_since_asc_uses_max_id(self, nc_mcp: McpTestHelper) -> None:
145145
await _generate_activity(nc_mcp)
146-
result = await nc_mcp.call("get_activity", sort="asc", limit=5)
146+
result = await nc_mcp.call("get_activity", sort="asc", limit=1)
147147
parsed = json.loads(result)
148148
data = parsed["data"]
149-
assert len(data) >= 2
150-
assert parsed["pagination"]["since"] == max(a["activity_id"] for a in data)
149+
assert len(data) >= 1
150+
since = parsed["pagination"]["since"]
151+
assert since == max(a["activity_id"] for a in data)
152+
result2 = await nc_mcp.call("get_activity", sort="asc", limit=5, since=since)
153+
data2 = json.loads(result2)["data"]
154+
if data2:
155+
assert all(a["activity_id"] > since for a in data2)
151156

152157
@pytest.mark.asyncio
153158
async def test_invalid_filter_raises(self, nc_mcp: McpTestHelper) -> None:

0 commit comments

Comments
 (0)