From 6442f61943f628731ed29e6310c766d83d74c15a Mon Sep 17 00:00:00 2001
From: Michael Render <render.michael@gmail.com>
Date: Wed, 11 Dec 2024 15:29:08 -0500
Subject: [PATCH] [dotnet] Add nullability to `Alert`s (#14669)

---
 dotnet/src/webdriver/Alert.cs  | 11 +++++++----
 dotnet/src/webdriver/Cookie.cs |  6 ++----
 dotnet/src/webdriver/IAlert.cs |  7 ++++++-
 3 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/dotnet/src/webdriver/Alert.cs b/dotnet/src/webdriver/Alert.cs
index 11e860af922a7..d7637b7465a6b 100644
--- a/dotnet/src/webdriver/Alert.cs
+++ b/dotnet/src/webdriver/Alert.cs
@@ -20,6 +20,8 @@
 using System;
 using System.Collections.Generic;
 
+#nullable enable
+
 namespace OpenQA.Selenium
 {
     /// <summary>
@@ -27,7 +29,7 @@ namespace OpenQA.Selenium
     /// </summary>
     internal class Alert : IAlert
     {
-        private WebDriver driver;
+        private readonly WebDriver driver;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="Alert"/> class.
@@ -41,12 +43,12 @@ public Alert(WebDriver driver)
         /// <summary>
         /// Gets the text of the alert.
         /// </summary>
-        public string Text
+        public string? Text
         {
             get
             {
                 Response commandResponse = this.driver.InternalExecute(DriverCommand.GetAlertText, null);
-                return commandResponse.Value.ToString();
+                return (string?)commandResponse.Value;
             }
         }
 
@@ -70,9 +72,10 @@ public void Accept()
         /// Sends keys to the alert.
         /// </summary>
         /// <param name="keysToSend">The keystrokes to send.</param>
+        /// <exception cref="ArgumentNullException">If <paramref name="keysToSend" /> is <see langword="null"/>.</exception>
         public void SendKeys(string keysToSend)
         {
-            if (keysToSend == null)
+            if (keysToSend is null)
             {
                 throw new ArgumentNullException(nameof(keysToSend), "Keys to send must not be null.");
             }
diff --git a/dotnet/src/webdriver/Cookie.cs b/dotnet/src/webdriver/Cookie.cs
index 693012815d699..b15dca6d196fe 100644
--- a/dotnet/src/webdriver/Cookie.cs
+++ b/dotnet/src/webdriver/Cookie.cs
@@ -351,14 +351,12 @@ public override string ToString()
         public override bool Equals(object obj)
         {
             // Two cookies are equal if the name and value match
-            Cookie cookie = obj as Cookie;
-
             if (this == obj)
             {
                 return true;
             }
 
-            if (cookie == null)
+            if (obj is not Cookie cookie)
             {
                 return false;
             }
@@ -368,7 +366,7 @@ public override bool Equals(object obj)
                 return false;
             }
 
-            return !(this.cookieValue != null ? !this.cookieValue.Equals(cookie.cookieValue) : cookie.Value != null);
+            return string.Equals(this.cookieValue, cookie.cookieValue);
         }
 
         /// <summary>
diff --git a/dotnet/src/webdriver/IAlert.cs b/dotnet/src/webdriver/IAlert.cs
index 6dd0c8db63e99..78c15d992c1b3 100644
--- a/dotnet/src/webdriver/IAlert.cs
+++ b/dotnet/src/webdriver/IAlert.cs
@@ -17,6 +17,10 @@
 // under the License.
 // </copyright>
 
+using System;
+
+#nullable enable
+
 namespace OpenQA.Selenium
 {
     /// <summary>
@@ -27,7 +31,7 @@ public interface IAlert
         /// <summary>
         /// Gets the text of the alert.
         /// </summary>
-        string Text { get; }
+        string? Text { get; }
 
         /// <summary>
         /// Dismisses the alert.
@@ -43,6 +47,7 @@ public interface IAlert
         /// Sends keys to the alert.
         /// </summary>
         /// <param name="keysToSend">The keystrokes to send.</param>
+        /// <exception cref="ArgumentNullException">If <paramref name="keysToSend"/> is <see langword="null"/>.</exception>
         void SendKeys(string keysToSend);
     }
 }