-
Notifications
You must be signed in to change notification settings - Fork 1.3k
sync-integration-history-bookmark-fenix-desktop #7808
sync-integration-history-bookmark-fenix-desktop #7808
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7808 +/- ##
=========================================
Coverage 18.65% 18.65%
Complexity 452 452
=========================================
Files 316 316
Lines 12343 12343
Branches 1621 1621
=========================================
Hits 2303 2303
Misses 9841 9841
Partials 199 199 Continue to review full report at Codecov.
|
3276f14
to
3621344
Compare
3621344
to
cf163cd
Compare
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.
LGTM, just two comments not blocking
typeEmail() | ||
tapOnContinueButton() | ||
typePassword() | ||
sleep(TestAssetHelper.waitingTimeShort) |
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.
Do you have to hide the save login prompt once you sign-in?
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.
No, I have not found an issue with having to hide it. Once the tap on Sign In button is done, the android part of the test finishes so that prompt does not interfere with the rest of this or other tests...
val defaultWebPage = "example.com".toUri()!! | ||
navigationToolbar { | ||
}.enterURLAndEnterToBrowser(defaultWebPage) { | ||
} |
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.
Should be able to I think use .openThreeDotMenu
here without needing to use browserScreen {}
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.
@AaronMT you are right! removing that line. Thanks!
452cf65
to
15cc4cd
Compare
clean code remove unused import fix reviewer comments
15cc4cd
to
5f6740d
Compare
This PR adds two tests so that the sync tests environment is complete in both ways Fenix <-> Desktop.
Pull Request checklist
After merge
To download an APK when reviewing a PR: