[java] bug(#12791) Grid vnc session is not using the --sub-path argument (#13219)

Signed-off-by: Viet Nguyen Duc <[email protected]>

Author: VietND96

java/src/org/openqa/selenium/grid/node/local/LocalNode.java

@@ -838,6 +838,9 @@ private Session createExternalSession(
   private URI rewrite(String path) {
     try {
       String scheme = "https".equals(gridUri.getScheme()) ? "wss" : "ws";
+      if (gridUri.getPath() != null && !gridUri.getPath().isEmpty()) {
+        path = gridUri.getPath() + path;
+      }
       return new URI(
           scheme, gridUri.getUserInfo(), gridUri.getHost(), gridUri.getPort(), path, null, null);
     } catch (URISyntaxException e) {

java/test/org/openqa/selenium/grid/node/local/LocalNodeTest.java

@@ -279,6 +279,42 @@ void nodeDrainsAfterSessionCountIsReached() throws URISyntaxException {
     assertThat(localNode.isDraining()).isTrue();
   }
 
+  @Test
+  void seVncCdpUrlCapabilityWhenGridUrlWithSubPath() throws URISyntaxException {
+    Tracer tracer = DefaultTestTracer.createTracer();
+    EventBus bus = new GuavaEventBus();
+    String gridUrl = "http://localhost:7890/subPath";
+    URI uri = new URI(gridUrl);
+    Capabilities stereotype =
+        new ImmutableCapabilities(
+            "se:vncLocalAddress", "ws://localhost:7900",
+            "se:cdp", "ws://localhost:9222/devtools/browser/1a2b3c4d5e6f");
+
+    LocalNode.Builder builder =
+        LocalNode.builder(tracer, bus, uri, uri, registrationSecret)
+            .enableCdp(true)
+            .add(
+                stereotype,
+                new TestSessionFactory(
+                    (id, caps) -> new Session(id, uri, stereotype, caps, Instant.now())));
+    LocalNode localNode = builder.build();
+
+    Either<WebDriverException, CreateSessionResponse> response =
+        localNode.newSession(
+            new CreateSessionRequest(ImmutableSet.of(W3C), stereotype, ImmutableMap.of()));
+    assertThat(response.isRight()).isTrue();
+
+    CreateSessionResponse sessionResponse = response.right();
+    Capabilities capabilities = sessionResponse.getSession().getCapabilities();
+    Object seVnc = capabilities.getCapability("se:vnc");
+    assertThat(seVnc).isNotNull();
+    assertThat(seVnc.toString().contains(gridUrl.replace("http", "ws"))).isTrue();
+
+    Object seCdp = capabilities.getCapability("se:cdp");
+    assertThat(seCdp).isNotNull();
+    assertThat(seCdp.toString().contains(gridUrl.replace("http", "ws"))).isTrue();
+  }
+
   @Test
   void cdpIsDisabledAndResponseCapsShowThat() throws URISyntaxException {
     Tracer tracer = DefaultTestTracer.createTracer();