Closed Bug 1277983 Opened 3 years ago Closed 3 years ago

Improve existing bookmarks tracker tests

Categories

(Firefox :: Sync, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: lina, Assigned: lina)

References

Details

(Whiteboard: [sync-data-integrity])

Attachments

(2 files)

Splitting this out of bug 1258127 so that it can help us make changes to the tracker (bug 1274496, for example).
The intention of this patch is 2-fold:

* Capture more existing semantics of the tracker that aren't currently tested.
  The intention is that this patch doesn't touch the existing tracker or
  bookmarks engine implementation at all.

* Make structural changes such that later patches that want to ensure the
  same semantics exist using SQL queries are more obvious and limited
  only to things directly related to the new tracker - for example, this patch
  uses tasks/promises even though they aren't necessary here, but will become
  necessary in later patches.

Review commit: https://reviewboard.mozilla.org/r/57712/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/57712/
Comment on attachment 8759877 [details]
Bug 1277983 - Improve bookmarks engine test logging and remove a deprecated call.

Carrying over r+ from bug 1258127, comment 44.
Attachment #8759877 - Flags: review+
Comment on attachment 8759878 [details]
Bug 1277983 - Improve existing bookmarks tracker tests.

Carrying over r+ from bug 1258127, comment 54.
Attachment #8759878 - Flags: review+
Pushed by kcambridge@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/219c434c410e
Improve bookmarks engine test logging and remove a deprecated call. r=kitcambridge
https://hg.mozilla.org/integration/fx-team/rev/9e2fcb41e05a
Improve existing bookmarks tracker tests. r=markh
https://hg.mozilla.org/mozilla-central/rev/219c434c410e
https://hg.mozilla.org/mozilla-central/rev/9e2fcb41e05a
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Whiteboard: [sync-data-integrity]
You need to log in before you can comment on or make changes to this bug.