nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1756827 validate add_task properties r=jmaher,rpl

Browse source 
Differential Revision: https://phabricator.services.mozilla.com/D139495
This commit is contained in:
Shane Caraveo 2022-02-23 18:22:32 +00:00
parent 3bd2b256cf
commit 256cdcb8e1
2 changed files with 9 additions and 2 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

8
testing/xpcshell/head.js
View file

@ -1529,6 +1529,7 @@ async function schedulePreciseGCAndForceCC(maxCount) {
* Supported properties:
* skip_if : An arrow function which has an expression to be
* evaluated whether the test is skipped or not.
* pref_set: An array of preferences to set for the test, reset at end of test.
* @param func
* A function to be run only if the funcOrProperies is not a function.
* @param isTask
@ -1540,6 +1541,7 @@ async function schedulePreciseGCAndForceCC(maxCount) {
*
* @return the test function that was passed in.
*/
var _gSupportedProperties = ["skip_if", "pref_set"];
var _gTests = [];
var _gRunOnlyThisTest = null;
function add_test(properties, func = properties, isTask = false) {
@ -1547,6 +1549,12 @@ function add_test(properties, func = properties, isTask = false) {
properties = { isTask };
_gTests.push([properties, func]);
} else if (typeof properties == "object") {
// Ensure only documented properties are in the object.
for (let prop of Object.keys(properties)) {
if (!_gSupportedProperties.includes(prop)) {
do_throw(`Task property is not supported: ${prop}`);
}
}
properties.isTask = isTask;
_gTests.push([properties, func]);
} else {

3
toolkit/components/extensions/test/xpcshell/test_ext_persistent_events.js
View file

@ -570,7 +570,7 @@ add_task(async function test_background_restarted() {
// restart the background once the background has been shutdown or
// put to sleep.
add_task(
{ prefs_set: [["extensions.eventPages.enabled", true]] },
{ pref_set: [["extensions.eventPages.enabled", true]] },
async function test_eventpage_startup() {
await AddonTestUtils.promiseStartupManager();
@ -672,6 +672,5 @@ add_task(
await extension.unload();
await AddonTestUtils.promiseShutdownManager();
Services.prefs.setBoolPref("extensions.eventPages.enabled", false);
}
);