Issue 4878 - Start using ESLint for adblockpluscore

test/stub-modules/io.js

+"use strict";
+
 let data = {};
 
 //
 // Fake nsIFile implementation for our I/O
 //
 function FakeFile(path)
 {
   this.path = path;
 }
 FakeFile.prototype =
 {
   get leafName()
   {
     return this.path;
   },
   set leafName(value)
   {
     this.path = value;
   },
-  append: function(path)
+  append(path)
   {
     this.path += path;
   },
-  exists: function()
+  exists()
   {
     return this.path in data;
   },
   get contents()
   {
     return (data[this.path] || {}).contents;
   },
   set contents(value)
   {
     data[this.path] = {lastModified: Date.now()};
     return data[this.path].contents = value;
   },
   get lastModifiedTime()
   {
     return (data[this.path] || {}).lastModified;
   },
   set lastModifiedTime(value)
   {
     return (data[this.path] || {}).lastModified = value;
   },
-  clone: function()
+  clone()
   {
     return new FakeFile(this.path);
   },
   get parent()
   {
-    return {create: function() {}};
+    return {create() {}};
   },
-  normalize: function() {}
+  normalize() {}
 };
 
 exports.IO = {
   lineBreak: "\n",
-  resolveFilePath: function(path)
+  resolveFilePath(path)
   {
     return new FakeFile(path);
   },
-  writeToFile: function(file, generator, callback)
+  writeToFile(file, generator, callback)
   {
     Promise.resolve().then(() =>
     {
-      let data = [];
+      let lines = [];
       for (let line of generator)
-        data.push(line);
-      file.contents = data.join("\n") + "\n";
+        lines.push(line);
+      file.contents = lines.join("\n") + "\n";
     }).then(() => callback(null)).catch(e => callback(e));
   },
-  readFromFile: function(file, listener, callback)
+  readFromFile(file, listener, callback)
   {
     Promise.resolve().then(() =>
     {
       if (!data.hasOwnProperty(file.path))
         throw new Error("File doesn't exist");
 
       let lines = file.contents.split("\n");
       if (lines.length && lines[lines.length - 1] == "")
         lines.pop();
       for (let line of lines)
         listener.process(line);
       listener.process(null);
     }).then(() => callback(null)).catch(e => callback(e));
   },
-  copyFile: function(from, to, callback)
+  copyFile(from, to, callback)
   {
     Promise.resolve().then(() =>
     {
       if (!data.hasOwnProperty(from.path))
         throw new Error("File doesn't exist");
       if (from.path == to.path)
         throw new Error("Cannot copy file to itself");
 
       to.contents = from.contents;
     }).then(() => callback(null)).catch(e => callback(e));
   },
-  renameFile: function(from, newName, callback)
+  renameFile(from, newName, callback)
   {
     Promise.resolve().then(() =>
     {
       if (!data.hasOwnProperty(from.path))
         throw new Error("File doesn't exist");
       if (from.path == newName)
         throw new Error("Cannot move file to itself");
 
       data[newName] = data[from.path];
       delete data[from.path];
     }).then(() => callback(null)).catch(e => callback(e));
   },
-  removeFile: function(file, callback)
+  removeFile(file, callback)
   {
     Promise.resolve().then(() =>
     {
       if (!data.hasOwnProperty(file.path))
         throw new Error("File doesn't exist");
 
       delete data[file.path];
     }).then(() => callback(null)).catch(e => callback(e));
   },
-  statFile: function(file, callback)
+  statFile(file, callback)
   {
     Promise.resolve().then(() =>
     {
       if (file.exists())
       {
         return {
           exists: true,
           isDirectory: false,
           isFile: true,
           lastModified: file.lastModifiedTime
         };
       }
-      else
-      {
-        return {
-          exists: false,
-          isDirectory: false,
-          isFile: false,
-          lastModified: 0
-        };
-      }
+      return {
+        exists: false,
+        isDirectory: false,
+        isFile: false,
+        lastModified: 0
+      };
     }).then(result => callback(null, result)).catch(e => callback(e));
-  },
+  }
 };