844409 - `cfx test` no longer works on Fennec

Status: RESOLVED FIXED

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

Description

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

Running cfx test on fennec does not work since https://github.com/mozilla/addon-sdk/commit/a4dac4ac74dbc841e3052912dbc8a756c31f2031

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Pull request 811

I still need to test it extensively with all possible flag combination,
but that should fix mobile issue.

Obviously you identified a leak in my workflow as I'm not used to test on mobile... thanks a lot for doing it!!

So the issue was that cfx test for sdk modules depends on --overload-modules option which appears to be broken on device as we were using local filesystem paths for overload instead of remote device paths.
In this patch I upload SDK module on device and use device path instead.

Comment 2

[Alexandre Poirot [:ochameau]]

Just to tell that I've uploaded a new revision based on Erik's preliminary review.
Feel free to redirect this review to irakli if you are overloaded with private browsing stuffs.

Comment 3

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

Irakli can you review this one for us please?

Comment 4

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Comment on attachment 717503 [details]
Pull request 811

Alex I think it's ok to land it as is, but we should probably include files into .xpi insntead of copying them one by one to the device. I'm ok with doing that in a followup change.

Comment 5

[[github robot]]

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

https://github.com/mozilla/addon-sdk/commit/403eb1f3ca19bdc33ba2da5c1b53a19e473baf52
Bug 844409: Implement proper support of --overload-modules for mobile.

https://github.com/mozilla/addon-sdk/commit/e91251e4348b3f6baf6303b0653eda64661d66be
Merge pull request #811 from ochameau/overload-mobile

Bug 844409: Implement proper support of --overload-modules for mobile. r=@gozala

Comment 6

[Alexandre Poirot [:ochameau]]

This patch would need to be cherry picked to stabilization otherwise cfx will fail working with fennec on 1.14

Comment 7

[[github robot]]

Commit pushed to stabilization at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/8c1756d635737ba97d94c84881b4c089b3968100
Merge pull request #811 from ochameau/overload-mobile

Bug 844409: Implement proper support of --overload-modules for mobile. r=@gozala(cherry picked from commit e91251e4348b3f6baf6303b0653eda64661d66be)

Comment 8

[[github robot]]

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

https://github.com/mozilla/addon-sdk/commit/403eb1f3ca19bdc33ba2da5c1b53a19e473baf52
Bug 844409: Implement proper support of --overload-modules for mobile.

https://github.com/mozilla/addon-sdk/commit/e91251e4348b3f6baf6303b0653eda64661d66be
Merge pull request #811 from ochameau/overload-mobile