filesystem_type, get_owner: require path to exist

Author: scivision

+stdlib/checkRAM.m

@@ -8,7 +8,7 @@
 function [OK,newSizeBytes,freebytes] = checkRAM(newSize, myclass)
 arguments
   newSize (1,:) {mustBeNumeric}
-  myclass (1,1) string
+  myclass {mustBeTextScalar}
 end
 
 % get available RAM

+stdlib/create_symlink.m

@@ -9,8 +9,8 @@
 
 function ok = create_symlink(target, link)
 arguments
-  target (1,1) string
-  link (1,1) string
+  target {mustBeTextScalar}
+  link {mustBeTextScalar}
 end
 
 

+stdlib/disk_available.m

@@ -7,7 +7,7 @@
 
 function f = disk_available(d)
 arguments
-  d (1,1) string
+  d {mustBeTextScalar}
 end
 
 f = javaFileObject(d).getUsableSpace();

+stdlib/disk_capacity.m

@@ -5,7 +5,7 @@
 
 function f = disk_capacity(d)
 arguments
-  d (1,1) string
+  d {mustBeTextScalar}
 end
 
 f = javaFileObject(d).getTotalSpace();

+stdlib/exists.m

@@ -11,7 +11,7 @@
 
 function y = exists(p)
 arguments
-  p (1,1) string
+  p {mustBeTextScalar}
 end
 
 % Matlab >= R2024b allowed URLs to act like files or folders.

+stdlib/filesystem_type.m

@@ -8,19 +8,17 @@
 
 function t = filesystem_type(p)
 arguments
-  p (1,1) string = ""
+  p {mustBeTextScalar} = ''
 end
 
-t = "";
-
-if strlength(p) && ~stdlib.exists(p), return, end
+assert(stdlib.exists(p), "Path does not exist: %s", p);
 
 op = javaPathObject(p);
 
 if stdlib.isoctave()
   t = javaMethod("getFileStore", "java.nio.file.Files", op).type;
 else
-  t = string(java.nio.file.Files.getFileStore(op).type);
+  t = java.nio.file.Files.getFileStore(op).type.string;
 end
 
 end

+stdlib/get_owner.m

@@ -7,22 +7,21 @@
 % * n: owner, or empty if path does not exist
 function n = get_owner(p)
 arguments
-  p (1,1) string
+  p {mustBeTextScalar}
 end
 
 % https://docs.oracle.com/en/java/javase/22/docs/api/java.base/java/nio/file/Files.html#getOwner(java.nio.file.Path,java.nio.file.LinkOption...)
 % https://docs.oracle.com/en/java/javase/22/docs/api/java.base/java/nio/file/LinkOption.html
 
-n = "";
-if ~stdlib.exists(p), return, end
+assert(stdlib.exists(p), "Path %s does not exist.", p)
 
 op = javaPathObject(p);
 opt = javaLinkOption();
 
 if stdlib.isoctave()
   n = javaMethod("getOwner", "java.nio.file.Files", op, opt).toString();
 else
-  n = string(java.nio.file.Files.getOwner(op, opt));
+  n = java.nio.file.Files.getOwner(op, opt).string;
 end
 
 end

test/TestJava.m

@@ -1,7 +1,7 @@
 classdef TestJava < matlab.unittest.TestCase
 
 properties (TestParameter)
-Ps = {".", "", "not-exist"}
+Ps = {"."}
 end
 
 methods(TestClassSetup)