Merge pull request #49 from groupon/parse-no-content

fix: avoid error on empty cson/json
groupon · Mar 15, 2021 · 85ef301 · 85ef301
2 parents 315ab2b + ec50444
commit 85ef301
Show file tree

Hide file tree

Showing 7 changed files with 829 additions and 1,557 deletions.
diff --git a/.gitignore b/.gitignore
@@ -5,4 +5,5 @@ npm-debug.log
 test/tmp
 test/log
 /.idea
-.nyc_output
+/.nyc_output
+/coverage
diff --git a/coffeelint.json b/coffeelint.json
diff --git a/lib/file.js b/lib/file.js
@@ -60,7 +60,14 @@ function isMissingError(error) {
   return error.code === 'ENOENT';
 }
 
-function parseCSON(filename, content) {
+/**
+ * @param {string} filename
+ * @param {string} content
+ * @returns {{}}
+ */
+function parseCSON(filename, content = '') {
+  if (content.trim().length === 0) return {};
+
   try {
     return CSON.parse(content);
   } catch (err) {
@@ -69,7 +76,14 @@ function parseCSON(filename, content) {
   }
 }
 
-function parseJSON(filename, content) {
+/**
+ * @param {string} filename
+ * @param {string} content
+ * @returns {{}}
+ */
+function parseJSON(filename, content = '') {
+  if (content.trim().length === 0) return {};
+
   try {
     return JSON.parse(content);
   } catch (err) {

diff --git a/lib/shared-store.js b/lib/shared-store.js
@@ -38,7 +38,7 @@ const { EventEmitter } = require('events');
 
 const path = require('path');
 
-const freeze = require('deep-freeze');
+const freeze = require('deep-freeze-es6');
 
 const { Observable } = require('rx-lite');