fix and test multiplying a string by a foreign boolean

Author: timfel

graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/interop/JavaInteropTest.java

@@ -52,6 +52,15 @@
 import java.util.Arrays;
 import java.util.List;
 
+import com.oracle.graal.python.runtime.interop.InteropArray;
+import com.oracle.graal.python.test.PythonTests;
+import com.oracle.truffle.api.interop.ArityException;
+import com.oracle.truffle.api.interop.InteropLibrary;
+import com.oracle.truffle.api.interop.TruffleObject;
+import com.oracle.truffle.api.interop.UnknownIdentifierException;
+import com.oracle.truffle.api.library.ExportLibrary;
+import com.oracle.truffle.api.library.ExportMessage;
+
 import org.graalvm.polyglot.Context;
 import org.graalvm.polyglot.Context.Builder;
 import org.graalvm.polyglot.Engine;
@@ -67,15 +76,6 @@
 import org.junit.runners.Parameterized.Parameter;
 import org.junit.runners.Parameterized.Parameters;
 
-import com.oracle.graal.python.runtime.interop.InteropArray;
-import com.oracle.graal.python.test.PythonTests;
-import com.oracle.truffle.api.interop.ArityException;
-import com.oracle.truffle.api.interop.InteropLibrary;
-import com.oracle.truffle.api.interop.TruffleObject;
-import com.oracle.truffle.api.interop.UnknownIdentifierException;
-import com.oracle.truffle.api.library.ExportLibrary;
-import com.oracle.truffle.api.library.ExportMessage;
-
 @RunWith(Enclosed.class)
 public class JavaInteropTest {
     public static class GeneralInterop extends PythonTests {
@@ -509,6 +509,54 @@ public void writableBindings() {
             assertTrue(javaObj.isNumber());
             assertEquals(javaObj.asInt(), 42);
         }
+
+        @ExportLibrary(InteropLibrary.class)
+        static final class WrapString implements TruffleObject {
+            private final String str;
+
+            WrapString(String str) {
+                this.str = str;
+            }
+
+            @ExportMessage
+            @SuppressWarnings("static-method")
+            boolean isString() {
+                return true;
+            }
+
+            @ExportMessage
+            String asString() {
+                return str;
+            }
+        }
+
+        @ExportLibrary(InteropLibrary.class)
+        static final class WrapBoolean implements TruffleObject {
+            private final boolean flag;
+
+            WrapBoolean(boolean flag) {
+                this.flag = flag;
+            }
+
+            @ExportMessage
+            @SuppressWarnings("static-method")
+            boolean isBoolean() {
+                return true;
+            }
+
+            @ExportMessage
+            boolean asBoolean() {
+                return flag;
+            }
+        }
+
+        @Test
+        public void multiplyStrBool() {
+            context.getBindings("python").putMember("javaBool", new WrapBoolean(true));
+            context.getBindings("python").putMember("javaStr", new WrapString("test"));
+            assertEquals(context.eval("python", "javaStr * javaBool").asString(), "test");
+            assertEquals(context.eval("python", "javaBool * javaStr").asString(), "test");
+        }
     }
 
     @RunWith(Parameterized.class)

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/object/DefaultPythonObjectExports.java

@@ -85,6 +85,13 @@ static Object asIndex(Object receiver,
                 CompilerDirectives.transferToInterpreter();
                 throw new IllegalStateException(e);
             }
+        } else if (interopLib.isBoolean(receiver)) {
+            try {
+                return interopLib.asBoolean(receiver) ? 1 : 0;
+            } catch (UnsupportedMessageException e) {
+                CompilerDirectives.transferToInterpreter();
+                throw new IllegalStateException(e);
+            }
         } else {
             throw raise.raiseIntegerInterpretationError(receiver);
         }