Skip to content

test(ios): await clear-all completion state - #38

Merged
AnganSamadder merged 2 commits into
mainfrom
fix/ci-clear-all-completion-fixture
Aug 25, 2026
Merged

test(ios): await clear-all completion state#38
AnganSamadder merged 2 commits into
mainfrom
fix/ci-clear-all-completion-fixture

Conversation

@AnganSamadder

Copy link
Copy Markdown
Owner

Summary

  • replace fixed Task.yield polling with an awaited clear-all completion observation
  • cover both clear-all success and failure fixtures with the deterministic helper
  • prepare version 0.1.16 (build 131)

Failure evidence

  • post-merge CI run 32860441155 failed AppStoreTests.testClearAllUserData_keepsFailureVisibleAndStopsRemainingCloudWork after the fixed-yield poll raced the MainActor completion task
  • the product path remained green under hosted ASan, hosted TSan, local standard, and Xcode Cloud

Verification

  • targeted failing selector: 20/20 iterations passed
  • AppStoreTests: 79/79 passed
  • full standard iOS: 2790/2790 passed, 0 skipped
  • bun run ci: passed, including 573/573 backend tests

Copilot AI lite review requested due to automatic review settings August 25, 2026 15:52
@vercel

vercel Bot commented Aug 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
payback Ready Ready Preview Aug 25, 2026 3:52pm

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@AnganSamadder
AnganSamadder merged commit f25b2a1 into main Aug 25, 2026
11 checks passed
@AnganSamadder
AnganSamadder deleted the fix/ci-clear-all-completion-fixture branch August 25, 2026 16:44
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.

2 participants