[GR-57413] Cleanup of exposed embedding APIs.

PullRequest: graalpython/3552

Author: tomasstupka

graalpython/com.oracle.graal.python.test.integration/src/org/graalvm/python/embedding/utils/test/VirtualFileSystemTest.java renamed to graalpython/com.oracle.graal.python.test.integration/src/org/graalvm/python/embedding/utils/test/integration/VirtualFileSystemIntegrationTest.java

@@ -141,11 +141,17 @@
  * </ul>
  * </p>
  *
+ * For <b>more examples</b> on how to use this class refer to
+ * <a href="https://github.com/graalvm/graal-languages-demos/tree/main/graalpy">GraalPy Demos and
+ * Guides</a>.
+ *
  * @see VirtualFileSystem
  * @see VirtualFileSystem.Builder
  */
-// TODO: link to user guide
-public class GraalPyResources {
+public final class GraalPyResources {
+
+    private GraalPyResources() {
+    }
 
     /**
      * Creates a GraalPy context preconfigured with a {@link VirtualFileSystem} and other GraalPy
@@ -194,9 +200,10 @@ public static Context createContext() {
      *     }
      * }
      * </pre>
-     *
+     * 
+     * @see <a href=
+     *      "https://github.com/oracle/graalpython/blob/master/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/PythonOptions.java">PythonOptions</a>
      */
-    // TODO add link to python options doc
     public static Context.Builder contextBuilder() {
         VirtualFileSystem vfs = VirtualFileSystem.create();
         return contextBuilder(vfs);
@@ -252,17 +259,17 @@ public static Context.Builder contextBuilder() {
     public static Context.Builder contextBuilder(VirtualFileSystem vfs) {
         return createContextBuilder().
         // allow access to the virtual and the host filesystem, as well as sockets
-                        allowIO(IOAccess.newBuilder().allowHostSocketAccess(true).fileSystem(vfs).build()).
+                        allowIO(IOAccess.newBuilder().allowHostSocketAccess(true).fileSystem(vfs.impl).build()).
                         // The sys.executable path, a virtual path that is used by the interpreter
                         // to discover packages
-                        option("python.Executable", vfs.vfsVenvPath() + (VirtualFileSystem.isWindows() ? "\\Scripts\\python.exe" : "/bin/python")).
+                        option("python.Executable", vfs.impl.vfsVenvPath() + (VirtualFileSystemImpl.isWindows() ? "\\Scripts\\python.exe" : "/bin/python")).
                         // Set the python home to be read from the embedded resources
-                        option("python.PythonHome", vfs.vfsHomePath()).
+                        option("python.PythonHome", vfs.impl.vfsHomePath()).
                         // Set python path to point to sources stored in
                         // src/main/resources/org.graalvm.python.vfs/src
-                        option("python.PythonPath", vfs.vfsSrcPath()).
+                        option("python.PythonPath", vfs.impl.vfsSrcPath()).
                         // pass the path to be executed
-                        option("python.InputFilePath", vfs.vfsSrcPath());
+                        option("python.InputFilePath", vfs.impl.vfsSrcPath());
     }
 
     /**
@@ -308,14 +315,14 @@ public static Context.Builder contextBuilder(VirtualFileSystem vfs) {
      */
     public static Context.Builder contextBuilder(Path resourcesDirectory) {
         String execPath;
-        if (VirtualFileSystem.isWindows()) {
-            execPath = resourcesDirectory.resolve(VirtualFileSystem.VFS_VENV).resolve("Scripts").resolve("python.exe").toAbsolutePath().toString();
+        if (VirtualFileSystemImpl.isWindows()) {
+            execPath = resourcesDirectory.resolve(VirtualFileSystemImpl.VFS_VENV).resolve("Scripts").resolve("python.exe").toAbsolutePath().toString();
         } else {
-            execPath = resourcesDirectory.resolve(VirtualFileSystem.VFS_VENV).resolve("bin").resolve("python").toAbsolutePath().toString();
+            execPath = resourcesDirectory.resolve(VirtualFileSystemImpl.VFS_VENV).resolve("bin").resolve("python").toAbsolutePath().toString();
         }
 
-        String homePath = resourcesDirectory.resolve(VirtualFileSystem.VFS_HOME).toAbsolutePath().toString();
-        String srcPath = resourcesDirectory.resolve(VirtualFileSystem.VFS_SRC).toAbsolutePath().toString();
+        String homePath = resourcesDirectory.resolve(VirtualFileSystemImpl.VFS_HOME).toAbsolutePath().toString();
+        String srcPath = resourcesDirectory.resolve(VirtualFileSystemImpl.VFS_SRC).toAbsolutePath().toString();
         return createContextBuilder().
         // allow all IO access
                         allowIO(IOAccess.ALL).
@@ -362,7 +369,7 @@ private static Context.Builder createContextBuilder() {
      * <p>
      * <b>Example </b> creating a GraalPy context precofigured with an external resource directory
      * located next to a native image executable.
-     * 
+     *
      * <pre>
      * Path resourcesDir = GraalPyResources.getNativeExecutablePath().getParent().resolve("python-resources");
      * try (Context context = GraalPyResources.contextBuilder(resourcesDir).build()) {
@@ -405,7 +412,7 @@ public static Path getNativeExecutablePath() {
      * </p>
      * <p>
      * <b>Example</b>
-     * 
+     *
      * <pre>
      * Path resourcesDir = Path.of(System.getProperty("user.home"), ".cache", "my.java.python.app.resources");
      * FileSystem fs = GraalPyResources.createVirtualFileSystem();
@@ -423,6 +430,6 @@ public static void extractVirtualFileSystemResources(VirtualFileSystem vfs, Path
         if (Files.exists(resourcesDirectory) && !Files.isDirectory(resourcesDirectory)) {
             throw new IOException(String.format("%s has to be a directory", resourcesDirectory.toString()));
         }
-        vfs.extractResources(resourcesDirectory);
+        vfs.impl.extractResources(resourcesDirectory);
     }
 }