filesystem_type: functionalize, test

Author: scivision

+stdlib/+dotnet/filesystem_type.m

@@ -0,0 +1,11 @@
+function t = filesystem_type(p)
+
+if ~stdlib.exists(p)
+  t = string.empty;
+  return
+end
+
+t = string(System.IO.DriveInfo(stdlib.absolute(p)).DriveFormat);
+% https://learn.microsoft.com/en-us/dotnet/api/system.io.driveinfo.driveformat
+
+end

+stdlib/+java/filesystem_type.m

@@ -1,5 +1,10 @@
 function t = filesystem_type(p)
 
+if ~stdlib.exists(p)
+  t = string.empty;
+  return
+end
+
 t = string(javaMethod("getFileStore", "java.nio.file.Files", javaPathObject(p)).type);
 
 end

+stdlib/+python/filesystem_type.m

@@ -1,6 +1,9 @@
 function t = filesystem_type(p)
 
-t = string.empty;
+if ~stdlib.exists(p)
+  t = string.empty;
+  return
+end
 
 pr = stdlib.absolute(p);
 

+stdlib/+sys/filesystem_type.m

@@ -1,5 +1,10 @@
 function t = filesystem_type(p)
 
+if ~stdlib.exists(p)
+  t = string.empty;
+  return
+end
+
 if ispc()
   dl = extractBefore(stdlib.absolute(p), 2);
   cmd = "pwsh -c (Get-Volume -DriveLetter " + dl + ").FileSystem";
@@ -11,9 +16,9 @@
 
 [s, t] = system(cmd);
 if s == 0
-  t = strip(t);
+  t = string(strip(t));
 else
-  t = "";
+  t = string.empty;
 end
 
 end

+stdlib/filesystem_type.m

@@ -5,12 +5,10 @@
 function t = filesystem_type(p)
 
 
-t = "";
-if ~stdlib.exists(p), return, end
+t = string.empty;
 
 if stdlib.has_dotnet()
-  t = System.IO.DriveInfo(stdlib.absolute(p)).DriveFormat;
-  % https://learn.microsoft.com/en-us/dotnet/api/system.io.driveinfo.driveformat
+  t = stdlib.dotnet.filesystem_type(p);
 elseif stdlib.has_java()
   t = stdlib.java.filesystem_type(p);
 elseif stdlib.has_python()
@@ -21,10 +19,4 @@
   t = stdlib.sys.filesystem_type(p);
 end
 
-try  %#ok<*TRYNC>
-  t = string(t);
 end
-
-end
-
-%!assert(!isempty(filesystem_type(pwd)))

test/TestDisk.m

@@ -10,6 +10,7 @@
 device_fun = {@stdlib.device, @stdlib.sys.device, @stdlib.java.device, @stdlib.python.device}
 disk_available_fun = {@stdlib.disk_available, @stdlib.sys.disk_available, @stdlib.dotnet.disk_available, @stdlib.java.disk_available, @stdlib.python.disk_available}
 disk_capacity_fun  = {@stdlib.disk_capacity,  @stdlib.sys.disk_capacity,  @stdlib.dotnet.disk_capacity,  @stdlib.java.disk_capacity,  @stdlib.python.disk_capacity}
+fst_fun = {@stdlib.filesystem_type, @stdlib.sys.filesystem_type, @stdlib.dotnet.filesystem_type, @stdlib.java.filesystem_type, @stdlib.python.filesystem_type}
 owner_fun = {@stdlib.get_owner, @stdlib.sys.get_owner, @stdlib.dotnet.get_owner, @stdlib.java.get_owner, @stdlib.python.get_owner}
 end
 
@@ -49,19 +50,18 @@ function test_hard_link_count(tc)
 end
 
 
-function test_filesystem_type(tc, Ps)
+function test_filesystem_type(tc, Ps, fst_fun)
+is_capable(tc, fst_fun)
 
-s = stdlib.filesystem_type(Ps);
-tc.verifyClass(s, 'string')
-L = strlength(s);
-
-tc.assumeFalse(isempty(L) && tc.CI, "Some CI block viewing their filesystem type")
+t = fst_fun(Ps);
+tc.verifyClass(t, 'string')
 
+tc.assumeFalse(isempty(t) && tc.CI, "Some CI block viewing their filesystem type")
 
 if stdlib.exists(Ps)
-  tc.verifyGreaterThan(L, 0)
+  tc.verifyGreaterThan(strlength(t), 0)
 else
-  tc.verifyEqual(L, 0)
+  tc.verifyEmpty(t)
 end
 end