improve test coverage

scivision · scivision · commit cb4e25464da5 · 2025-09-17T14:51:01.000-04:00
diff --git a/+stdlib/+sys/cpu_load.m b/+stdlib/+sys/cpu_load.m
@@ -5,9 +5,9 @@
 if ispc()
   cmd = 'pwsh -c "Get-CimInstance -ClassName Win32_Processor | Measure-Object -Property LoadPercentage -Average | Select-Object -ExpandProperty Average"';
 elseif ismac()
-  cmd = "sysctl -n vm.loadavg | awk '{print $2}'";
+  cmd = 'sysctl -n vm.loadavg | awk ''{print $2}''';
 else
-  cmd = "cat /proc/loadavg | awk '{print $1}'";
+  cmd = 'cat /proc/loadavg | awk ''{print $1}''';
 end
 
 [status, m] = system(cmd);
diff --git a/+stdlib/+sys/ram_free.m b/+stdlib/+sys/ram_free.m
@@ -7,7 +7,7 @@
 elseif ismac()
   cmd = 'sysctl -n vm.page_free_count';
 else
-  cmd = "free -b | awk '/Mem:/ {print $4}'";
+  cmd = 'free -b | awk ''/Mem:/ {print $4}''';
 end
 
 [s, m] = system(cmd);
diff --git a/+stdlib/+sys/ram_total.m b/+stdlib/+sys/ram_total.m
@@ -7,7 +7,7 @@
 elseif ismac()
   cmd = 'sysctl -n hw.memsize';
 else
-  cmd = "free -b | awk '/Mem:/ {print $2}'";
+  cmd = 'free -b | awk ''/Mem:/ {print $2}''';
 end
 
 [s, m] = system(cmd);
diff --git a/test/TestAbsolute.m b/test/TestAbsolute.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'impure'}) ...
+          TestTags = {'R2017a', 'impure'}) ...
     TestAbsolute < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestCanonical.m b/test/TestCanonical.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b'}) ...
+          TestTags = {'R2017a'}) ...
   TestCanonical < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestFileImpure.m b/test/TestFileImpure.m
@@ -40,7 +40,7 @@ function test_null_file(tc)
 
 
 function test_makedir(tc)
-d = "test_makedir.dir";
+d = 'test_makedir.dir';
 stdlib.makedir(d)
 
 if stdlib.matlabOlderThan('R2018a')
diff --git a/test/TestHash.m b/test/TestHash.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'impure'}) ...
+          TestTags = {'R2017a'}) ...
     TestHash < matlab.unittest.TestCase
 
 properties
@@ -25,13 +25,11 @@ function create_file(tc)
 tc.assumeGreaterThan(fid, 0);
 fprintf(fid, "hello");
 fclose(fid);
-tc.assumeTrue(isfile(tc.file))
 tc.assertEqual(stdlib.file_size(tc.file), 5)
 
 fid = fopen(tc.empty, "w");
 tc.assumeGreaterThan(fid, 0);
 fclose(fid);
-tc.assumeTrue(isfile(tc.empty))
 tc.assertEqual(stdlib.file_size(tc.empty), 0)
 end
 end
diff --git a/test/TestIni.m b/test/TestIni.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'impure'}) ...
+          TestTags = {'R2017a', 'impure'}) ...
     TestIni < matlab.unittest.TestCase
 
 
diff --git a/test/TestIsAbsolute.m b/test/TestIsAbsolute.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestIsAbsolute < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestIsSubdir.m b/test/TestIsSubdir.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestIsSubdir < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestJoin.m b/test/TestJoin.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestJoin < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestNetCDF.m b/test/TestNetCDF.m
@@ -56,7 +56,7 @@ function setup_file(tc)
 end
 
 
-methods (Test, TestTags={'R2017b'})
+methods (Test, TestTags={'R2017a'})
 
 function test_netcdf_version(tc)
 tc.verifyTrue(stdlib.version_atleast(stdlib.nc_get_version(), '4.3'), "version unexpected")
diff --git a/test/TestNormalize.m b/test/TestNormalize.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestNormalize < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestParent.m b/test/TestParent.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b'}) ...
+          TestTags = {'R2017a'}) ...
     TestParent < matlab.unittest.TestCase
 
 properties (TestParameter)
@@ -12,33 +12,37 @@
 function test_parent(tc, p)
 pr = stdlib.parent(p{1});
 tc.verifyEqual(pr, p{2}, sprintf('parent(%s)', p{1}))
+
+if ~stdlib.matlabOlderThan('R2017b')
+  tc.verifyEqual(stdlib.parent(string(p{1})), string(p{2}))
 end
 
 end
 
 end
+end
 
 
 function p = init_parent()
 
 p = {
-{"", "."}, ...
-{".", "."}, ...
-{"..", "."}, ...
-{"../..", ".."}, ...
+{'', '.'}, ...
+{'.', '.'}, ...
+{'..', '.'}, ...
+{'../..', '..'}, ...
 {'a/', '.'}, ...
-{"a/b", "a"}, ...
-{"ab/.parent", "ab"}, ...
-{"ab/.parent.txt", "ab"}, ...
-{"a/b/../.parent.txt", "a/b/.."}};
+{'a/b', 'a'}, ...
+{'ab/.parent', 'ab'}, ...
+{'ab/.parent.txt', 'ab'}, ...
+{'a/b/../.parent.txt', 'a/b/..'}};
 
 if ispc
-p{end+1} = {"c:/", "c:/"};
-p{end+1} = {"c:\", "c:/"};
-p{end+1} = {"c:/a/b", "c:/a"};
-p{end+1} = {"c:\a/b", "c:/a"};
-p{end+1} = {"c:/a", "c:/"};
-p{end+1} = {"c:", "c:/"};
+p{end+1} = {'c:/', 'c:/'};
+p{end+1} = {'c:\', 'c:/'};
+p{end+1} = {'c:/a/b', 'c:/a'};
+p{end+1} = {'c:\a/b', 'c:/a'};
+p{end+1} = {'c:/a', 'c:/'};
+p{end+1} = {'c:', 'c:/'};
 end
 
 p{end+1} = {'a/b/', 'a'};
diff --git a/test/TestRelative.m b/test/TestRelative.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestRelative < matlab.unittest.TestCase
 
 
diff --git a/test/TestResolve.m b/test/TestResolve.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b'}) ...
+          TestTags = {'R2017a'}) ...
   TestResolve < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestRoot.m b/test/TestRoot.m
@@ -1,5 +1,5 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestRoot < matlab.unittest.TestCase
 
 properties (TestParameter)
diff --git a/test/TestStem.m b/test/TestStem.m
@@ -1,19 +1,19 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestStem < matlab.unittest.TestCase
 
 properties (TestParameter)
 
 p = {...
-{"/a/b/c", "c"}, ...
-{"/a/b/c/", ""}, ...
-{"a/b/c/", ""}, ...
-{"a/b/c.txt", "c"}, ...
-{"a/b/c.txt.gz", "c.txt"}, ...
-{"a/b/.c", ".c"}, ...
+{'/a/b/c', 'c'}, ...
+{'/a/b/c/', ''}, ...
+{'a/b/c/', ''}, ...
+{'a/b/c.txt', 'c'}, ...
+{'a/b/c.txt.gz', 'c.txt'}, ...
 {'a/b/.c', '.c'}, ...
-{".config", ".config"}, ...
-{"", ""}, ...
+{'a/b/.c', '.c'}, ...
+{'.config', '.config'}, ...
+{'', ''}, ...
 {'', ''}
 }
 end
@@ -23,6 +23,10 @@
 
 function test_stem(tc, p)
 tc.verifyEqual(stdlib.stem(p{1}), p{2})
+
+if ~stdlib.matlabOlderThan('R2017b')
+  tc.verifyEqual(stdlib.stem(string(p{1})), string(p{2}))
+end
 end
 
 end
diff --git a/test/TestSubprocess.m b/test/TestSubprocess.m
@@ -19,7 +19,7 @@ function test_dirs(tc)
 end
 
 
-methods (Test, TestTags = "native_exe")
+methods (Test, TestTags = {'native_exe'})
 
 function test_cwd(tc)
 
diff --git a/test/TestWithSuffix.m b/test/TestWithSuffix.m
@@ -1,26 +1,34 @@
 classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'R2017b', 'pure'}) ...
+          TestTags = {'R2017a', 'pure'}) ...
     TestWithSuffix < matlab.unittest.TestCase
 
 
 properties (TestParameter)
-p = {{"foo.h5", ".nc", "foo.nc"},...
-{"c", "", "c"}, ...
-{"c.nc", "", "c"}, ...
-{"", ".nc", ".nc"}, ...
-{"hello.txt.gz", ".bz", "hello.txt.bz"}, ...
-{"a/b.c/hello.txt", ".gz", "a/b.c/hello.gz"}, ...
+p = {{'foo.h5', '.nc', 'foo.nc'},...
+{'c', '', 'c'}, ...
+{'c.nc', '', 'c'}, ...
+{'', '.nc', '.nc'}, ...
+{'hello.txt.gz', '.bz', 'hello.txt.bz'}, ...
+{'a/b.c/hello.txt', '.gz', 'a/b.c/hello.gz'}, ...
 {'a/b/', '.h5', 'a/b/.h5'}, ...
-{"a/b/.h5", '.nc', "a/b/.h5.nc"}, ...
-{".h5", ".nc", ".h5.nc"}, ...
+{'a/b/.h5', '.nc', 'a/b/.h5.nc'}, ...
+{'.h5', '.nc', '.h5.nc'}, ...
 {'a/b', '.nc', 'a/b.nc'}};
 end
 
 
 methods (Test)
+
 function test_with_suffix(tc, p)
 tc.verifyEqual(stdlib.with_suffix(p{1}, p{2}), p{3})
+
+if ~stdlib.matlabOlderThan('R2017b')
+  tc.verifyEqual(stdlib.with_suffix(string(p{1}), p{2}), string(p{3}))
+  tc.verifyEqual(stdlib.with_suffix(p{1}, string(p{2})), string(p{3}))
+  tc.verifyEqual(stdlib.with_suffix(string(p{1}), string(p{2})), string(p{3}))
 end
 end
 
 end
+
+end
