Skip to content

Conversation

@findms
Copy link
Contributor

@findms findms commented Oct 16, 2025

TaskWPB-20998 [iOS/QA] XCTest with iPad compatibility


Summary

Briefly describe what this PR does.

This PR is about the critical flow tests making it compatible to iPad so that we can run on nightly if all green for ipad as well.
To Achieve this we needed to put some conditions, elements where flow is different from iPhone on iPad.

   @discardableResult
    func onPad(_ block: (() -> Void)? = nil) -> Bool {
        guard UIDevice.current.userInterfaceIdiom == .pad else { return false }
        block?()
        return true
    }

Testing

Describe how to verify the changes locally. Attach screenshots or reports if relevant.

Additional Information

any more info, add here.

Note: 1 Test is still pending to be fixed due to issue https://wearezeta.atlassian.net/browse/WPB-21169

@findms findms changed the title tests making compatible to iPad chore: tests making compatible to iPad - WPB-20998 Oct 17, 2025
@findms findms changed the title chore: tests making compatible to iPad - WPB-20998 chore: Tests compatible to iPad - WPB-20998 Oct 17, 2025
@findms findms added the QA label Oct 20, 2025
@findms findms marked this pull request as ready for review October 30, 2025 10:26
@github-actions
Copy link
Contributor

Test Results

2 351 tests   2 323 ✅  3m 6s ⏱️
  374 suites     28 💤
    3 files        0 ❌

Results for commit 98e1802.

@findms findms requested review from johnxnguyen and netbe October 31, 2025 10:14
Copy link
Collaborator

@johnxnguyen johnxnguyen left a comment

Choose a reason for hiding this comment

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

Looks good, just one question. Also please try to fill in the PR description with something to help the reviewer understand what he changes are about.

Copy link
Collaborator

Choose a reason for hiding this comment

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

question: were these changes part of automatic linting?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Some warnings were flagging, tried resolving them.

@findms findms requested a review from johnxnguyen November 6, 2025 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants