Use Glob Pattern

Signed-off-by: Airton Zanon <me@airton.dev>

README.md

@@ -11,11 +11,15 @@ Install jsoncs with npm to use the command line interface:
 
 Verify if the code style is fine:
 
-    ./node_modules/.bin/jsoncs my/file.json
+    ./node_modules/.bin/jsoncs 'my/file.json'
+
+or to multiple files
+
+    ./node_modules/.bin/jsoncs 'my/*.json'
 
 Fix the code style of a file or multiple files:
 
-    ./node_modules/.bin/jsoncs --fix my/directory/
+    ./node_modules/.bin/jsoncs --fix 'my/directory/*'
 
 ### Options 
 

lib/cli.js

@@ -40,11 +40,22 @@ function parse(path) {
 }
 
 function processFile(filePath) {
-  let source = fs.readFileSync(filePath, "utf8");
-  let formatted = formatter.formatJson(source);
-  let fileWithErrors = showDiff(program.quiet, source, formatted, filePath);
+  let formatted, source, fileWithErrors;
 
-  errors.push(fileWithErrors);
+  try {
+    source = fs.readFileSync(filePath, "utf8");
+    formatted = formatter.formatJson(source);
+    fileWithErrors = showDiff(program.quiet, source, formatted, filePath);
+
+    errors.push(fileWithErrors);
+  } catch (e) {
+    console.log("Couldn't read the file / directory");
+    console.log("Try to:");
+    console.log("  - use quotes (eg: jsoncs 'my/file.json')");
+    console.log("  - use glob pattern (eg: jsoncs 'my/*.json')");
+    console.log("  - use glob pattern (eg: jsoncs 'my/*/*.json')");
+    process.exit(1);
+  }
 
   if (program.fix) {
     let fixedFile = fixJsonFile(filePath, formatted);

lib/modules.js

@@ -1,17 +1,14 @@
 const fs = require("fs");
 const jsdiff = require("diff");
+const glob = require("glob")
 
 const blue = "\x1b[34m";
 const red = "\x1b[31m";
 const withe = "\x1b[37m";
 
 module.exports = {
   getFiles: (path) => {
-    if (fs.lstatSync(path).isDirectory()) {
-      return readDirectory(path);
-    }
-
-    return [path];
+    return glob.sync(path);
   },
   fixJsonFile: (filePath, formatted) => {
     fs.writeFileSync(filePath, formatted);
@@ -37,14 +34,3 @@ module.exports = {
     return fileWithErrors;
   }
 };
-
-function readDirectory(directoryPath) {
-  let files = fs.readdirSync(directoryPath);
-  let filesPath = [];
-
-  files.forEach((file) => {
-    filesPath.push(directoryPath + file);
-  });
-
-  return filesPath;
-}

package.json

@@ -26,7 +26,8 @@
   },
   "dependencies": {
     "commander": "^3.0.0",
-    "diff": "^4.0.1"
+    "diff": "^4.0.1",
+    "glob": "^7.1.4"
   },
   "devDependencies": {
     "eslint": "^6.2.1",

test/modulesTest.js

@@ -23,13 +23,21 @@ test("test get file path", (assert) => {
 });
 
 test("test get directory path", (assert) => {
-  let path = __dirname + "/fixture/expected-json/";
+  let path = __dirname + "/fixture/expected-json/*.json";
   let result = getFiles(path);
 
   assert.equal(result.length, 2);
   assert.end();
 });
 
+test("test get all files inside a directory path", (assert) => {
+  let path = __dirname + "/fixture/*/*.json";
+  let result = getFiles(path);
+
+  assert.equal(result.length, 5);
+  assert.end();
+});
+
 test("test show diff between two different json objects", (assert) => {
   let json1 = {foo:"bar"};
   let json2 = {ble:"bar"};