Add additional test

davglass · Sep 7, 2015 · 04a2294 · 04a2294
1 parent 52fa91c
commit 04a2294
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -290,10 +290,16 @@ exports.asMarkDown = function(sorted, customFormat) {
 
 exports.parseJson = function(jsonPath) {
     if (typeof jsonPath !== 'string') {
-        return {};
+        return new Error('did not specify a path');
     }
 
-    var jsonData = fs.readFileSync(jsonPath, {encoding: 'utf8'});
+    var jsonData = '';
+
+    try {
+        jsonData = fs.readFileSync(jsonPath, {encoding: 'utf8'});
+    } catch (err) {
+        return err;
+    }
 
     return JSON.parse(jsonData);
 };
diff --git a/tests/test.js b/tests/test.js
@@ -256,6 +256,15 @@ var tests = {
             assert.notEqual(json, null);
         }
     },
+    'should parse json with errors': {
+        topic: function() {
+            var path = './NotExitingFile.json';
+            return path;
+        },
+        'and check it': function(path) {
+            assert.throws(checker.parseJson(path));
+        }
+    },
 };
 
 vows.describe('license-checker').addBatch(tests).export(module);