Fix: 'os.open' should accept 'bytes' object.

Author: fangerer

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

@@ -35,6 +35,7 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
+import java.io.UnsupportedEncodingException;
 import java.lang.ProcessBuilder.Redirect;
 import java.math.BigInteger;
 import java.net.InetAddress;
@@ -673,6 +674,8 @@ int dupOvf(PInt fd) {
     @GenerateNodeFactory
     @TypeSystemReference(PythonArithmeticTypes.class)
     public abstract static class OpenNode extends PythonFileNode {
+        @Child private SequenceStorageNodes.ToByteArrayNode toByteArrayNode;
+
         private final BranchProfile gotException = BranchProfile.create();
 
         @Specialization(guards = {"isNoValue(mode)", "isNoValue(dir_fd)"})
@@ -694,6 +697,28 @@ Object open(String pathname, int flags, int fileMode, @SuppressWarnings("unused"
             }
         }
 
+        @Specialization(guards = {"isNoValue(dir_fd)"})
+        Object open(PBytes pathname, int flags, int fileMode, PNone dir_fd) {
+            return open(decode(getByteArray(pathname)), flags, fileMode, dir_fd);
+        }
+
+        private byte[] getByteArray(PIBytesLike pByteArray) {
+            if (toByteArrayNode == null) {
+                CompilerDirectives.transferToInterpreterAndInvalidate();
+                toByteArrayNode = insert(ToByteArrayNode.create());
+            }
+            return toByteArrayNode.execute(pByteArray.getSequenceStorage());
+        }
+
+        @TruffleBoundary
+        private String decode(byte[] raw) {
+            try {
+                return new String(raw, "ascii");
+            } catch (UnsupportedEncodingException e) {
+                throw raise(PythonBuiltinClassType.UnicodeDecodeError, e.getMessage());
+            }
+        }
+
         @SuppressWarnings({"unchecked", "rawtypes"})
         @TruffleBoundary(allowInlining = true)
         private static FileAttribute<Set<PosixFilePermission>>[] modeToAttributes(int fileMode) {