test, backend: more efficient, robust

Author: scivision

+stdlib/+dotnet/create_symlink.m

@@ -9,11 +9,9 @@
 end
 
 % https://learn.microsoft.com/en-us/dotnet/api/system.io.file.createsymboliclink
-try
+try %#ok<TRYNC>
   System.IO.File.CreateSymbolicLink(link, target);
   ok = true;
-catch e
-  warning(e.identifier, "%s", e.message)
 end
 
 end

+stdlib/+dotnet/get_owner.m

@@ -5,9 +5,11 @@
 % This is not yet possible with .NET on Unix, even with .NET 10.
 % It would require Pinvoke or external Mono.Unix
 
+o = "";
+
 ntAccountType = System.Type.GetType('System.Security.Principal.NTAccount');
 if isempty(ntAccountType)
-  error('NTAccount type not found. Ensure you are running on a Windows system with .NET support.');
+  return
 end
 
 if isfolder(p)

+stdlib/+dotnet/is_admin.m

@@ -2,13 +2,13 @@
 
 function y = is_admin()
 
-% com.sun.security.auth.module.NTSystem().getGroupIDs();
-% Administrator group SID (S-1-5-32-544) is not an appropriate check .getGroupIDs because it
-% only tells if a user is *allowed* to run as admin, not if they are currently running as admin.
-
 % Use .NET System.Security.Principal to check for admin privileges
-identity = System.Security.Principal.WindowsIdentity.GetCurrent();
-principal = System.Security.Principal.WindowsPrincipal(identity);
-y = principal.IsInRole(System.Security.Principal.WindowsBuiltInRole.Administrator);
+try
+  identity = System.Security.Principal.WindowsIdentity.GetCurrent();
+  principal = System.Security.Principal.WindowsPrincipal(identity);
+  y = principal.IsInRole(System.Security.Principal.WindowsBuiltInRole.Administrator);
+catch
+  y = logical.empty;
+end
 
 end

+stdlib/+dotnet/ram_total.m

@@ -3,5 +3,11 @@
 function bytes = ram_total()
   % .NET is 2-3x faster than Java for this
   % https://learn.microsoft.com/en-us/dotnet/api/system.gcmemoryinfo.totalavailablememorybytes
-  bytes = uint64(System.GC.GetGCMemoryInfo().TotalAvailableMemoryBytes);
+  try
+    bytes = System.GC.GetGCMemoryInfo().TotalAvailableMemoryBytes;
+  catch
+    bytes = [];
+  end
+
+  bytes = uint64(bytes);
 end

+stdlib/+dotnet/read_symlink.m

@@ -1,17 +1,16 @@
 %% DOTNET.READ_SYMLINK resolve the symbolic links of a filepath
 
-function r = read_symlink(p)
+function r = read_symlink(file)
 
 try
-  h = System.IO.FileInfo(p);
+  h = System.IO.FileInfo(file);
   r = string(h.LinkTarget);
   % on Unix, this can be empty if the file is not a symlink
+  if isempty(r)
+    r = "";
+  end
 catch
   r = "";
 end
 
-if isempty(r)
-  r = "";
-end
-
 end

+stdlib/+java/device.m

@@ -6,7 +6,11 @@
 % Java 1.8 benefits from the absolute() for stability
 % seen on older Matlab versions on HPC
 
-i = java.nio.file.Files.getAttribute(jp, 'unix:dev', opt);
+try
+  i = java.nio.file.Files.getAttribute(jp, 'unix:dev', opt);
+catch
+  i = [];
+end
 
 i = uint64(i);
 end

+stdlib/+java/hard_link_count.m

@@ -1,6 +1,9 @@
 function c = hard_link_count(p)
 
 opt = java.nio.file.LinkOption.values();
-c = java.nio.file.Files.getAttribute(javaPathObject(p), "unix:nlink", opt);
-
-end
+try
+  c = java.nio.file.Files.getAttribute(javaPathObject(p), "unix:nlink", opt);
+catch
+  c = [];
+end
+end

+stdlib/+java/inode.m

@@ -6,7 +6,11 @@
 % Java 1.8 benefits from the absolute() for stability--it's not an issue
 % on every computer.
 
-i = java.nio.file.Files.getAttribute(jp, "unix:ino", opt);
+try
+  i = java.nio.file.Files.getAttribute(jp, "unix:ino", opt);
+catch
+  i = [];
+end
 
 i = uint64(i);
 end

+stdlib/+java/is_admin.m

@@ -1,5 +1,13 @@
-function y = is_admin()
+function ok = is_admin()
 
-y = com.sun.security.auth.module.UnixSystem().getUid() == 0;
+% com.sun.security.auth.module.NTSystem().getGroupIDs();
+% Administrator group SID (S-1-5-32-544) is not an appropriate check .getGroupIDs because it
+% only tells if a user is *allowed* to run as admin, not if they are currently running as admin.
+
+ok = logical.empty;
+
+try %#ok<TRYNC>
+  ok = com.sun.security.auth.module.UnixSystem().getUid() == 0;
+end
 
 end

+stdlib/+native/is_exe.m

@@ -16,8 +16,11 @@
   props = "Readable";
 end
 
-t = getPermissions(filePermissions(file(y)), props);
-
-y(y) = any(t{:, :}, 2);
+try
+  t = getPermissions(filePermissions(file(y)), props);
+  y(y) = any(t{:, :}, 2);
+catch
+  y = logical.empty;
+end
 
 end