test-content-events.test multiple tabs, fails on Fennec

RESOLVED FIXED

Status

Add-on SDK
General
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jsantell, Assigned: jsantell)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

May be a byproduct of 863502
Priority: -- → P1
(Assignee)

Updated

5 years ago
Assignee: nobody → jsantell
Created attachment 782762 [details]
GH PR 1137

This may be the last change for all tests passing on Fennec.. *fingers crossed*
Attachment #782762 - Flags: review?(evold)
Fixes content-events tests, but bug 899357 preventing tabs/windows from passing, so almost all tests!
Also ignores HTTP requests made by about:home on Fennec which queries possible add-ons to install
Attachment #782762 - Flags: review?(evold) → review+
(In reply to Jordan Santell [:jsantell] [@jsantell] from comment #3)
> Also ignores HTTP requests made by about:home on Fennec which queries
> possible add-ons to install

We should make another bug and find a way to disable/avoid this.
New bug for disabling AMO requests during testing: bug 899747

Comment 6

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

https://github.com/mozilla/addon-sdk/commit/0a390cb20c14c123836a49b3ffff2287b014b4e8
Bug 893191, fix content-events test for Fennec, ignoring about:home and http addon requests

https://github.com/mozilla/addon-sdk/commit/49bf73d99222bfaf56d8c85534344dbda5a83c2d
Merge pull request #1137 from jsantell/893131-fix-content-events-test

Fix Bug 893191 Fix test-content-events for Fennec, r+=@erikvold

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.