853334 - require testaddon fails on fennec

Status: RESOLVED FIXED

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

Description

[Erik Vold [:erikvold] (please needinfo? me)]

Testing require...
Launching mobile application with intent name org.mozilla.fennec
Killing running Firefox instance ...
Pushing the addon to your device
Starting: Intent { act=android.activity.MAIN cmp=org.mozilla.fennec/.App (has extras) }
--------- beginning of /dev/log/main
--------- beginning of /dev/log/system
Could not read chrome manifest 'file:///data/data/org.mozilla.fennec/chrome.manifest'.
zerdatime 1363848059137 - browser chrome startup finished.
Remote debugger listening on port 6000
OpenGL LayerManager Initialized Succesfully.
Version: OpenGL ES 2.0 build 1.8@905891
Vendor: Imagination Technologies
Renderer: PowerVR SGX 540
FBO Texture Target: TEXTURE_2D
Running tests on Fennec 22.0a1/Gecko 22.0a1 ({aa3c5121-dab2-40e2-81ca-7ea25febc110}) under android/arm-eabi-gcc3.
info: require: executing 'require/main.test 3rd party vs sdk module'
info: require: pass: Third party modules don't overload sdk modules
info: require: pass: page-mod module is really the sdk one
info: require: pass: panel/page-mod is the 3rd party
info: require: pass: Third party main module overload sdk modules
info: require: pass: require(panel) maps to require(panel/main)
info: require: pass: require(./panel) maps to the local module
error: require: An exception occurred.
Error: Unsupported Application: The module sdk/panel currently supports only Firefox.
resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/loader/cuddlefish.js 80
Traceback (most recent call last):
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/timers.js", line 31, in notify
callback.apply(null, args);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/test/runner.js", line 48, in 
onDone: onDone
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/test/harness.js", line 443, in runTests
nextIteration();
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/test/harness.js", line 273, in nextIteration
findAndRunTests(loader, nextIteration);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/test/runner.js", line 127, in findAndRunTests
onDone: nextIteration
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/deprecated/unit-test.js", line 445, in startMany
runNextTest(this);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/deprecated/unit-test.js", line 440, in runNextTest
self.start({test: test, onDone: runNextTest});
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/deprecated/unit-test.js", line 471, in start
this.test.testFunction(this);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/test.js", line 72, in 
test(assert);
File "resource://test-require-at-jetpack/require/lib/main.js", line 47, in exports["test 3rd party vs sdk module"]
assert.ok(require("sdk/panel").Panel,
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/toolkit/loader.js", line 339, in require
freeze(load(loader, module));
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/loader/cuddlefish.js", line 133, in options<.load
error = incompatibility(module) || error;
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/loader/cuddlefish.js", line 80, in incompatibility
" currently supports only " + applications.join(", ") + ".")
info: require: executing 'require/main.test local vs sdk module'
info: require: pass: Local module takes the priority over sdk modules
info: require: pass: this module is really the local one
info: require: executing 'require/main.testMultipleRequirePerLine'
info: require: pass: undefined
info: require: pass: undefined
info: require: executing 'require/main.testRelativeRequire'
info: require: pass: undefined
info: require: executing 'require/main.testRelativeSubFolderRequire'
info: require: pass: undefined
info: require: executing 'require/main.testSDKRequire'
error: require: An exception occurred.
Error: Unsupported Application: The module sdk/widget currently supports only Firefox.
resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/loader/cuddlefish.js 80
Traceback (most recent call last):
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/timers.js", line 31, in notify
callback.apply(null, args);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/deprecated/unit-test.js", line 288, in 
timer.setTimeout(function() { onDone(self); }, 0);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/deprecated/unit-test.js", line 458, in 
options.onDone(self);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/deprecated/unit-test.js", line 440, in runNextTest
self.start({test: test, onDone: runNextTest});
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/deprecated/unit-test.js", line 471, in start
this.test.testFunction(this);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/test.js", line 72, in 
test(assert);
File "resource://test-require-at-jetpack/require/lib/main.js", line 73, in exports.testSDKRequire
assert.deepEqual(Object.keys(require('sdk/widget')), ['Widget']);
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/toolkit/loader.js", line 339, in require
freeze(load(loader, module));
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/loader/cuddlefish.js", line 133, in options<.load
error = incompatibility(module) || error;
File "resource://extensions.modules.test-require-at-jetpack.commonjs.path/sdk/loader/cuddlefish.js", line 80, in incompatibility
" currently supports only " + applications.join(", ") + ".")
12 of 14 tests passed.

Comment 1

[Erik Vold [:erikvold] (please needinfo? me)]

Can you take a look at this one Alex?

Comment 2

[Alexandre Poirot [:ochameau]]

Sure

Comment 3

[Alexandre Poirot [:ochameau]]

Attachment: Pull request 884

Comment 4

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/1a19c1ab738821a0ff1bb90dc2af2276a2e43660
Bug 853334 - Fix testaddons on fennec

https://github.com/mozilla/addon-sdk/commit/924fd2f296aa7f1183ce1cc49fc12a03de5d8f0c
Merge pull request #884 from ochameau/tests-addons-fennec

Fix Bug 853334 - Fix testaddons on fennec r=@erikvold

Fix Bug 853335