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

Unified Diff: test/notification.js

Issue 29375915: Issue 4878 - Start using ESLint for adblockpluscore (Closed)
Patch Set: Rebased Created March 10, 2017, 7:02 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 | « test/matcher.js ('k') | test/regexpFilters_matching.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/notification.js
diff --git a/test/notification.js b/test/notification.js
index 921f2b69d1e00784eb88de6d6132b96424a84eb3..0bdf17031368090c548ded48ed3d160f85173af0 100644
--- a/test/notification.js
+++ b/test/notification.js
@@ -21,7 +21,6 @@ let {
createSandbox, setupTimerAndXMLHttp, setupRandomResult, unexpectedError, Cr
} = require("./_common");
-let info = null;
let Prefs = null;
let Notification = null;
@@ -34,7 +33,6 @@ exports.setUp = function(callback)
let sandboxedRequire = createSandbox({globals});
(
- info = sandboxedRequire("./stub-modules/info"),
{Prefs} = sandboxedRequire("./stub-modules/prefs"),
{Notification} = sandboxedRequire("../lib/notification")
);
@@ -59,9 +57,13 @@ function showNotifications(url)
function* pairs(array)
{
for (let element1 of array)
+ {
for (let element2 of array)
+ {
if (element1 != element2)
yield [element1, element2];
+ }
+ }
}
function registerHandler(notifications, checkCallback)
@@ -172,7 +174,7 @@ for (let [propName, value, result] of [
["platformMaxVersion", "12.0", true],
["platformMaxVersion", "12", true],
["platformMaxVersion", "13", true],
- ["platformMaxVersion", "11", false],
+ ["platformMaxVersion", "11", false]
])
{
exports.testTargetSelection[`${propName}=${value}`] = function(test)
@@ -211,7 +213,7 @@ for (let [[propName1, value1, result1], [propName2, value2, result2]] of pairs([
["platform", "chromium", true],
["platform", "gecko", false],
["platformMinVersion", "12", true],
- ["platformMinVersion", "13", false],
+ ["platformMinVersion", "13", false]
]))
{
exports.testMultipleTargets[`${propName1}=${value1},${propName2}=${value2}`] = function(test)
@@ -231,10 +233,10 @@ for (let [[propName1, value1, result1], [propName2, value2, result2]] of pairs([
registerHandler.call(this, [information]);
this.runScheduledTasks(1).then(() =>
{
- let expected = (result1 || result2 ? [information] : [])
+ let expected = (result1 || result2 ? [information] : []);
test.deepEqual(showNotifications(), expected, "Selected notification for " + JSON.stringify(information.targets));
}).catch(unexpectedError.bind(test)).then(() => test.done());
- }
+ };
}
exports.testParametersSent = function(test)
@@ -242,11 +244,11 @@ exports.testParametersSent = function(test)
Prefs.notificationdata = {
data: {
version: "3"
- },
+ }
};
let parameters = null;
- registerHandler.call(this, [], function(metadata)
+ registerHandler.call(this, [], metadata =>
{
parameters = decodeURI(metadata.queryString);
});
@@ -335,7 +337,7 @@ exports.testUsingSeverityInsteadOfType = function(test)
let responseText = JSON.stringify({
notifications: [severityNotification]
});
- Notification._onDownloadSuccess({}, responseText, function() {}, function() {});
+ Notification._onDownloadSuccess({}, responseText, () => {}, () => {});
};
exports.testURLSpecificNotification = function(test)
@@ -505,7 +507,7 @@ exports.testLanguageOnly = function(test)
exports.testLanguageAndCountry = function(test)
{
- let notification = {message: {fr: "fr", "fr-CA": "fr-CA"}};
+ let notification = {message: {"fr": "fr", "fr-CA": "fr-CA"}};
let texts = Notification.getLocalizedTexts(notification, "fr-CA");
test.equal(texts.message, "fr-CA");
texts = Notification.getLocalizedTexts(notification, "fr");
« no previous file with comments | « test/matcher.js ('k') | test/regexpFilters_matching.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld