[java] Removing ParallelSessionsTest from remote tests

diemol · diemol · commit 7a7a6fedb488 · 2022-01-24T08:43:52.000-05:00
This scenario is already covered by StressTest.java
diff --git a/java/test/org/openqa/selenium/remote/BUILD.bazel b/java/test/org/openqa/selenium/remote/BUILD.bazel
@@ -2,7 +2,6 @@ load("@rules_jvm_external//:defs.bzl", "artifact")
 load("//java:defs.bzl", "java_selenium_test_suite", "java_test_suite")
 
 REMOTE_TESTS = [
-    "ParallelSessionsTest.java",
     "RemoteWebDriverScreenshotTest.java",
 ]
 
diff --git a/java/test/org/openqa/selenium/remote/ParallelSessionsTest.java b/java/test/org/openqa/selenium/remote/ParallelSessionsTest.java
@@ -17,6 +17,9 @@
 
 package org.openqa.selenium.remote;
 
+import static java.util.concurrent.TimeUnit.SECONDS;
+import static org.openqa.selenium.testing.Safely.safelyCall;
+
 import org.junit.Test;
 import org.openqa.selenium.WebDriver;
 import org.openqa.selenium.testing.JUnit4TestBase;
@@ -28,11 +31,9 @@
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
 
-import static java.util.concurrent.TimeUnit.SECONDS;
-import static org.openqa.selenium.testing.Safely.safelyCall;
-
 public class ParallelSessionsTest extends JUnit4TestBase {
-  private ExecutorService service = Executors.newFixedThreadPool(3);
+
+  private final ExecutorService service = Executors.newFixedThreadPool(3);
 
   @Test
   public void shouldBeAbleToRunMultipleBrowsersAtTheSameTime() throws Exception {

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,6 @@ load("@rules_jvm_external//:defs.bzl", "artifact")`
`2`	`2`	`load("//java:defs.bzl", "java_selenium_test_suite", "java_test_suite")`
`3`	`3`
`4`	`4`	`REMOTE_TESTS = [`
`5`		`- "ParallelSessionsTest.java",`
`6`	`5`	`"RemoteWebDriverScreenshotTest.java",`
`7`	`6`	`]`
`8`	`7`