get_owner: choose_method

Author: scivision

+stdlib/get_owner.m

@@ -5,19 +5,14 @@
 %%% Outputs
 % * n: owner, or empty if path does not exist
 
-function n = get_owner(p)
+function n = get_owner(file, method)
 arguments
-  p {mustBeTextScalar}
+  file {mustBeTextScalar}
+  method (1,:) string = ["java", "dotnet", "python" "sys"]
 end
 
-if stdlib.has_java()
-  n = stdlib.java.get_owner(p);
-elseif ispc() && stdlib.has_dotnet()
-  n = stdlib.dotnet.get_owner(p);
-elseif ~ispc() && stdlib.has_python()
-  n = stdlib.python.get_owner(p);
-else
-  n = stdlib.sys.get_owner(p);
-end
+fun = choose_method(method, "get_owner");
+
+n = fun(file);
 
 end

+stdlib/private/choose_method.m

@@ -19,7 +19,7 @@
         if stdlib.dotnet_api() < 6, continue, end
       end
 
-      if endsWith(name, "is_admin")
+      if endsWith(name, ["get_owner", "is_admin"])
         if ~ispc(), continue, end
       end
 
@@ -38,11 +38,11 @@
         if ~stdlib.python.has_psutil(); continue, end
       end
 
-      if endsWith(name, "cpu_load")
+      if endsWith(name, ["cpu_load", "get_owner"])
         if ~isunix(), continue, end
       end
 
-      if endsWith(name, "is_admin") 
+      if endsWith(name, "is_admin")
         if ~isunix() || isMATLABReleaseOlderThan('R2024a'), continue, end
       end
 

test/TestDisk.m

@@ -13,7 +13,6 @@
 disk_ac_name = {'disk_available', 'disk_capacity'}
 hl_fun = {'java', 'python'}
 fst_fun = {'sys', 'dotnet', 'java', 'python'}
-owner_fun = {@stdlib.get_owner, @stdlib.sys.get_owner, @stdlib.dotnet.get_owner, @stdlib.java.get_owner, @stdlib.python.get_owner}
 end
 
 methods(TestClassSetup)
@@ -111,17 +110,22 @@ function test_inode_device(tc, id_fun, id_name)
 end
 
 
-function test_owner(tc, Po, owner_fun)
-is_capable(tc, owner_fun)
+function test_owner(tc, Po, fst_fun)
+tc.assertNotEmpty(which("stdlib." + fst_fun + ".get_owner"))
 
-s = owner_fun(Po);
+try
+  o = stdlib.get_owner(Po, fst_fun);
+catch e
+  tc.verifyEqual(e.identifier, 'stdlib:choose_method:NameError', e.message)
+  return
+end
 
-tc.verifyClass(s, 'string')
+tc.verifyClass(o, 'string')
 
 if stdlib.exists(Po)
-  tc.verifyGreaterThan(strlength(s), 0)
+  tc.verifyGreaterThan(strlength(o), 0)
 else
-  tc.verifyEmpty(s)
+  tc.verifyEmpty(o)
 end
 
 end

test/is_capable.m

@@ -11,10 +11,6 @@ function is_capable(tc, f)
   dapi = stdlib.dotnet_api();
   tc.assumeGreaterThan(dapi, 0, ".NET not available")
 
-  if endsWith(n, "owner")
-    tc.assumeTrue(ispc(), "Windows only function")
-  end
-
 elseif contains(n, ".java.")
 
   japi = stdlib.java_api();
@@ -24,10 +20,6 @@ function is_capable(tc, f)
 
    tc.assumeTrue(stdlib.has_python(), "Python not available")
 
-   if endsWith(n, "owner")
-     tc.assumeFalse(ispc(), "unix only function")
-   end
-
 elseif contains(n, ".sys.")
 
   if endsWith(n, "samepath")