Skip to content

test(sharing): align provisioning retry lifecycle - #1354

Closed
kunickiaj wants to merge 1 commit into
graphite-base/1354from
07-23-test_sharing_align_provisioning_retry_lifecycle
Closed

test(sharing): align provisioning retry lifecycle#1354
kunickiaj wants to merge 1 commit into
graphite-base/1354from
07-23-test_sharing_align_provisioning_retry_lifecycle

Conversation

@kunickiaj

@kunickiaj kunickiaj commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Description

Align the capability-preflight retry regression with the truthful lifecycle introduced downstack: accepted exact-Project operations begin provisioning immediately, remain provisioning through retryable failures, and move to needs attention on the third failure.

Type of Change

  • 🚀 Feature (new functionality)
  • 🐛 Bug fix (fixes an issue)
  • 📚 Documentation (docs-only change)
  • 🔧 Maintenance (refactor, chore, CI, etc.)
  • 🧪 Testing (test-only changes)

Testing

  • Relevant checks pass locally (pnpm run tsc, pnpm run lint, pnpm run test)
  • Added/updated tests for changes
  • Manually verified changes work as expected

The targeted provisioning retry test and complete pnpm run check passed with 3,207 tests. Automated Project-sharing E2E also passed.

Checklist

  • Code follows project style (pnpm run lint passes for touched files)
  • Self-review completed
  • Documentation updated (if needed)
  • No new warnings introduced

kunickiaj commented Jul 23, 2026

Copy link
Copy Markdown
Owner Author

@kunickiaj
kunickiaj marked this pull request as ready for review July 23, 2026 15:15
@kunickiaj
kunickiaj changed the base branch from 07-23-fix_sharing_clarify_recipient_access_dialogs to graphite-base/1354 July 23, 2026 21:11
@kunickiaj

Copy link
Copy Markdown
Owner Author

Closing as redundant: the lifecycle assertion now lives in #1350, and #1354 no longer carries a unique required change.

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