[ci] Stop skipping some tests that are green and fast now

Author: shs96c

.skipped-tests

@@ -10,22 +10,11 @@
 -//dotnet/test/support/UI:SelectTests
 -//java/test/org/openqa/selenium/bidi/browsingcontext:BrowsingContextTest
 -//java/test/org/openqa/selenium/bidi/browsingcontext:BrowsingContextTest-remote
--//java/test/org/openqa/selenium/bidi/log:LogInspectorTest-remote
--//java/test/org/openqa/selenium/bidi:BiDiSessionTest-remote
--//java/test/org/openqa/selenium/bidi:BiDiTest-remote
 -//java/test/org/openqa/selenium/chrome:ChromeDriverFunctionalTest
 -//java/test/org/openqa/selenium/chrome:ChromeDriverFunctionalTest-remote
--//java/test/org/openqa/selenium/devtools:CdpFacadeTest-remote
--//java/test/org/openqa/selenium/devtools:ChangeUserAgentTest-remote
--//java/test/org/openqa/selenium/devtools:DevToolsReuseTest-remote
--//java/test/org/openqa/selenium/devtools:JavascriptExceptionsTest-remote
--//java/test/org/openqa/selenium/devtools:NetworkInterceptorRestTest-chrome-remote
--//java/test/org/openqa/selenium/devtools:NetworkInterceptorTest-chrome-remote
 -//java/test/org/openqa/selenium/federatedcredentialmanagement:FederatedCredentialManagementTest
 -//java/test/org/openqa/selenium/firefox:FirefoxDriverBuilderTest
--//java/test/org/openqa/selenium/firefox:RemoteFirefoxDriverTest-remote
 -//java/test/org/openqa/selenium/grid/gridui:OverallGridTest
--//java/test/org/openqa/selenium/grid/router:RemoteWebDriverBiDiTest
 -//java/test/org/openqa/selenium/grid/router:RemoteWebDriverDownloadTest
 -//java/test/org/openqa/selenium/grid/router:RemoteWebDriverDownloadTest-chrome
 -//java/test/org/openqa/selenium/grid/router:RemoteWebDriverDownloadTest-chrome-remote
@@ -34,7 +23,6 @@
 -//java/test/org/openqa/selenium/interactions:DefaultMouseTest-remote
 -//java/test/org/openqa/selenium/remote:RemoteWebDriverBuilderTest
 -//java/test/org/openqa/selenium/remote:RemoteWebDriverScreenshotTest-remote
--//java/test/org/openqa/selenium:FormHandlingTest-chrome
 -//javascript/atoms:test-chrome
 -//javascript/atoms:test-firefox-beta
 -//javascript/atoms:test-firefox-dev

java/test/org/openqa/selenium/grid/gridui/OverallGridTest.java

@@ -49,31 +49,20 @@
 import org.openqa.selenium.remote.http.HttpRequest;
 import org.openqa.selenium.remote.http.HttpResponse;
 import org.openqa.selenium.support.ui.FluentWait;
-import org.openqa.selenium.support.ui.Wait;
-import org.openqa.selenium.support.ui.WebDriverWait;
+import org.openqa.selenium.testing.JupiterTestBase;
 import org.openqa.selenium.testing.drivers.Browser;
-import org.openqa.selenium.testing.drivers.WebDriverBuilder;
 
-class OverallGridTest {
+class OverallGridTest extends JupiterTestBase {
 
   private Server<?> server;
-  private WebDriver driver;
-  private WebDriver remoteWebDriver;
-  private Wait<WebDriver> wait;
 
   @BeforeEach
   public void setup() {
     server = createStandalone();
-
-    driver = new WebDriverBuilder().get();
-
-    wait = new WebDriverWait(driver, Duration.ofSeconds(10));
   }
 
   @AfterEach
   public void tearDown() {
-    safelyCall(() -> driver.quit());
-    safelyCall(() -> remoteWebDriver.quit());
     safelyCall(() -> server.stop());
   }
 
@@ -101,10 +90,15 @@ void shouldShowOneNodeRegistered() {
 
   @Test
   void shouldIncrementSessionCountWhenSessionStarts() {
-    remoteWebDriver = new RemoteWebDriver(server.getUrl(), Browser.detect().getCapabilities());
-    driver.get(whereIs(server, "/ui#/sessions"));
-
-    wait.until(textToBe(By.cssSelector("div[data-testid='session-count']"), "1"));
+    WebDriver remoteWebDriver =
+        new RemoteWebDriver(server.getUrl(), Browser.detect().getCapabilities());
+    try {
+      driver.get(whereIs(server, "/ui#/sessions"));
+
+      wait.until(textToBe(By.cssSelector("div[data-testid='session-count']"), "1"));
+    } finally {
+      remoteWebDriver.quit();
+    }
   }
 
   private Server<?> createStandalone() {

scripts/github-actions/ci-build.sh

@@ -18,7 +18,7 @@ bazel test --config=remote-ci --build_tests_only \
   --keep_going --flaky_test_attempts=2 \
   //dotnet/...  \
   //java/... \
-  //javascript/atoms/... //javascript/selenium-atoms/... //javascript/webdriver/... \
+  //javascript/atoms/... //javascript/webdriver/... \
   //py/... -- $(cat .skipped-tests | tr '\n' ' ')
 # Build the packages we want to ship to users
 bazel build --config=remote-ci //dotnet:all java/src/... //javascript/node/selenium-webdriver:selenium-webdriver //py:selenium-wheel