Merge pull request #726 from Remi-Gau/fixes

[FIX] add more explicit error message for input parsing of getData

Author: Remi-Gau

src/IO/getData.m

@@ -25,7 +25,16 @@
   addRequired(args, 'opt', @isstruct);
   addRequired(args, 'bidsDir', @isdir);
 
-  parse(args, varargin{:});
+  try
+    parse(args, varargin{:});
+  catch ME
+    if bids.internal.starts_with(ME.message, 'The value of ')
+      msg = sprintf('The following directory does not exist:\n\t%s', varargin{2});
+      errorHandling(mfilename(), 'notADirectory', msg, false);
+    else
+      rethrow(ME);
+    end
+  end
 
   opt = args.Results.opt;
   bidsDir = args.Results.bidsDir;
@@ -64,7 +73,7 @@
                   strjoin(opt.taskName), ...
                   createUnorderedList(bids.query(BIDS, 'tasks')));
 
-    errorHandling(mfilename(), 'noMatchingTask', msg, false, opt.verbosity);
+    errorHandling(mfilename(), 'noMatchingTask', msg, false);
 
   end
 

tests/tests_bids/test_getData.m

@@ -8,6 +8,16 @@
   initTestSuite;
 end
 
+function test_getData_invalid_dir()
+
+  opt = struct([]);
+  bidsDir = fullfile('/foo');
+
+  assertExceptionThrown(@()getData(opt, bidsDir), ...
+                        'getData:notADirectory');
+
+end
+
 function test_getData_error_no_matching_task()
 
   opt = setOptions('dummy');