Closed Bug 1236122 Opened 9 years ago Closed 7 years ago

Complete test coverage for the WebExtension webRequest API.

Categories

(WebExtensions :: General, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: kmag, Unassigned)

References

Details

(Whiteboard: [test] triaged)

This is still missing coverage for:

* Events with filters with |types|, |tabId|, or |windowId| properties.
* Attempt to use blocking listeners without the appropriate permissions.
* Initial process data with any initial content policy data.
* |addContentPolicy| and |WebRequest:RemoveContentPolicy| calls where |contentPolicies.size| is anything other than 0.
* |shouldLoad| calls with non-blocking policies.
* |typeMatches| calls with non-null |filterTypes|.
* |urlMatches| calls with a null |urlFilter|.
* |addListener| without a |urls| property in the filter.
* The |getHeaders| method.
* |runChannelListener| with |opts.requestHeaders| or |opts.responseHeaders| in the listener options, or |result.requestHeaders|, |result.responseHeaders|, or |result.cancel| in the listener result.
* |examine| calls with |onStart.size| of 0, and |onStop.size| of both 0 and non-0.


https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/components/extensions/ext-webRequest.js.html
https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/modules/addons/WebRequestContent.js.html
https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/modules/addons/WebRequestCommon.jsm.html
https://people.mozilla.org/~kmaglione/webextension-test-coverage/toolkit/modules/addons/WebRequest.jsm.html
Priority: -- → P3
Whiteboard: [test] triaged
Flags: blocking-webextensions?
Flags: blocking-webextensions? → blocking-webextensions+
Depends on: 1176092
QA Contact: g.maone
Component: WebExtensions: Untriaged → WebExtensions: General
Flags: blocking-webextensions+
We have pretty extensive coverage at this point.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
Product: Toolkit → WebExtensions
You need to log in before you can comment on or make changes to this bug.