must pay attention to not dupe test from the new API
(In reply to Fred Lin [:gasolin] from comment #1) > search dxr with `nsINavBookmarksService`, the effected files (exclude from > bookmarks/ under bug 1094903) are this bug is only about tests under bookmarks/, maybe you wanted to post this in bug 1094903? Btw, it's not enough to search nsINavBookmarksService, since some tests could be using PlacesUtils.bookmarks.OLDAPI... this is more problematic to search cause also the new API is in PlacesUtils.bookmarks. We may have to go test by test...
After bug 1415835, there's still a few tests remaining that need transitioning, but I probably won't be picking them up until closer to the time we do more work on removing the async APIs. Some notes: - test_417228-exclude-from-backup.js - test_417228-other-roots.js -- insert folders into the root, the new APIs won't allow this, so we either need to drop it, or write sql injection for testing these. - test_bookmarks_notifications,js -- Largely async based, for some reason has one or two sync APIs in. Needs investigation. - test_bookmarks.js -- See bug 1403487 - test_bookmarks_treecache.js -- Probably needs sql injection as a replacement. - test_keywords.js -- Probably just needs converting - test_nsINavBookmarkObserver.js -- Seems to be the sync version of test_bookmarks_notifications.js but needs investigating to ensure all tests are covered by the new one. - test_sync_fields.js -- Appears to test both sync and async APIs already.
Assignee: standard8 → nobody
Status: ASSIGNED → NEW
At this point we will directly fix these tests when removing the APIs.
Status: NEW → RESOLVED
Last Resolved: 24 days ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.