Skip to content

Conversation

@piotr-iohk
Copy link
Collaborator

@piotr-iohk piotr-iohk commented Nov 10, 2025

Adjustments for boost tests for iOS/Android.

Related:
synonymdev/bitkit-ios#213
synonymdev/bitkit-android#462

@piotr-iohk piotr-iohk requested a review from Copilot November 10, 2025 15:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adjusts boost tests to handle platform-specific differences between iOS and Android, primarily addressing the fact that Android shows an additional boost transaction item while iOS does not.

Key changes:

  • Added platform-specific conditional logic using driver.isAndroid to handle different transaction list behaviors
  • Updated element selectors from generic strings to more specific IDs (e.g., 'later_button''BackupIntroViewCancel')
  • Modified text matching strategy to use 'contains' instead of exact matching for better test stability

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
test/specs/boost.e2e.ts Added conditional logic to handle platform differences in boost transaction display, updated text matching strategy
test/helpers/actions.ts Updated element IDs to more specific names, removed iOS platform restriction from timed sheet helpers, and added triggerTimedSheet parameter usage

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@piotr-iohk piotr-iohk merged commit cccafc5 into main Nov 12, 2025
@piotr-iohk piotr-iohk deleted the test/boost branch November 12, 2025 16:40
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