Skip to content

test: stabilize activity list pagination - #1171

Open
chaptersix wants to merge 1 commit into
temporalio:mainfrom
chaptersix:codex/activity-list-pagination-stable
Open

test: stabilize activity list pagination#1171
chaptersix wants to merge 1 commit into
temporalio:mainfrom
chaptersix:codex/activity-list-pagination-stable

Conversation

@chaptersix

Copy link
Copy Markdown
Contributor

Related issues

N/A

What changed?

Make TestActivity_List_Pagination exercise a deterministic mix of three completed and two running standalone Activities.

The test now waits for completed results, blocks running Activities until pagination finishes, and waits for visibility to report the exact status mix before listing. It also verifies each Activity appears exactly once with the expected status, avoiding the race where an Activity could complete and move in visibility ordering between page requests.

Checklist

Stability

  • No user-facing behavior or output changes

Tests

  • Updated functional test (SharedServerSuite)
  • Focused pagination test passed 20 consecutive runs
  • Neighboring Activity list/count tests passed

Testing

go test ./internal/temporalcli -run 'TestSharedServerSuite/TestActivity_List_Pagination$' -count=20
go test ./internal/temporalcli -run 'TestSharedServerSuite/TestActivity_(List(_Pagination)?|Count)$' -count=1
git diff --check

@chaptersix
chaptersix marked this pull request as ready for review August 20, 2026 15:25
@chaptersix
chaptersix requested a review from a team as a code owner August 20, 2026 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant