Closed Bug 1312031 Opened 4 years ago Closed 3 years ago

TPS failing: test-sync.js ASSERTION FAILED! places item not found; Incorrect keyword - expected: bugzilla, actual: bz for bookmark

Categories

(Firefox :: Sync, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 52
Tracking Status
firefox52 --- fixed

People

(Reporter: tcsc, Assigned: tcsc)

Details

Attachments

(3 files)

Attached file tps.log
Haven't dug into this at all but it seems the keyword isn't getting synced for some reason. No obvious errors.

Attaching tps.log and console output.
Assignee: nobody → tchiovoloni
Priority: -- → P1
I can fairly easily and reliably cause bugs when using programmatic keyword apis, including those in PlacesSyncUtils, but these seem to be bugs in places (As soon as I track them down some more I'll file them), however AFAICT if I only change things via the UI and actual syncs, I'm unable to cause problems.

Very tempting to just remove the keyword manipulation from this test.
Comment on attachment 8806424 [details]
Bug 1312031 - Fix failing TPS test_sync.js test.

https://reviewboard.mozilla.org/r/89858/#review89448
Attachment #8806424 - Flags: review?(markh) → review+
Pushed by tchiovoloni@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b1938dba555d
Fix failing TPS test_sync.js test. r=markh
https://hg.mozilla.org/mozilla-central/rev/b1938dba555d
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
You need to log in before you can comment on or make changes to this bug.