Test foreign exception to string conversion.

Author: woess

graal-js/src/com.oracle.truffle.js.test/js/interop/foreign_error_prototype.js

@@ -38,7 +38,7 @@ try {
     assertSame(message, e.message);
     assertTrue(e.stack.includes('foreign_error_prototype'));
     assertTrue(e.stack.includes('throwJavaError'));
-    assertSame('Error: ' + message, Error.prototype.toString.call(e));
+    assertSame('java.lang.RuntimeException: ' + message, Error.prototype.toString.call(e));
 
     assertSame(undefined, e.cause);
 }
@@ -55,7 +55,7 @@ try {
         assertSame(otherMessage, e.message);
         assertTrue(e.stack.includes('foreign_error_prototype'));
         assertTrue(e.stack.includes('throwJavaErrorWithCause'));
-        assertSame('Error: ' + otherMessage, Error.prototype.toString.call(e));
+        assertSame('java.lang.RuntimeException: ' + otherMessage, Error.prototype.toString.call(e));
 
         assertSame(cause, e.cause);
 
@@ -64,7 +64,7 @@ try {
         assertSame(message, e.message);
         assertTrue(e.stack.includes('foreign_error_prototype'));
         assertTrue(e.stack.includes('throwJavaError'));
-        assertSame('Error: ' + message, Error.prototype.toString.call(e));
+        assertSame('java.lang.RuntimeException: ' + message, Error.prototype.toString.call(e));
 
         assertSame(undefined, e.cause);
     }
@@ -96,7 +96,7 @@ try {
         assertSame(message, e.message);
         assertTrue(e.stack.includes('foreign_error_prototype'));
         assertTrue(e.stack.includes('throwJavaError'));
-        assertSame('Error: ' + message, Error.prototype.toString.call(e));
+        assertSame('java.lang.RuntimeException: ' + message, Error.prototype.toString.call(e));
 
         assertSame(undefined, e.cause);
     }
@@ -115,7 +115,10 @@ try {
     assertTrue(e.stack.includes('foreign_error_prototype'));
     assertTrue(e.stack.includes('throwJavaErrorFromJava'));
     assertTrue(e.stack.includes('java.util.Objects.requireNonNull'));
-    assertSame('Error: ' + message, Error.prototype.toString.call(e));
+    assertSame('java.lang.NullPointerException: ' + message, Error.prototype.toString.call(e));
 
     assertSame(undefined, e.cause);
 }
+
+assertSame('java.lang.RuntimeException', String(new java.lang.RuntimeException()));
+assertSame('java.lang.RuntimeException: ' + message, String(new java.lang.RuntimeException(message)));

graal-js/src/com.oracle.truffle.js.test/src/com/oracle/truffle/js/test/interop/JavaScriptHostInteropTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2022, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2020, 2023, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * The Universal Permissive License (UPL), Version 1.0
@@ -293,6 +293,23 @@ public void explicitMethodOverloadTest() {
         }
     }
 
+    @Test
+    public void hostExceptionToString() {
+        // No host access to Throwable.toString().
+        try (Context context = JSTest.newContextBuilder().allowHostAccess(HostAccess.EXPLICIT).build()) {
+            Value toString = context.eval(ID, "String");
+
+            String message = "someMessage";
+            var exceptionWithoutMessage = new RuntimeException();
+            var exceptionWithMessage = new RuntimeException(message);
+
+            assertFalse(context.asValue(exceptionWithMessage).hasMember("toString"));
+            assertFalse(context.asValue(exceptionWithMessage).hasMember("toString"));
+            assertEquals(exceptionWithoutMessage.getClass().getName(), toString.execute(exceptionWithoutMessage).asString());
+            assertEquals(exceptionWithMessage.getClass().getName() + ": " + message, toString.execute(exceptionWithMessage).asString());
+        }
+    }
+
     public static class ExceptionVerifier implements Consumer<PolyglotException> {
         private final String[] substrings;