-
Notifications
You must be signed in to change notification settings - Fork 24
chore: Tests compatible to iPad - WPB-20998 #3746
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
Test Results2 351 tests 2 323 ✅ 3m 6s ⏱️ Results for commit 98e1802. |
johnxnguyen
left a comment
There was a problem hiding this 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.
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
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.
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