Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: qunit/tests/prefs.js

Issue 29555663: Noissue - Fix prefs unit tests, account for a changed default value (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome
Patch Set: Created Sept. 25, 2017, 10:43 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: qunit/tests/prefs.js
===================================================================
--- a/qunit/tests/prefs.js
+++ b/qunit/tests/prefs.js
@@ -42,19 +42,19 @@
{
deepEqual(items[key], expectedValue, description);
done();
});
}
test("Numerical pref", assert =>
{
- Prefs.patternsbackups = 5;
+ Prefs.patternsbackups = 0;
equal(
- Prefs.patternsbackups, 5,
+ Prefs.patternsbackups, 0,
"Prefs object returns the correct value after setting pref to " +
"default value"
);
checkPrefExists(
"patternsbackups", false, "User-defined pref has been removed", assert
);
Prefs.patternsbackups = 12;
equal(
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld