Clean up

Author: m-dzianishchyts

jjava-distro/src/test/java/org/dflib/jjava/distro/ContainerizedKernelCase.java

@@ -39,10 +39,8 @@ public abstract class ContainerizedKernelCase {
 
     @BeforeAll
     static void setUp() throws IOException, InterruptedException {
-        System.out.println("ContainerizedKernelCase.setUp");
         initializeContainer();
         Assumptions.assumeTrue(container != null, "Docker tests are disabled. Enable with -Pdocker");
-        System.out.println("container = " + container);
 
         String source = "$(find " + CONTAINER_RESOURCES + "/src -name '*.java')";
         Container.ExecResult compileResult = executeInContainer("javac -d " + TEST_CLASSPATH + " " + source);

jjava-distro/src/test/java/org/dflib/jjava/distro/KernelEnvIT.java

@@ -10,10 +10,10 @@
 import static org.hamcrest.CoreMatchers.not;
 import static org.hamcrest.MatcherAssert.assertThat;
 
-public class KernelEnvIT extends ContainerizedKernelCase {
+class KernelEnvIT extends ContainerizedKernelCase {
 
     @Test
-    public void compilerOpts() throws Exception {
+    void compilerOpts() throws Exception {
         Map<String, String> env = Map.of(Env.JJAVA_COMPILER_OPTS, "-source 9");
         String snippet = "var value = 1;";
         Container.ExecResult snippetResult = executeInKernel(snippet, env);
@@ -27,7 +27,7 @@ public void compilerOpts() throws Exception {
     }
 
     @Test
-    public void timeout() throws Exception {
+    void timeout() throws Exception {
         Map<String, String> env = Map.of(Env.JJAVA_TIMEOUT, "3000");
         String snippet = "Thread.sleep(5000);";
         Container.ExecResult snippetResult = executeInKernel(snippet, env);
@@ -39,7 +39,7 @@ public void timeout() throws Exception {
     }
 
     @Test
-    public void classpath() throws Exception {
+    void classpath() throws Exception {
         Map<String, String> env = Map.of(Env.JJAVA_CLASSPATH, TEST_CLASSPATH);
         String snippet = String.join("; ",
                 "import org.dflib.jjava.Dummy",
@@ -52,7 +52,7 @@ public void classpath() throws Exception {
     }
 
     @Test
-    public void startUpScriptsPath() throws Exception {
+    void startUpScriptsPath() throws Exception {
         Map<String, String> env = Map.of(Env.JJAVA_STARTUP_SCRIPTS_PATH,  CONTAINER_RESOURCES + "/test-ping.jshell");
         String snippet = "ping()";
         Container.ExecResult snippetResult = executeInKernel(snippet, env);
@@ -62,7 +62,7 @@ public void startUpScriptsPath() throws Exception {
     }
 
     @Test
-    public void startUpScript() throws Exception {
+    void startUpScript() throws Exception {
         Map<String, String> env = Map.of(Env.JJAVA_STARTUP_SCRIPT, "public String ping() { return \"pong!\"; }");
         String snippet = "ping()";
         Container.ExecResult snippetResult = executeInKernel(snippet, env);
@@ -72,7 +72,7 @@ public void startUpScript() throws Exception {
     }
 
     @Test
-    public void loadExtensions_Default() throws Exception {
+    void loadExtensions_Default() throws Exception {
         String snippet = "printf(\"Hello, %s!\", \"world\");";
         Container.ExecResult snippetResult = executeInKernel(snippet);
 
@@ -81,7 +81,7 @@ public void loadExtensions_Default() throws Exception {
     }
 
     @Test
-    public void loadExtensions_Disable() throws Exception {
+    void loadExtensions_Disable() throws Exception {
         Map<String, String> env = Map.of(Env.JJAVA_LOAD_EXTENSIONS, "0");
         String snippet = "printf(\"Hello, %s!\", \"world\");";
         Container.ExecResult snippetResult = executeInKernel(snippet, env);
@@ -93,7 +93,7 @@ public void loadExtensions_Disable() throws Exception {
     }
 
     @Test
-    public void jvmOpts() throws Exception {
+    void jvmOpts() throws Exception {
         Map<String, String> env = Map.of(Env.JJAVA_JVM_OPTS, "-Xmx300m");
         String snippet = "Runtime.getRuntime().maxMemory()";
         Container.ExecResult snippetResult = executeInKernel(snippet, env);

jjava-distro/src/test/java/org/dflib/jjava/distro/KernelMagicIT.java

@@ -8,11 +8,11 @@
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-public class KernelMagicIT extends ContainerizedKernelCase {
+class KernelMagicIT extends ContainerizedKernelCase {
 
     @Deprecated
     @Test
-    public void jars() throws Exception {
+    void jars() throws Exception {
         String jar = CONTAINER_RESOURCES + "/jakarta.annotation-api-3.0.0.jar";
         Container.ExecResult fetchResult = container.execInContainer(
                 "curl", "-L", "-s", "-S", "-f",
@@ -33,7 +33,7 @@ public void jars() throws Exception {
     }
 
     @Test
-    public void classpath() throws Exception {
+    void classpath() throws Exception {
         String snippet = String.join("\n",
                 "%classpath " + TEST_CLASSPATH,
                 "import org.dflib.jjava.Dummy;",
@@ -46,7 +46,7 @@ public void classpath() throws Exception {
     }
 
     @Test
-    public void maven() throws Exception {
+    void maven() throws Exception {
         String snippet = String.join("\n",
                 "%maven org.dflib:dflib-jupyter:1.0.0-RC1",
                 "System.getProperty(\"java.class.path\")"
@@ -59,7 +59,7 @@ public void maven() throws Exception {
 
     @Deprecated
     @Test
-    public void mavenIvySyntax() throws Exception {
+    void mavenIvySyntax() throws Exception {
         String snippet = String.join("\n",
                 "%maven jakarta.annotation#jakarta.annotation-api;3.0.0",
                 "System.getProperty(\"java.class.path\")"
@@ -72,7 +72,7 @@ public void mavenIvySyntax() throws Exception {
     }
 
     @Test
-    public void load() throws Exception {
+    void load() throws Exception {
         String script = CONTAINER_RESOURCES + "/test-ping.jshell";
         String snippet = String.join("\n",
                 "%load " + script,
@@ -85,7 +85,7 @@ public void load() throws Exception {
     }
 
     @Test
-    public void loadFromPOM() throws Exception {
+    void loadFromPOM() throws Exception {
         String pom = CONTAINER_RESOURCES + "/test-pom.xml";
         String snippet = String.join("\n",
                 "%loadFromPOM " + pom,

jjava-distro/src/test/java/org/dflib/jjava/distro/KernelStartupIT.java

@@ -7,10 +7,10 @@
 import static org.hamcrest.CoreMatchers.not;
 import static org.hamcrest.MatcherAssert.assertThat;
 
-public class KernelStartupIT extends ContainerizedKernelCase {
+class KernelStartupIT extends ContainerizedKernelCase {
 
     @Test
-    public void startUp() throws Exception {
+    void startUp() throws Exception {
         String snippet = "1000d + 1";
         Container.ExecResult snippetResult = executeInKernel(snippet);