read_symlink: always string class

Author: scivision

+stdlib/read_symlink.m

@@ -1,6 +1,7 @@
 %% READ_SYMLINK read symbolic link
 %
 % empty string if path is not a symlink
+% always of string class in Matlab
 
 function r = read_symlink(p)
 arguments
@@ -10,7 +11,7 @@
 
 try
   [ok, r] = isSymbolicLink(p);
-  if ~ok, r = ''; end
+  if ~ok, r = string.empty; end
 catch e
   switch e.identifier
     case "Octave:undefined-function", r = readlink(p);
@@ -23,16 +24,12 @@
         % https://docs.oracle.com/en/java/javase/21/docs/api/java.base/java/nio/file/Files.html#readSymbolicLink(java.nio.file.Path)
         r = java.nio.file.Files.readSymbolicLink(javaPathObject(r)).string;
       else
-        r = '';
+        r = string.empty;
       end
     otherwise, rethrow(e)
   end
 end
 
-if isstring(p)
-  r = string(r);
-end
-
 end
 
 %!test

test/TestSymlink.m

@@ -16,9 +16,9 @@ function setup_symlink(tc)
 
 tc.tempDir = tc.createTemporaryFolder();
 
-tc.link = fullfile(tc.tempDir, "my.lnk");
+tc.link = fullfile(tc.tempDir, 'my.lnk');
 
-tc.target = mfilename("fullpath") + ".m";
+tc.target = strcat(mfilename("fullpath"), '.m');
 
 tc.assumeTrue(stdlib.create_symlink(tc.target, tc.link), ...
     "failed to create test link " + tc.link)
@@ -36,15 +36,17 @@ function test_is_symlink(tc, p)
 
 function test_read_symlink(tc)
 
-tc.verifyEqual(stdlib.read_symlink(""), "")
-tc.verifyEqual(stdlib.read_symlink(''), '')
-tc.verifyEqual(stdlib.read_symlink(tempname), '')
-tc.verifyEqual(stdlib.read_symlink(tc.target), "")
+tc.verifyEmpty(stdlib.read_symlink(""))
+tc.verifyEmpty(stdlib.read_symlink(''))
+tc.verifyEmpty(stdlib.read_symlink(tempname))
+tc.verifyEmpty(stdlib.read_symlink(tc.target))
+
 
 t = stdlib.read_symlink(tc.link);
-tc.verifyNotEmpty(t)
-tc.verifyClass(t, 'string')
-tc.verifyEqual(tc.target, t)
+
+targ = string(tc.target);
+
+tc.verifyEqual(targ, t)
 
 end