[grid] Ignoring NoSuchSessionException in SESSION_CLOSED event listen…

…er on a node.

This exception is thrown if the session with the given id does not exist on this node. When a node stops the session on driver.quit() call it notifies all the parties by sending SESSION_CLOSED event to MQ. The distributor handles this event and commands the "local node" to stop the session. But if the distributor and the node are working in the same process the session is already closed on the "local node" and it throws. Ignoring this exception looks safe because we can't do (and shouldn't do) anything in this case.

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

@@ -115,7 +115,11 @@ private LocalNode(
     this.regularly = new Regularly("Local Node: " + externalUri);
     regularly.submit(currentSessions::cleanUp, Duration.ofSeconds(30), Duration.ofSeconds(30));
 
-    bus.addListener(SESSION_CLOSED, event -> this.stop(event.getData(SessionId.class)));
+    bus.addListener(SESSION_CLOSED, event -> {
+      try {
+        this.stop(event.getData(SessionId.class));
+      } catch (NoSuchSessionException ignore) {}
+    });
   }
 
   @VisibleForTesting