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)
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
Assignee: evold → jsantell
| Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8493959 -
Flags: review?(evold)
| Reporter | ||
Updated•11 years ago
|
Attachment #8493959 -
Flags: review?(evold) → review+
Comment 2•11 years ago
|
||
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
| Assignee | ||
Updated•11 years ago
|
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.
Description
•