Skip to content

test: handle paykit ui switch#163

Merged
piotr-iohk merged 3 commits into
mainfrom
codex/paykit-feature-flag-followups
May 22, 2026
Merged

test: handle paykit ui switch#163
piotr-iohk merged 3 commits into
mainfrom
codex/paykit-feature-flag-followups

Conversation

@piotr-iohk
Copy link
Copy Markdown
Collaborator

@piotr-iohk piotr-iohk commented May 21, 2026


Note

Low Risk
Low-risk test-only changes that mainly adjust E2E setup/navigation; primary risk is increased flakiness if the Paykit toggle/toasts or dev-settings UI changes.

Overview
Ensures Paykit-related E2E coverage runs with the Paykit UI feature enabled by adding a test/helpers/paykit.ts helper that navigates to Dev Settings, scrolls to/toggles PaykitUiToggle, handles the optional enable confirmation dialog, and waits for new PaykitUiEnabledToast/PaykitUiDisabledToast.

Updates navigation helpers with openDevSettings() and modifies paykit.e2e.ts and pubky-profile.e2e.ts to call enablePaykitUi() during beforeEach (and after reinstall mid-test), making tests resilient to the UI being off by default.

Reviewed by Cursor Bugbot for commit cb9378c. Bugbot is set up for automated code reviews on this repo. Configure here.

@piotr-iohk piotr-iohk self-assigned this May 21, 2026
@piotr-iohk piotr-iohk added Android Changes related to bitkit-android iOS Changes related to bitkit-ios labels May 21, 2026
@piotr-iohk
Copy link
Copy Markdown
Collaborator Author

@cursor review

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit cb9378c. Configure here.

@piotr-iohk piotr-iohk merged commit 204c963 into main May 22, 2026
@piotr-iohk piotr-iohk deleted the codex/paykit-feature-flag-followups branch May 22, 2026 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Android Changes related to bitkit-android iOS Changes related to bitkit-ios

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant