get_permissions: just file mode, add exe test

Author: scivision

+stdlib/+legacy/get_permissions.m

@@ -3,7 +3,7 @@
 p = '';
 
 if stdlib.exists(file)
-  p = stdlib.native.perm2char(file_attributes(file), file);
+  p = stdlib.native.perm2char(file_attributes(file));
 end
 
 end

+stdlib/+native/get_permissions.m

@@ -3,7 +3,7 @@
 p = '';
 
 if stdlib.exists(file)
-  p = stdlib.native.perm2char(filePermissions(file), file);
+  p = stdlib.native.perm2char(filePermissions(file));
 end
 
 end

+stdlib/+native/perm2char.m

@@ -1,9 +1,8 @@
 %% NATIVE.PERM2CHAR convert file permissions to permission string
 
-function p = perm2char(v, file)
+function p = perm2char(v)
 arguments
   v (1,1)
-  file {mustBeTextScalar}
 end
 
 
@@ -23,7 +22,7 @@
 
 if isa(v, "matlab.io.WindowsPermissions") || ispc()
 
-  if p(1) == 'r' && stdlib.native.has_windows_executable_suffix(file)
+  if p(1) == 'r'
     p(3) = 'x';
   end
 

test/TestPermissions.m

@@ -37,6 +37,20 @@ function test_get_permissions(tc, Ps, fname)
 end
 
 
+function test_get_permissions_exe(tc)
+matlab_exe = fullfile(matlabroot, "bin/matlab");
+if ispc()
+  matlab_exe = matlab_exe + ".exe";
+end
+
+tc.assertThat(matlab_exe, matlab.unittest.constraints.IsFile)
+p = stdlib.get_permissions(matlab_exe);
+tc.assertNotEmpty(p)
+tc.verifyEqual(p(3), 'x')
+
+end
+
+
 function test_set_permissions_noread(tc)
 import matlab.unittest.constraints.StartsWithSubstring