Updated unit test

Author: Franziska Geiger

graalpython/com.oracle.graal.python.test/src/tests/test_posix.py

@@ -41,6 +41,7 @@
 
 
 class PosixTests(unittest.TestCase):
+
     def test_uname(self):
         # just like cpython, a simple smoke test
         import posix
@@ -52,12 +53,23 @@ def test_uname(self):
         self.assertIsNotNone(uname.version)
         self.assertIsNotNone(uname.machine)
 
-    def test_execv(self, tmpdir):
-        print tmpdir
-
-        sub = tmpdir.mkdir("sub")
-        sub.join("testfile.sh").write("echo \"something echo\" > {}/sub/test.txt".format(tmpdir))
+    def test_execv(self):
         import os
-        toExecute = tmpdir + '/testfile.sh'
-        os.execv(toExecute, [toExecute])
-        assert os.path.isfile(tmpdir + '/sub/test.txt')
+        import stat
+        cwd = os.getcwd()
+        new_file_path = os.path.join(cwd , 'myscript.sh')
+
+        with open(new_file_path, 'w') as script:
+            script.write('#!/bin/sh\n')
+            script.write("echo \"something echo\" > {}/test.txt".format(cwd))
+        assert os.path.isfile(cwd + '/myscript.sh')
+
+        st = os.stat(new_file_path)
+        os.chmod(new_file_path, st.st_mode | stat.S_IEXEC)
+
+        os.execv(new_file_path, [new_file_path, 'Something'])
+        assert os.path.isfile(cwd + '/test.txt')
+
+        os.remove(new_file_path)
+        os.remove(cwd + '/test.txt')
+

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/PosixModuleBuiltins.java

@@ -293,14 +293,19 @@ Object execute(String path, PList args) {
             return doExecute(path, args);
         }
 
+        @Specialization
+        Object execute(PString path, PList args) {
+            return doExecute(path.getValue(), args);
+        }
+
         @TruffleBoundary
         Object doExecute(String path, PList args) {
             try {
                 int size = args.getSequenceStorage().length();
-                String[] cmd = new String[1 + size];
+                String[] cmd = new String[size];
                 cmd[0] = path;
                 for (int i = 1; i < size; i++) {
-                    cmd[i + 3] = args.getSequenceStorage().getItemNormalized(i).toString();
+                    cmd[i] = args.getSequenceStorage().getItemNormalized(i).toString();
                 }
                 new ProcessBuilder(cmd).start();
             } catch (IOException e) {