Closed Bug 1060843 Opened 12 years ago Closed 11 years ago

Error displayed when running places test addon

Categories

(Add-on SDK Graveyard :: General, defect, P2)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: evold, Assigned: jsantell)

Details

Attachments

(1 file)

$ cfx testaddons -f places on nightly Testing places... Using binary at '/Applications/FirefoxNightly.app/Contents/MacOS/firefox-bin'. Using profile at '/var/folders/lc/2v6jzk5525gd4l6zr5hxt2yc0000gn/T/tmplMTvLN.mozrunner'. Running tests on Firefox 34.0a1/Gecko 34.0a1 ({ec8030f7-c20a-464f-9b0e-13a3a9e97384}) under darwin/x86_64. console.info: places: executing 'places/main.test async walk' console.info: places: pass: function applied to each node from root console.info: places: pass: function applied to each node from root console.info: places: pass: function applied to each node from root console.info: places: pass: function applied to each node from root console.info: places: pass: function applied to each node from root console.info: places: pass: function applied to each node from root console.info: places: pass: child should wait for parent to complete console.info: places: pass: child should wait for parent to complete console.info: places: executing 'places/main.test bookmark-item-added' console.info: places: pass: correct type in bookmark-added event console.info: places: pass: correct data.type in bookmark-added event console.info: places: pass: correct data.id in bookmark-added event console.info: places: pass: correct data.parentId in bookmark-added event console.info: places: pass: correct data.index in bookmark-added event console.info: places: pass: correct data.url in bookmark-added event console.info: places: pass: correct data.dateAdded in bookmark-added event console.error: places: Message: [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsINavBookmarksService.getItemLastModified]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" location: "JS frame :: resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///var/folders/lc/2v6jzk5525gd4l6zr5hxt2yc0000gn/T/tmplMTvLN.mozrunner/extensions/test-places@jetpack.xpi!/bootstrap.js -> resource://extensions.modules.test-places-at-jetpack.commonjs.path/toolkit/loader.js -> resource://extensions.modules.test-places-at-jetpack.commonjs.path/sdk/places/host/host-bookmarks.js :: createBookmarkItem :: line 94" data: no] Stack: createBookmarkItem@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///var/folders/lc/2v6jzk5525gd4l6zr5hxt2yc0000gn/T/tmplMTvLN.mozrunner/extensions/test-places@jetpack.xpi!/bootstrap.js -> resource://extensions.modules.test-places-at-jetpack.commonjs.path/toolkit/loader.js -> resource://extensions.modules.test-places-at-jetpack.commonjs.path/sdk/places/host/host-bookmarks.js:94:19 execute@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///var/folders/lc/2v6jzk5525gd4l6zr5hxt2yc0000gn/T/tmplMTvLN.mozrunner/extensions/test-places@jetpack.xpi!/bootstrap.js -> resource://extensions.modules.test-places-at-jetpack.commonjs.path/toolkit/loader.js -> resource://extensions.modules.test-places-at-jetpack.commonjs.path/sdk/core/promise.js:37:26 Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:865:22 this.PromiseWalker.walkerLoop@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend.js:744:6
Assignee: nobody → evold
Priority: -- → P2
Attached file gh pr 1639
Attachment #8493959 - Flags: review?(evold)
Attachment #8493959 - Flags: review?(evold) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/1b5ef1c704363382903de6f61089685d3295e528 Bug 1060843 - Wait a tick before destroying underlying bookmarks in places tests https://github.com/mozilla/addon-sdk/commit/09baa541115f70b4b4107cef599689d66e4140d6 Merge pull request #1639 from jsantell/1060843 bug 1060843 - wait a tick before clearing up places tests, r=@erikvold
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: