set_permissions: choose_method

Author: scivision

+stdlib/+legacy/set_permissions.m

@@ -1,5 +1,9 @@
 function ok = set_permissions(file, readable, writable, executable)
 
+ok = false;
+
+if ~stdlib.exists(file), return, end
+
 mode = '';
 % mode is space-delimited
 if ~ispc()

+stdlib/+native/set_permissions.m

@@ -1,5 +1,9 @@
 function ok = set_permissions(file, readable, writable, executable)
 
+ok = false;
+
+if ~stdlib.exists(file), return, end
+
 p = filePermissions(file);
 
 k = string.empty;

+stdlib/set_permissions.m

@@ -9,23 +9,17 @@
 %%% Outputs
 % * ok (1,1) logical
 
-function ok = set_permissions(file, readable, writable, executable)
+function ok = set_permissions(file, readable, writable, executable, method)
 arguments
   file {mustBeTextScalar}
   readable (1,1) {mustBeInteger, mustBeInRange(readable, -1, 1)}
   writable (1,1) {mustBeInteger, mustBeInRange(writable, -1, 1)}
   executable (1,1) {mustBeInteger, mustBeInRange(executable, -1, 1)}
+  method (1,:) string = ["native", "legacy"]
 end
 
-ok = false;
+fun = choose_method(method, "set_permissions", 'R2025a');
 
-if ~stdlib.exists(file), return, end
-
-
-if isMATLABReleaseOlderThan('R2025a')
-  ok = stdlib.legacy.set_permissions(file, readable, writable, executable);
-else
-  ok = stdlib.native.set_permissions(file, readable, writable, executable);
-end
+ok = fun(file, readable, writable, executable);
 
 end

test/TestPermissions.m

@@ -2,7 +2,7 @@
 
 properties (TestParameter)
 Ps = {".", pwd(), "", tempname(), mfilename('fullpath') + ".m"}
-sp_fun = {@stdlib.native.set_permissions, @stdlib.legacy.set_permissions}
+sp_fun = {'native', 'legacy'}
 end
 
 methods(TestClassSetup)
@@ -56,18 +56,23 @@ function test_set_permissions_noread(tc)
 
 function test_set_permissions_nowrite(tc, sp_fun)
 import matlab.unittest.constraints.StartsWithSubstring
-is_capable(tc, sp_fun)
 
 tc.assumeFalse(isMATLABReleaseOlderThan('R2022a'))
 td = tc.createTemporaryFolder();
 
 nw = fullfile(td, "no-write");
 
 tc.verifyTrue(stdlib.touch(nw))
-tc.verifyTrue(sp_fun(nw, 0, -1, 0))
-p = stdlib.get_permissions(nw);
+try
+  tc.verifyTrue(stdlib.set_permissions(nw, 0, -1, 0, sp_fun))
+catch e
+  tc.verifyEqual(e.identifier, 'stdlib:choose_method:NameError')
+  return
+end
 
-if ~ispc() || ~contains(func2str(sp_fun), ".legacy.")
+
+p = stdlib.get_permissions(nw);
+if ~ispc() || sp_fun ~= "legacy"
   tc.verifyThat(p, StartsWithSubstring("r-"), "no-write permission failed to set")
 end
 

test/is_capable.m

@@ -53,7 +53,7 @@ function is_capable(tc, f)
 
 elseif contains(n, ".native.")
 
-  if endsWith(n, ["is_exe", "set_permissions"])
+  if endsWith(n, "is_exe")
     tc.assumeFalse(isMATLABReleaseOlderThan('R2025a'))
   end