Repository navigation
test(mosaic): fix flaky invitation revoke feature tests - #10178
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 236b82c The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (3)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 1 remain after this review. 📝 WalkthroughWalkthroughThree organization-profile failure tests now use Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~5 minutes Suggested reviewers: Merge Risk: ⚪ Minimal · up to This test-only change waits for the expected error messages while retaining the related state checks. The empty changeset is valid for a change that does not require a package release; no actionable merge risk remains. Pre-merge checks |
|
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-biometrics
@clerk/expo-google-signin
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/mosaic
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
Description
Fixes a flake in
organization-profile-invitations.feature.test.tsx› "keeps the dialog open with the server message when the invitation is no longer pending" (failed on #10177's CI: alert found, but with empty text).Card.Banneralways renders itsrole="alert"wrapper, empty until a message arrives, sofindByRole('alert')can resolve on the empty region and the text assertion then fails without retrying. The two invitation-revoke assertions and one members-panel assertion that used this pattern now usewaitFor(() => expect(getByRole('alert')).toHaveTextContent(...)), the pattern used elsewhere in the suite.This was meant to land with #10177, which merged before it was pushed.
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change