[h5,nc]exists: improve validation, speed test

Author: scivision

+stdlib/+hdf5nc/h5exists.m

@@ -11,7 +11,7 @@
 
 arguments
   file (1,1) string {mustBeFile}
-  vars (1,:) string {mustBeNonempty,mustBeNonzeroLengthText}
+  vars string
 end
 
 i = startsWith(vars, "/");

+stdlib/+hdf5nc/ncexists.m

@@ -11,7 +11,7 @@
 
 arguments
   file (1,1) string {mustBeFile}
-  vars (1,:) string {mustBeNonempty,mustBeNonzeroLengthText}
+  vars string
 end
 
 % NOT contains because we want exact string match

+stdlib/+test/TestHDF5.m

@@ -8,7 +8,6 @@
 type = {'single', 'double', 'float32', 'float64', ...
   'int8', 'int16', 'int32', 'int64', ...
   'uint8', 'uint16', 'uint32', 'uint64'}
-vars = {'A0', 'A1', 'A2', 'A3', 'A4'}
 str = {"string", 'char'}
 end
 
@@ -102,16 +101,25 @@ function test_get_variables(tc)
 end
 
 
-function test_exists(tc, vars)
+function test_exists(tc)
 import stdlib.hdf5nc.h5exists
 basic = tc.TestData.basic;
 
-e = h5exists(basic, "/" + vars);
+tc.verifyEmpty(h5exists(basic, string.empty))
+
+e = h5exists(basic, "");
+
 tc.verifyTrue(isscalar(e))
-tc.verifyTrue(e)
+tc.verifyFalse(e)
+
+vars = {'A0', 'A1', 'A2', 'A3', 'A4', '/A0'};
+
+e = h5exists(basic, vars);
+tc.verifyTrue(isvector(e))
+tc.verifyTrue(all(e))
 
 % vector
-e = h5exists(basic, [vars, "oops"]);
+e = h5exists(basic, ["/A1", "oops"]);
 tc.verifyTrue(isrow(e))
 tc.verifyEqual(e, [true, false])
 end

+stdlib/+test/TestNetCDF.m

@@ -8,7 +8,6 @@
 type = {'single', 'double', 'float32', 'float64', ...
   'int8', 'int16', 'int32', 'int64', ...
   'uint8', 'uint16', 'uint32', 'uint64'}
-vars = {'A0', 'A1', 'A2', 'A3', 'A4'}
 end
 
 
@@ -93,20 +92,23 @@ function test_get_variables(tc)
 end
 
 
-function test_exists(tc, vars)
+function test_exists(tc)
 import stdlib.hdf5nc.ncexists
-import matlab.unittest.constraints.IsScalar
 basic = tc.TestData.basic;
 
-e = ncexists(basic, vars);
-tc.verifyThat(e, IsScalar)
-tc.verifyTrue(e)
+tc.verifyEmpty(ncexists(basic, string.empty))
+
+e = ncexists(basic, "");
+
+tc.verifyTrue(isscalar(e))
+tc.verifyFalse(e)
 
-% vector
-e = ncexists(basic, [vars, "oops"]);
-tc.verifyTrue(isrow(e))
+e = ncexists(basic, ["A1", "oops"]);
+tc.verifyTrue(isvector(e))
 tc.verifyEqual(e, [true, false])
 
+e = ncexists(basic, {'A0', 'A1', 'A2', 'A3', 'A4'});
+tc.verifyTrue(all(e))
 end