Merge pull request #401 from SetCodesToFire/dir-check

Add directory existence check

Author: chrisgorgo

tests/bids.spec.js

@@ -151,4 +151,13 @@ var suite = describe('BIDS example datasets ', function() {
             isdone();
         });
     });
+
+    // test for directory path not existing
+    it('checks for non-existing directory', function (isdone) {
+        var options = {ignoreNiftiHeaders: true};
+        validate.BIDS("path", options, function (issues) {
+            assertErrorCode(issues.errors, '68');
+        });
+        isdone();
+    });
 });

utils/issues/list.js

@@ -341,5 +341,10 @@ module.exports = {
         key: 'NO_VALID_DATA_FOUND_FOR_SUBJECT',
         severity: 'error',
         reason: 'No BIDS compatible data found for at least one subject.'
+    },
+    68: {
+      key: 'DIRECTORY_DOES_NOT_EXIST',
+      severity: 'error',
+      reason: 'Either wrong directory is choosen or there is an error in joining the path.'
     }
 };

validators/bids.js

@@ -31,6 +31,7 @@ BIDS = {
      * arguments to the callback.
      */
     start: function (dir, options, callback) {
+      if (fs.existsSync(dir)) {
         var self = BIDS;
         utils.options.parse(options, function (issues, options) {
             if (issues && issues.length > 0) {
@@ -60,6 +61,18 @@ BIDS = {
                 });
             }
         });
+      }else{
+        var filename = path.basename(dir);
+        var issue = new Issue({
+            code: 68,
+            file: {
+                name: filename,
+                path: path.join('.', filename),
+                relativePath: path.join('', filename)
+            }
+        });
+        return issue;
+      }
     },
 
     /*