[GR-59108] Backport to 24.1: Shared engine cannot be used with GraalPyResources.

PullRequest: graalpython/3564

Author: OracleLabsAutomation

graalpython/com.oracle.graal.python.test.integration/src/org/graalvm/python/embedding/utils/test/GraalPyResourcesTests.java

@@ -0,0 +1,61 @@
+/*
+ * Copyright (c) 2024, 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
+ *
+ * Subject to the condition set forth below, permission is hereby granted to any
+ * person obtaining a copy of this software, associated documentation and/or
+ * data (collectively the "Software"), free of charge and under any and all
+ * copyright rights in the Software, and any and all patent rights owned or
+ * freely licensable by each licensor hereunder covering either (i) the
+ * unmodified Software as contributed to or provided by such licensor, or (ii)
+ * the Larger Works (as defined below), to deal in both
+ *
+ * (a) the Software, and
+ *
+ * (b) any piece of software and/or hardware listed in the lrgrwrks.txt file if
+ * one is included with the Software each a "Larger Work" to which the Software
+ * is contributed by such licensors),
+ *
+ * without restriction, including without limitation the rights to copy, create
+ * derivative works of, display, perform, and distribute the Software and make,
+ * use, sell, offer for sale, import, export, have made, and have sold the
+ * Software and the Larger Work(s), and to sublicense the foregoing rights on
+ * either these or other terms.
+ *
+ * This license is subject to the following condition:
+ *
+ * The above copyright notice and either this complete permission notice or at a
+ * minimum a reference to the UPL must be included in all copies or substantial
+ * portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+package org.graalvm.python.embedding.utils.test;
+
+import org.graalvm.polyglot.Engine;
+import org.graalvm.python.embedding.utils.GraalPyResources;
+import org.graalvm.python.embedding.utils.VirtualFileSystem;
+import org.junit.Test;
+
+import java.nio.file.Path;
+
+public class GraalPyResourcesTests {
+
+    @Test
+    public void sharedEngine() {
+        // simply check if we are able to create a context with a shared engine
+        Engine sharedEngine = Engine.newBuilder().build();
+        GraalPyResources.contextBuilder().engine(sharedEngine).build().close();
+        GraalPyResources.contextBuilder().engine(sharedEngine).build().close();
+        GraalPyResources.contextBuilder(Path.of("test")).engine(sharedEngine).build().close();
+        GraalPyResources.contextBuilder(VirtualFileSystem.newBuilder().build()).engine(sharedEngine).build().close();
+    }
+}

graalpython/org.graalvm.python.embedding/src/org/graalvm/python/embedding/utils/GraalPyResources.java

@@ -356,9 +356,6 @@ private static Context.Builder createContextBuilder() {
                         // Force to automatically import site.py module, to make Python packages
                         // available
                         option("python.ForceImportSite", "true").
-                        // Do not warn if running without JIT. This can be desirable for short
-                        // running scripts to reduce memory footprint.
-                        option("engine.WarnInterpreterOnly", "false").
                         // causes the interpreter to always assume hash-based pycs are valid
                         option("python.CheckHashPycsMode", "never");
     }