consistent output type

Author: scivision

+stdlib/+python/is_mount.m

@@ -4,21 +4,24 @@
 
 function y = is_mount(filepath)
 
-y = logical.empty;
 
-
-try %#ok<TRYNC>
+try
+  y = false;
 
   p = py.pathlib.Path(filepath);
-  if ~p.exists(), return, end
+  if ~p.exists()
+    return
+  end
 
   % some Python on CI needs this. Didn't replicate on local Windows PC.
   if ispc() && strcmp(filepath, string(p.drive)) && ~endsWith(filepath, ["/", filesep])
-    y = false;
     return
   end
 
   y = py.os.path.ismount(p);
+catch e
+  pythonException(e)
+  y = logical.empty;
 end
 
 end

+stdlib/+sys/get_owner.m

@@ -13,6 +13,7 @@
   cmd = sprintf('stat -c %%U "%s"', file);
 end
 
+% Windows needs exists() rather than just ~strempty()
 if stdlib.exists(file)
   [s, m] = system(cmd);
   if s == 0

+stdlib/+sys/is_mount.m

@@ -4,8 +4,10 @@
 end
 
 cmd = '';
-y = logical.empty;
-if ~stdlib.exists(filepath), return, end
+if ~stdlib.exists(filepath)
+  y = false;
+  return
+end
 
 if ispc()
   if ismember(filepath, ["/", "\"]) || ...

+stdlib/get_owner.m

@@ -13,7 +13,7 @@
   backend (1,:) string = ["java", "dotnet", "python", "sys"]
 end
 
-r = string.empty;
+r = '';
 
 for b = backend
   switch b

test/TestDisk.m

@@ -67,18 +67,23 @@ function test_is_removable(tc, B_ps)
 end
 
 function test_is_mount(tc, B_ps)
-y = stdlib.is_mount(pwd(), B_ps);
-
+[y,b] = stdlib.is_mount(pwd(), B_ps);
+tc.assertEqual(char(b), B_ps)
 tc.verifyClass(y, 'logical')
-tc.verifyTrue(stdlib.is_mount("/", B_ps))
-tc.verifyEmpty(stdlib.is_mount(tempname(), B_ps))
 
-if ispc()
-  sd = getenv("SystemDrive");
-  tc.assertTrue(sd == stdlib.root_name(sd), sd)
-  tc.verifyFalse(stdlib.is_mount(sd, B_ps), sd)
-  tc.verifyTrue(stdlib.is_mount(sd + "/", B_ps), sd)
-  tc.verifyTrue(stdlib.is_mount(sd + "\", B_ps), sd)
+if ismember(B_ps, stdlib.Backend().select('is_mount'))
+  tc.verifyTrue(stdlib.is_mount("/", B_ps))
+  tc.verifyFalse(stdlib.is_mount(tempname(), B_ps))
+
+  if ispc()
+    sd = getenv("SystemDrive");
+    tc.assertTrue(sd == stdlib.root_name(sd), sd)
+    tc.verifyFalse(stdlib.is_mount(sd, B_ps), sd)
+    tc.verifyTrue(stdlib.is_mount(sd + "/", B_ps), sd)
+    tc.verifyTrue(stdlib.is_mount(sd + "\", B_ps), sd)
+  end
+else
+  tc.verifyEmpty(y)
 end
 end
 
@@ -105,12 +110,16 @@ function test_filesystem_type(tc, Ps, B_jdps)
 tc.assertEqual(char(b), B_jdps)
 tc.verifyClass(t, 'char')
 
-if ~stdlib.exists(Ps)
-  tc.verifyEmpty(t)
+if ismember(B_jdps, stdlib.Backend().select('filesystem_type'))
+  if ~stdlib.exists(Ps)
+    tc.verifyEmpty(t)
+  else
+    tc.assumeFalse(isempty(t) && tc.CI, "Some CI block viewing their filesystem type")
+    tc.assertNotEmpty(t)
+    tc.verifyGreaterThan(strlength(t), 0)
+  end
 else
-  tc.assumeFalse(isempty(t) && tc.CI, "Some CI block viewing their filesystem type")
-  tc.assertNotEmpty(t)
-  tc.verifyGreaterThan(strlength(t), 0)
+  tc.verifyEmpty(t)
 end
 end