remove duplicate checks

Author: scivision

+stdlib/makedir.m

@@ -10,11 +10,15 @@ function makedir(direc)
 %% to avoid confusing making ./~/mydir
 direc = stdlib.expanduser(direc);
 
-if ~stdlib.strempty(direc) && ~isfolder(direc)
-  [~] = mkdir(direc);
+if ~stdlib.strempty(direc)
+  [s, ~] = mkdir(direc);
 end
 
-ok = isfolder(direc);
+ok = s == 1;
+
+if ~stdlib.matlabOlderThan('R2017b')
+  ok = ok && isfolder(direc);
+end
 
 if nargout == 0
   assert(ok, 'Failed to create directory: %s', direc)

+stdlib/which.m

@@ -61,7 +61,7 @@
   else
     e = stdlib.append(p, '/', cmd);
   end
-  if isfile(e) && stdlib.is_exe(e)
+  if stdlib.is_exe(e)
     if find_all
       exe(end+1) = e; %#ok<AGROW>
     else

test/TestExpanduser.m

@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'impure'}) ...
+          TestTags = {'R2017a', 'impure'}) ...
     TestExpanduser < matlab.unittest.TestCase
 
 properties (TestParameter)

test/TestFileImpure.m

@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'impure'}) ...
+          TestTags = {'R2017a', 'impure'}) ...
     TestFileImpure < matlab.unittest.TestCase
 
 properties (TestParameter)

test/TestSuffix.m

@@ -1,13 +1,13 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestSuffix < matlab.unittest.TestCase
 
 properties (TestParameter)
-p = {{"", ""}, {"/a/b/c", ""}, ...
-  {"/a/b/c/", ""}, {"a/b/c.txt", ".txt"}, ...
-  {"a/a.b/hi.txt", ".txt"}, ...
-  {"a/a.b/matlab", ""}, ...
-  {"a/b/c.txt.gz", ".gz"}, ...
+p = {{'', ''}, {'/a/b/c', ''}, ...
+  {'/a/b/c/', ''}, {'a/b/c.txt', '.txt'}, ...
+  {'a/a.b/hi.txt', '.txt'}, ...
+  {'a/a.b/matlab', ''}, ...
+  {'a/b/c.txt.gz', '.gz'}, ...
   {'.stat', ''}, ...
   {'a/.stat', ''}, ...
   {'.stat.txt', '.txt'}}
@@ -23,6 +23,11 @@ function test_suffix(tc, p)
 else
   tc.verifyEqual(r, p{2})
 end
+
+if ~stdlib.matlabOlderThan('R2017b')
+  tc.verifyEqual(stdlib.suffix(string(p{1})), string(p{2}))
+end
+
 end
 
 end

test/TestSys.m

@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b'}) ...
+          TestTags = {'R2017a'}) ...
     TestSys < matlab.unittest.TestCase
 
 

test/TestTime.m

@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b'}) ...
+          TestTags = {'R2017a'}) ...
     TestTime < matlab.unittest.TestCase
 
 properties (TestParameter)

test/TestVersion.m

@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestVersion < matlab.unittest.TestCase
 
 properties (TestParameter)