Skip to content

Conversation

@piotr-iohk
Copy link
Collaborator

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

Description

Enabling boost tests and accompanying adjustments.

Linked Issues/Tasks

Related:
synonymdev/bitkit-e2e-tests#44
synonymdev/bitkit-android#462

Screenshot / Video

Insert relevant screenshot / recording

@piotr-iohk piotr-iohk requested a review from Copilot November 10, 2025 15:27
@piotr-iohk piotr-iohk self-assigned this Nov 10, 2025
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 adds accessibility identifiers to support E2E testing for the boost transaction feature, which allows users to speed up pending Bitcoin transactions using either RBF (Replace-By-Fee) or CPFP (Child-Pays-For-Parent) methods.

  • Adds accessibility identifiers to boost-related UI components for automated testing
  • Implements navigation reset after successful boost completion
  • Enables boost E2E tests in the CI/CD workflow

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Bitkit/Views/Wallets/Sheets/BoostSheet.swift Added accessibility identifiers for boost UI controls and navigation reset on completion
Bitkit/Views/Wallets/Activity/ActivityItemView.swift Added conditional status accessibility identifiers and restructured amount/fee identifiers
Bitkit/Views/Wallets/Activity/ActivityIcon.swift Wrapped icon in Group and added boosting icon identifier
Bitkit/Views/Wallets/Activity/ActivityExplorerView.swift Extended InfoSection to support test identifiers for TXID and boost transaction references
Bitkit/Components/SheetIntro.swift Added accessibility identifiers to sheet intro components using baseTestID pattern
Bitkit/Components/NavigationBar.swift Added accessibility identifier to navigation back button
.github/workflows/e2e-tests.yml Added @boost tag to E2E test workflow

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