Closed Bug 1190331 Opened 8 years ago Closed 7 years ago

Test coverage for windows extension API


(WebExtensions :: Untriaged, defect)

Not set


(Not tracked)

48.3 - Apr 25


(Reporter: gkrizsanits, Assigned: bsilverberg)



(Whiteboard: [testing] triaged)

Blocks: 1185459
This is still missing coverage for:

* The |onCreated|, |onRemoved|, and |onFocusChanged| events.
* The |get|, |getLastFocused|, |create|, and |remove| API methods.
* The |update| method without the |focused| property.
Component: Extension Compatibility → WebExtensions
Product: Firefox → Toolkit
Flags: blocking-webextensions-
Flags: blocking-webextensions- → blocking-webextensions+
Assignee: nobody → evilpies
Assignee: evilpies → nobody
Assignee: nobody → bob.silverberg
Whiteboard: [testing] triaged
Iteration: --- → 48.2 - Apr 4
The above mentioned list of missing coverage does not seem to be entirely accurate, according to I am going to open some blocking bugs for the specific coverage that is needed.
Depends on: 1261185
Depends on: 1261949
Iteration: 48.2 - Apr 4 → 48.3 - Apr 18
Once both dependencies for this are fixed this can be marked as fixed as well.
Depends on: 1262976
All of the existing dependencies are marked as resolved, which seems to suggest this can also be marked as resolved, although looking at the coverage report at [1] there is still coverage missing for |onFocusChanged| and |getLastFocused|. Kris, are those reports still updated on a weekly basis?

Flags: blocking-webextensions+ → needinfo?(kmaglione+bmo)
No, the coverage tests are currently broken due to some changes in the content process shutdown sequence. I'm hopping to have it fixed this week.
Flags: needinfo?(kmaglione+bmo)
It looks like both |onFocusChanged| [1] and |getLastFocused| [2] are now covered. I will leave this open until we get the coverage reports running again, at which point I will verify that ext-windows.js is fully covered. 

Kris, if you could add a comment to this bug once coverage reports are restored that would be great, thanks.

According to [1], which was run on August 24, ext-windows.js has 100% coverage. Yay!

Closed: 7 years ago
Resolution: --- → FIXED
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.