Enforce function names

diff --git a/.eslintrc b/.eslintrc
index f0c27a945590ca1883449549ee1df7e003b4160d..9f7d67f7c509c28ddd894debbcfc239fe9948883 100644 (file)
--- a/.eslintrc
+++ b/.eslintrc
@@ -32,5 +32,6 @@
         "comma-spacing": ["error"],
         "comma-style": ["error"],
         "func-call-spacing": ["error"],
+        "func-names": ["error"],
     }
 }

diff --git a/app/error-handler.js b/app/error-handler.js
index 96dc9261ef081b4e7e3c184f85c649289e2f8817..f02f5355f59a5dfcd016d42e7ae42372cc01d025 100644 (file)
--- a/app/error-handler.js
+++ b/app/error-handler.js
@@ -5,7 +5,7 @@
 // No ES6 can be used in this file since it's used for the translation
 /* eslint-disable prefer-arrow-callback */
 
-(function() {
+(function _scope() {
     "use strict";
 
     // Fallback for all uncought errors
@@ -53,6 +53,6 @@
         // from being printed to the browser console.
         return false;
     }
-    window.addEventListener('error', function (evt) { handleError(evt, evt.error); });
-    window.addEventListener('unhandledrejection', function (evt) { handleError(evt.reason, evt.reason); });
+    window.addEventListener('error', function onerror(evt) { handleError(evt, evt.error); });
+    window.addEventListener('unhandledrejection', function onreject(evt) { handleError(evt.reason, evt.reason); });
 })();

diff --git a/tests/.eslintrc b/tests/.eslintrc
index fa6fd4029f8893e4422a2aa7cebb2a07e248c715..545fa2ed25ed36a6856f93575177f4d0d4bdd288 100644 (file)
--- a/tests/.eslintrc
+++ b/tests/.eslintrc
@@ -8,6 +8,8 @@
         "sinon": false
     },
     "rules": {
-        "prefer-arrow-callback": 0
+        "prefer-arrow-callback": 0,
+        // Too many anonymous callbacks
+        "func-names": "off",
     }
 }

diff --git a/utils/use_require.js b/utils/use_require.js
index f301e1d2658c5674564f8d39d3ba8f95c94e7f98..9d6f41652f3c0189090da32f6698de9ee298274e 100755 (executable)
--- a/utils/use_require.js
+++ b/utils/use_require.js
@@ -43,7 +43,7 @@ no_copy_files.forEach(file => no_transform_files.add(file));
 
 // util.promisify requires Node.js 8.x, so we have our own
 function promisify(original) {
-    return function () {
+    return function promise_wrap() {
         const args = Array.prototype.slice.call(arguments);
         return new Promise((resolve, reject) => {
             original.apply(this, args.concat((err, value) => {

diff --git a/utils/use_require_helpers.js b/utils/use_require_helpers.js
index 4daa81fb073a9fd947b6ea1d08a22ed1ccbd35bf..90342cb1dfef99ff9ef4a9ccb0f00dcfea4b4dbd 100644 (file)
--- a/utils/use_require_helpers.js
+++ b/utils/use_require_helpers.js
@@ -4,7 +4,7 @@ const path = require('path');
 
 // util.promisify requires Node.js 8.x, so we have our own
 function promisify(original) {
-    return function () {
+    return function promise_wrap() {
         const args = Array.prototype.slice.call(arguments);
         return new Promise((resolve, reject) => {
             original.apply(this, args.concat((err, value) => {