[dotnet] Quit fails after not successful new session (SeleniumHQ#14242)

* [dotnet] do not execute Quit command when sessionId is not available

If new session creation fails a Quit() method is called which calls Dispose and then Quit command, unfortunately in this case
session is still not created and it throws null reference exception and this hides the actual exception

* [dotnet] ignore exceptions from Quit when new session fails

In order to propagate the original error when session fails, surround the Quit call in try/catch in case it also throws an exception.

---------

Co-authored-by: Nikolay Borisenko <[email protected]>

Author: 2 people

dotnet/src/webdriver/WebDriver.cs

@@ -62,8 +62,15 @@ protected WebDriver(ICommandExecutor executor, ICapabilities capabilities)
             }
             catch (Exception)
             {
-                // Failed to start driver session, disposing of driver
-                this.Quit();
+                try
+                {
+                    // Failed to start driver session, disposing of driver
+                    this.Quit();
+                }
+                catch
+                {
+                    // Ignore the clean-up exception. We'll propagate the original failure.
+                }
                 throw;
             }
 
@@ -703,7 +710,10 @@ protected virtual void Dispose(bool disposing)
         {
             try
             {
-                this.Execute(DriverCommand.Quit, null);
+                if (this.sessionId is not null)
+                {
+                    this.Execute(DriverCommand.Quit, null);
+                }
             }
             catch (NotImplementedException)
             {