Closed
Bug 1472709
Opened 7 years ago
Closed 7 years ago
Integration Tests: sync Bookmark Desktop-iOS
Categories
(Firefox for iOS :: Build & Test, enhancement)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fxios | + | --- |
People
(Reporter: isabel_rios, Assigned: isabel_rios)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
The aim of this test is to go one steps further and check that the sync works not only in one way but on both.
The steps for this test would be:
-Create a bookmark on desktop
-Verify bookmark on iOS
-Create bookmark on iOS
-Verify bookmarks on desktop
This test has to be enabled only for this schema: Fennec_Enterprise_XCUITests_Integration
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → irios.mozilla
Assignee | ||
Comment 1•7 years ago
|
||
Assignee | ||
Comment 2•7 years ago
|
||
Comment on attachment 9023545 [details] [review]
GitHub Pull Request
Would you like to have the Integration tests starting on desktop and then checking on iOS? This PR is only for bookmarks and the test in the PR covers the firs part described in the bug:
-Create a bookmark on desktop
-Verify bookmark on iOS
The second part would be already checked within the other existing test
-Create bookmark on iOS
-Verify bookmarks on desktop
I don't think it is necessary to add that to this test to, but if your thinking is different, it can be added.
Thanks!
Attachment #9023545 -
Flags: review?(npark)
Attachment #9023545 -
Flags: review?(dave.hunt)
Comment 3•7 years ago
|
||
Comment on attachment 9023545 [details] [review]
GitHub Pull Request
This is excellent! :D
Attachment #9023545 -
Flags: review?(dave.hunt) → review+
Assignee | ||
Updated•7 years ago
|
Summary: Integration Tests: sync Bookmark Desktop-iOS-Desktop → Integration Tests: sync Bookmark Desktop-iOS
Updated•7 years ago
|
Attachment #9023545 -
Flags: review?(npark) → review+
Assignee | ||
Comment 4•7 years ago
|
||
Landed on master: https://github.com/mozilla-mobile/firefox-ios/commit/a14b486ebc2212be0f9d61b66ce20bca3e9dccbd
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•