1095442 - enable content worker tests in an e10s test addon

Status: RESOLVED FIXED

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

Description

[Tomislav Jovanovic :zombie]

include test-content-worker-parent, test-page-mod, and anything else as it gets switched to use the new content worker..

this is blocked on switching to mochitestification test suite..

Comment 1

[Tomislav Jovanovic :zombie]

Attachment: Link to Github pull-request: https://github.com/mozilla/addon-sdk/pull/1774

> this is blocked on switching to mochitestification test suite..

i figured i can proceed without waiting for this:
1) our current test suite can handle e10s addons just fine
2) i can just skip-if this for mochitests, until we solve that part


(i plan to add other content worker related tests files to this addon one by one)

Comment 2

[Dave Townsend [:mossop]]

Comment on attachment 8541878 [details] [review]
Link to Github pull-request: https://github.com/mozilla/addon-sdk/pull/1774

Is this just a copy of the tests from the main suite? At some point maybe we can just remove the tests in the main suit then run this one add-on in both non-e10s and e10s modes then we don;t have to have the tests in two places.

Comment 3

[[github robot]]

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

https://github.com/mozilla/addon-sdk/commit/e01cacf68f7bf499b0ab11be912af7fe8232a9d9
bug 1095442 - enable content worker tests in e10s mode

https://github.com/mozilla/addon-sdk/commit/2ec89f13dda3e32e3379bf36627eb8ff1d39f41c
Merge pull request #1774 from zombie/1095442-e10s-content-tests

bug 1095442 - enable content worker tests in e10s mode, r=@Mossop

Comment 4

[Tomislav Jovanovic :zombie]

(In reply to Dave Townsend [:mossop] from comment #2)
> Is this just a copy of the tests from the main suite? At some point maybe we
> can just remove the tests in the main suit then run this one add-on in both
> non-e10s and e10s modes then we don;t have to have the tests in two places.

yeah, i have a vague plan to do something like that, which i hope crystallizes by the time we have the majority of package tests passing in e10s mode.