Fix user execute check

src/aiida/orm/nodes/data/code/installed.py

@@ -139,8 +139,10 @@ def validate_filepath_executable(self):
                     file_exists = transport.isfile(str(self.filepath_executable))
                     if file_exists:
                         mode = transport.get_mode(str(self.filepath_executable))
-                        # check if execute but is set
-                        user_has_execute = format(mode, 'b')[6] == '1'
+                        # `format(mode, 'b')` with default permissions
+                        # gives 110110100, representing rw-rw-r--
+                        # Check on index 2 if user has execute
+                        user_has_execute = format(mode, 'b')[2] == '1'
 
         except Exception as exception:
             raise exceptions.ValidationError(

tests/orm/data/code/test_installed.py

@@ -126,8 +126,7 @@ def test_validate_filepath_executable(ssh_key, computer, bash_path, tmp_path):
         ValidationError,
         match=r'The executable at the remote absolute path .* exists, but might not actually be executable\.',
     ):
-        # Didn't put this in the if-else and use transport.put, as we anyway only connect to localhost via SSH in this
-        # test
+        # Didn't put this in the if, using transport.put, as we anyway only connect to localhost via SSH in this test
         dummy_code.validate_filepath_executable()
 
     code.filepath_executable = str(bash_path.absolute())