commit 85608884499571ddda34155e349e6cc61cd9b834
author Jesse MacFadyen <purplecabbage@gmail.com> Tue Dec 05 23:03:31 2017 -0800
committer Jesse MacFadyen <purplecabbage@gmail.com> Tue Dec 05 23:03:31 2017 -0800
tree fd1d5bc312f70e19ddb719779796012474478ba4
parent a5bb1fd5bc9152e6639c9a9d4816e674fbc6628c

LINTINGOMGURSOSTRICT

bin/template/cordova/browser_handler.js

diff --git a/bin/template/cordova/browser_handler.js b/bin/template/cordova/browser_handler.js
index ae699e4..bccddb4 100644
--- a/bin/template/cordova/browser_handler.js
+++ b/bin/template/cordova/browser_handler.js
@@ -117,7 +117,7 @@
             var src = path.join(plugin_dir, asset.src);
             var dest = path.join(wwwDest, asset.target);
             var destDir = path.parse(dest).dir;
-            if(destDir != '' && !fs.existsSync(destDir)) {
+            if (destDir !== '' && !fs.existsSync(destDir)) {
                 shell.mkdir('-p', destDir);
             }
 

spec/browser_handler.spec.js

diff --git a/spec/browser_handler.spec.js b/spec/browser_handler.spec.js
index 8fe71d7..ad73be0 100644
--- a/spec/browser_handler.spec.js
+++ b/spec/browser_handler.spec.js
@@ -25,14 +25,14 @@
 describe('Asset install tests', function () {
     var fsstatMock;
     var asset = { itemType: 'asset',
-                  src: path.join('someSrc', 'ServiceWorker.js'),
-                  target: 'ServiceWorker.js' };
+        src: path.join('someSrc', 'ServiceWorker.js'),
+        target: 'ServiceWorker.js' };
     var assetWithPath = { itemType: 'asset',
-                          src: path.join('someSrc', 'reformat.js'),
-                          target: path.join('js', 'deepdown', 'reformat.js') };
+        src: path.join('someSrc', 'reformat.js'),
+        target: path.join('js', 'deepdown', 'reformat.js') };
     var assetWithPath2 = { itemType: 'asset',
-                           src: path.join('someSrc', 'reformat.js'),
-                           target: path.join('js', 'deepdown', 'reformat2.js') };
+        src: path.join('someSrc', 'reformat.js'),
+        target: path.join('js', 'deepdown', 'reformat2.js') };
 
     var plugin_dir = 'pluginDir';
     var wwwDest = 'dest';
@@ -51,7 +51,7 @@
     it('if src is not a directory and asset has no path, should be called with cp, -f', function () {
         var cp = spyOn(shell, 'cp').and.returnValue('-f');
         var mkdir = spyOn(shell, 'mkdir');
-        var exSync = spyOn(fs, 'existsSync').and.returnValue(true);
+        spyOn(fs, 'existsSync').and.returnValue(true);
         fsstatMock = {
             isDirectory: function () {
                 return false;
@@ -82,10 +82,9 @@
         /*
             Now test that a second call to the same dest folder skips mkdir because the first asset call should have created it.
         */
-        var exSync = spyOn(fs, 'existsSync').and.returnValue(true);
+        spyOn(fs, 'existsSync').and.returnValue(true);
         browser_handler.asset.install(assetWithPath2, plugin_dir, wwwDest);
         expect(mkdir.calls.count()).toBe(1); // not called again
 
-
     });
 });