Rework known capabilities setting logic for Enum types to handle "unhandledPromptBehavior" capability (#196)

mialeska · web-flow · commit e734cd549a18 · 2020-11-18T16:21:51.000+02:00
resolves #195
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/ChromeSettings.cs b/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/ChromeSettings.cs
@@ -30,7 +30,7 @@ public override DriverOptions DriverOptions
                 SetChromePrefs(options);
                 SetCapabilities(options, (name, value) => options.AddAdditionalCapability(name, value, isGlobalCapability: true));
                 SetChromeArguments(options);
-                SetPageLoadStratergy(options);
+                SetPageLoadStrategy(options);
                 return options;
             }
         }
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/DriverSettings.cs b/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/DriverSettings.cs
@@ -67,7 +67,7 @@ protected IReadOnlyDictionary<string, object> BrowserCapabilities
                             args: string.Join(",", capabilities.Select(cap => $"{Environment.NewLine}{cap.Key}: {cap.Value}")));
                     }
                 }
-                
+
                 return capabilities;
             }
         }
@@ -78,14 +78,14 @@ protected IReadOnlyDictionary<string, object> BrowserOptions
             {
                 if (options == null)
                 {
-                    options = SettingsFile.GetValueDictionaryOrEmpty<object>($"{DriverSettingsPath}.{nameof(options)}"); 
+                    options = SettingsFile.GetValueDictionaryOrEmpty<object>($"{DriverSettingsPath}.{nameof(options)}");
                     if (options.Any())
                     {
                         AqualityServices.LocalizedLogger.Debug("loc.browser.options",
                             args: string.Join(",", options.Select(opt => $"{Environment.NewLine}{opt.Key}: {opt.Value}")));
                     }
-                }                
-                
+                }
+
                 return options;
             }
         }
@@ -107,13 +107,13 @@ protected IReadOnlyList<string> BrowserStartArguments
             }
         }
 
-        private string DriverSettingsPath => $".driverSettings.{BrowserName.ToString().ToLowerInvariant()}";        
+        private string DriverSettingsPath => $".driverSettings.{BrowserName.ToString().ToLowerInvariant()}";
 
         protected abstract BrowserName BrowserName { get; }
 
         protected virtual IDictionary<string, Action<DriverOptions, object>> KnownCapabilitySetters => new Dictionary<string, Action<DriverOptions, object>>();
 
-        protected void SetPageLoadStratergy(DriverOptions options)
+        protected void SetPageLoadStrategy(DriverOptions options)
         {
             options.PageLoadStrategy = PageLoadStrategy;
         }
@@ -126,10 +126,10 @@ protected void SetCapabilities(DriverOptions options, Action<string, object> add
                 {
                     var defaultAddCapabilityMethod = addCapabilityMethod ?? options.AddAdditionalCapability;
                     defaultAddCapabilityMethod(capability.Key, capability.Value);
-                } 
+                }
                 catch (ArgumentException exception)
                 {
-                    if(exception.Message.StartsWith("There is already an option"))
+                    if (exception.Message.StartsWith("There is already an option"))
                     {
                         SetKnownProperty(options, capability, exception);
                     }
@@ -150,7 +150,7 @@ private void SetKnownProperty(DriverOptions options, KeyValuePair<string, object
             else
             {
                 SetOptionByPropertyName(options, capability, exception);
-            }            
+            }
         }
 
         protected void SetOptionsByPropertyNames(DriverOptions options)
@@ -168,7 +168,41 @@ private void SetOptionByPropertyName(DriverOptions options, KeyValuePair<string,
                             .GetProperties()
                             .FirstOrDefault(property => IsPropertyNameMatchOption(property.Name, option.Key) && property.CanWrite)
                             ?? throw exception;
-            optionProperty.SetValue(options, option.Value);
+            var propertyType = optionProperty.PropertyType;
+            var valueToSet = IsEnumValue(propertyType, option.Value)
+                ? ParseEnumValue(propertyType, option.Value)
+                : option.Value;
+            optionProperty.SetValue(options, valueToSet);
+        }
+
+        private object ParseEnumValue(Type propertyType, object optionValue)
+        {
+            return optionValue is string
+                ? Enum.Parse(propertyType, optionValue.ToString(), ignoreCase: true)
+                : Enum.ToObject(propertyType, Convert.ChangeType(optionValue, Enum.GetUnderlyingType(propertyType)));
+        }
+
+        private bool IsEnumValue(Type propertyType, object optionValue)
+        {
+            var valueAsString = optionValue.ToString();
+            if (!propertyType.IsEnum || string.IsNullOrEmpty(valueAsString))
+            {
+                return false;
+            }
+            var normalizedValue = char.ToUpper(valueAsString[0]) +
+                (valueAsString.Length > 1 ? valueAsString.Substring(1) : string.Empty);
+            return propertyType.IsEnumDefined(normalizedValue)
+                || propertyType.IsEnumDefined(valueAsString)
+                || (IsValueOfIntegralNumericType(optionValue)
+                    && propertyType.IsEnumDefined(Convert.ChangeType(optionValue, Enum.GetUnderlyingType(propertyType))));
+        }
+
+        private bool IsValueOfIntegralNumericType(object value)
+        {
+            return value is byte || value is sbyte
+                || value is ushort || value is short
+                || value is uint || value is int
+                || value is ulong || value is long;
         }
 
         private bool IsPropertyNameMatchOption(string propertyName, string optionKey)
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/EdgeChromiumSettings.cs b/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/EdgeChromiumSettings.cs
@@ -33,7 +33,7 @@ public override DriverOptions DriverOptions
                 SetEdgeChromiumPrefs(options);
                 SetCapabilities(options, (name, value) => options.AddAdditionalCapability(name, value, isGlobalCapability: true));
                 SetEdgeChromiumArguments(options);
-                SetPageLoadStratergy(options);
+                SetPageLoadStrategy(options);
                 return options;
             }
         }
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/EdgeSettings.cs b/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/EdgeSettings.cs
@@ -28,7 +28,7 @@ public override DriverOptions DriverOptions
                 var options = new EdgeOptions();
                 SetCapabilities(options);
                 SetOptionsByPropertyNames(options);
-                SetPageLoadStratergy(options);
+                SetPageLoadStrategy(options);
                 return options;
             }
         }
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/FirefoxSettings.cs b/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/FirefoxSettings.cs
@@ -40,7 +40,7 @@ public override DriverOptions DriverOptions
                 SetCapabilities(options, (name, value) => options.AddAdditionalCapability(name, value, isGlobalCapability: true));
                 SetFirefoxPrefs(options);
                 SetFirefoxArguments(options);
-                SetPageLoadStratergy(options);
+                SetPageLoadStrategy(options);
                 return options;
             }
         }
@@ -54,21 +54,21 @@ private void SetFirefoxPrefs(FirefoxOptions options)
                 {
                     options.SetPreference(option.Key, DownloadDir);
                 }
-                else if (value is bool)
+                else if (value is bool boolean)
                 {
-                    options.SetPreference(option.Key, (bool) value);
+                    options.SetPreference(option.Key, boolean);
                 }
-                else if (value is int)
+                else if (value is int @int)
                 {
-                    options.SetPreference(option.Key, (int) value);
+                    options.SetPreference(option.Key, @int);
                 }
-                else if (value is long)
+                else if (value is long @long)
                 {
-                    options.SetPreference(option.Key, (long)value);
+                    options.SetPreference(option.Key, @long);
                 }
-                else if (value is float)
+                else if (value is float @float)
                 {
-                    options.SetPreference(option.Key, (float) value);
+                    options.SetPreference(option.Key, @float);
                 }
                 else if (value is string)
                 {
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/InternetExplorerSettings.cs b/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/InternetExplorerSettings.cs
@@ -2,7 +2,6 @@
 using System.Collections.Generic;
 using Aquality.Selenium.Browsers;
 using Aquality.Selenium.Core.Configurations;
-using Aquality.Selenium.Core.Utilities;
 using OpenQA.Selenium;
 using OpenQA.Selenium.IE;
 using OpenQA.Selenium.Remote;
@@ -29,9 +28,7 @@ public InternetExplorerSettings(ISettingsFile settingsFile) : base(settingsFile)
             { "ignoreProtectedModeSettings", (options, value) => ((InternetExplorerOptions) options).IntroduceInstabilityByIgnoringProtectedModeSettings = (bool) value },
             { "ignoreZoomSetting", (options, value) => ((InternetExplorerOptions) options).IgnoreZoomLevel = (bool) value },
             { CapabilityType.HasNativeEvents, (options, value) => ((InternetExplorerOptions) options).EnableNativeEvents = (bool) value },
-            { CapabilityType.UnexpectedAlertBehavior, (options, value) => ((InternetExplorerOptions) options).UnhandledPromptBehavior = value.ToEnum<UnhandledPromptBehavior>() },
-            { "ie.browserCommandLineSwitches", (options, value) => ((InternetExplorerOptions) options).BrowserCommandLineArguments = value.ToString() },
-            { "elementScrollBehavior", (options, value) => ((InternetExplorerOptions) options).ElementScrollBehavior = value.ToEnum<InternetExplorerElementScrollBehavior>() }
+            { "ie.browserCommandLineSwitches", (options, value) => ((InternetExplorerOptions) options).BrowserCommandLineArguments = value.ToString() }
         };
 
         public override DriverOptions DriverOptions
@@ -41,7 +38,7 @@ public override DriverOptions DriverOptions
                 var options = new InternetExplorerOptions();
                 SetCapabilities(options);
                 SetOptionsByPropertyNames(options);
-                SetPageLoadStratergy(options);
+                SetPageLoadStrategy(options);
                 options.BrowserCommandLineArguments = string.Join(" ", BrowserStartArguments);
                 return options;
             }
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/SafariSettings.cs b/Aquality.Selenium/src/Aquality.Selenium/Configurations/WebDriverSettings/SafariSettings.cs
@@ -27,7 +27,7 @@ public override DriverOptions DriverOptions
                 var options = new SafariOptions();
                 SetCapabilities(options);
                 SetOptionsByPropertyNames(options);
-                SetPageLoadStratergy(options);
+                SetPageLoadStrategy(options);
                 return options;
             }
         }
diff --git a/Aquality.Selenium/src/Aquality.Selenium/Resources/settings.json b/Aquality.Selenium/src/Aquality.Selenium/Resources/settings.json
@@ -13,7 +13,8 @@
           "driver": "INFO",
           "server": "OFF",
           "browser": "FINE"
-        }
+        },
+        "unhandledPromptBehavior": "default"
       },
       "options": {
         "intl.accept_languages": "en",
@@ -29,7 +30,8 @@
     "firefox": {
       "webDriverVersion": "Latest",
       "capabilities": {
-        "enableVNC": true
+        "enableVNC": true,
+        "unhandledPromptBehavior": "default"
       },
       "options": {
         "intl.accept_languages": "en",
@@ -52,7 +54,8 @@
       "webDriverVersion": "Latest",
       "systemArchitecture": "X32",
       "capabilities": {
-        "ignoreProtectedModeSettings": true
+        "ignoreProtectedModeSettings": true,
+        "unhandledPromptBehavior": "default"
       },
       "options": {
       },
@@ -61,6 +64,7 @@
     "edge": {
       "systemArchitecture": "X32",
       "capabilities": {
+        "unhandledPromptBehavior": "default"
       },
       "options": {
       },
diff --git a/Aquality.Selenium/tests/Aquality.Selenium.Tests/Integration/AlertTests.cs b/Aquality.Selenium/tests/Aquality.Selenium.Tests/Integration/AlertTests.cs
@@ -31,6 +31,28 @@ public void Should_BePossibleTo_AcceptConfirmationAlert()
             Assert.AreEqual("You clicked: Ok", alertsForm.ResultLabel.GetText());
         }
 
+        [Test]
+        public void Should_BePossibleTo_AcceptConfirmationAlert_InWaitFor()
+        {
+            alertsForm.JsConfirmButton.Click();
+            AqualityServices.ConditionalWait.WaitFor(driver =>
+            {
+                try
+                {
+                    AqualityServices.Logger.Debug($"Current url: {driver.Url}");
+                    return false;
+                }
+                catch (UnhandledAlertException e)
+                {
+                    AqualityServices.Logger.Debug($"Alert appeared: {e.Message}");
+                    AqualityServices.Browser.HandleAlert(AlertAction.Accept);
+                    return true;
+                }
+                
+            });
+            Assert.AreEqual("You clicked: Ok", alertsForm.ResultLabel.GetText());
+        }
+
         [Test]
         public void Should_BePossibleTo_DeclineConfirmationAlert()
         {
diff --git a/Aquality.Selenium/tests/Aquality.Selenium.Tests/Resources/settings.azure.json b/Aquality.Selenium/tests/Aquality.Selenium.Tests/Resources/settings.azure.json
@@ -8,7 +8,8 @@
     "chrome": {
       "webDriverVersion": "MatchingBrowser",
       "capabilities": {
-        "enableVNC": true
+        "enableVNC": true,
+        "unhandledPromptBehavior": "ignore"
       },
       "options": {
         "intl.accept_languages": "en",
diff --git a/Aquality.Selenium/tests/Aquality.Selenium.Tests/Resources/settings.json b/Aquality.Selenium/tests/Aquality.Selenium.Tests/Resources/settings.json
@@ -8,7 +8,8 @@
     "chrome": {
       "webDriverVersion": "MatchingBrowser",
       "capabilities": {
-        "enableVNC": true
+        "enableVNC": true,
+        "unhandledPromptBehavior": "ignore"
       },
       "options": {
         "intl.accept_languages": "en",
@@ -24,7 +25,8 @@
       "webDriverVersion": "Latest",
       "systemArchitecture": "X64",
       "capabilities": {
-        "enableVNC": true
+        "enableVNC": true,
+        "unhandledPromptBehavior": "ignore"
       },
       "options": {
         "intl.accept_languages": "en",
@@ -49,12 +51,14 @@
       "capabilities": {
         "ignoreProtectedModeSettings": true,
         "requireWindowFocus": false,
-        "elementScrollBehavior": 2
+        "elementScrollBehavior": 2,
+        "unhandledPromptBehavior": "ignore"
       }
     },
     "edge": {
       "systemArchitecture": "X64",
       "capabilities": {
+        "unhandledPromptBehavior": "ignore"
       },
       "options": {
       },
@@ -63,7 +67,8 @@
     "edgechromium": {
       "webDriverVersion": "82.0.458.0",
       "capabilities": {
-        "enableVNC": true
+        "enableVNC": true,
+        "unhandledPromptBehavior": "ignore"
       },
       "options": {
         "intl.accept_languages": "en",

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ public override DriverOptions DriverOptions`
`30`	`30`	`SetChromePrefs(options);`
`31`	`31`	`SetCapabilities(options, (name, value) => options.AddAdditionalCapability(name, value, isGlobalCapability: true));`
`32`	`32`	`SetChromeArguments(options);`
`33`		`- SetPageLoadStratergy(options);`
	`33`	`+ SetPageLoadStrategy(options);`
`34`	`34`	`return options;`
`35`	`35`	`}`
`36`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ protected IReadOnlyDictionary<string, object> BrowserCapabilities`
`67`	`67`	`args: string.Join(",", capabilities.Select(cap => $"{Environment.NewLine}{cap.Key}: {cap.Value}")));`
`68`	`68`	`}`
`69`	`69`	`}`
`70`		`-`
	`70`	`+`
`71`	`71`	`return capabilities;`
`72`	`72`	`}`
`73`	`73`	`}`
`@@ -78,14 +78,14 @@ protected IReadOnlyDictionary<string, object> BrowserOptions`
`78`	`78`	`{`
`79`	`79`	`if (options == null)`
`80`	`80`	`{`
`81`		`- options = SettingsFile.GetValueDictionaryOrEmpty<object>($"{DriverSettingsPath}.{nameof(options)}");`
	`81`	`+ options = SettingsFile.GetValueDictionaryOrEmpty<object>($"{DriverSettingsPath}.{nameof(options)}");`
`82`	`82`	`if (options.Any())`
`83`	`83`	`{`
`84`	`84`	`AqualityServices.LocalizedLogger.Debug("loc.browser.options",`
`85`	`85`	`args: string.Join(",", options.Select(opt => $"{Environment.NewLine}{opt.Key}: {opt.Value}")));`
`86`	`86`	`}`
`87`		`- }`
`88`		`-`
	`87`	`+ }`
	`88`	`+`
`89`	`89`	`return options;`
`90`	`90`	`}`
`91`	`91`	`}`
`@@ -107,13 +107,13 @@ protected IReadOnlyList<string> BrowserStartArguments`
`107`	`107`	`}`
`108`	`108`	`}`
`109`	`109`
`110`		`- private string DriverSettingsPath => $".driverSettings.{BrowserName.ToString().ToLowerInvariant()}";`
	`110`	`+ private string DriverSettingsPath => $".driverSettings.{BrowserName.ToString().ToLowerInvariant()}";`
`111`	`111`
`112`	`112`	`protected abstract BrowserName BrowserName { get; }`
`113`	`113`
`114`	`114`	`protected virtual IDictionary<string, Action<DriverOptions, object>> KnownCapabilitySetters => new Dictionary<string, Action<DriverOptions, object>>();`
`115`	`115`
`116`		`- protected void SetPageLoadStratergy(DriverOptions options)`
	`116`	`+ protected void SetPageLoadStrategy(DriverOptions options)`
`117`	`117`	`{`
`118`	`118`	`options.PageLoadStrategy = PageLoadStrategy;`
`119`	`119`	`}`
`@@ -126,10 +126,10 @@ protected void SetCapabilities(DriverOptions options, Action<string, object> add`
`126`	`126`	`{`
`127`	`127`	`var defaultAddCapabilityMethod = addCapabilityMethod ?? options.AddAdditionalCapability;`
`128`	`128`	`defaultAddCapabilityMethod(capability.Key, capability.Value);`
`129`		`- }`
	`129`	`+ }`
`130`	`130`	`catch (ArgumentException exception)`
`131`	`131`	`{`
`132`		`- if(exception.Message.StartsWith("There is already an option"))`
	`132`	`+ if (exception.Message.StartsWith("There is already an option"))`
`133`	`133`	`{`
`134`	`134`	`SetKnownProperty(options, capability, exception);`
`135`	`135`	`}`
`@@ -150,7 +150,7 @@ private void SetKnownProperty(DriverOptions options, KeyValuePair<string, object`
`150`	`150`	`else`
`151`	`151`	`{`
`152`	`152`	`SetOptionByPropertyName(options, capability, exception);`
`153`		`- }`
	`153`	`+ }`
`154`	`154`	`}`
`155`	`155`
`156`	`156`	`protected void SetOptionsByPropertyNames(DriverOptions options)`
`@@ -168,7 +168,41 @@ private void SetOptionByPropertyName(DriverOptions options, KeyValuePair<string,`
`168`	`168`	`.GetProperties()`
`169`	`169`	`.FirstOrDefault(property => IsPropertyNameMatchOption(property.Name, option.Key) && property.CanWrite)`
`170`	`170`	`?? throw exception;`
`171`		`- optionProperty.SetValue(options, option.Value);`
	`171`	`+ var propertyType = optionProperty.PropertyType;`
	`172`	`+ var valueToSet = IsEnumValue(propertyType, option.Value)`
	`173`	`+ ? ParseEnumValue(propertyType, option.Value)`
	`174`	`+ : option.Value;`
	`175`	`+ optionProperty.SetValue(options, valueToSet);`
	`176`	`+ }`
	`177`	`+`
	`178`	`+ private object ParseEnumValue(Type propertyType, object optionValue)`
	`179`	`+ {`
	`180`	`+ return optionValue is string`
	`181`	`+ ? Enum.Parse(propertyType, optionValue.ToString(), ignoreCase: true)`
	`182`	`+ : Enum.ToObject(propertyType, Convert.ChangeType(optionValue, Enum.GetUnderlyingType(propertyType)));`
	`183`	`+ }`
	`184`	`+`
	`185`	`+ private bool IsEnumValue(Type propertyType, object optionValue)`
	`186`	`+ {`
	`187`	`+ var valueAsString = optionValue.ToString();`
	`188`	`+ if (!propertyType.IsEnum \|\| string.IsNullOrEmpty(valueAsString))`
	`189`	`+ {`
	`190`	`+ return false;`
	`191`	`+ }`
	`192`	`+ var normalizedValue = char.ToUpper(valueAsString[0]) +`
	`193`	`+ (valueAsString.Length > 1 ? valueAsString.Substring(1) : string.Empty);`
	`194`	`+ return propertyType.IsEnumDefined(normalizedValue)`
	`195`	`+ \|\| propertyType.IsEnumDefined(valueAsString)`
	`196`	`+ \|\| (IsValueOfIntegralNumericType(optionValue)`
	`197`	`+ && propertyType.IsEnumDefined(Convert.ChangeType(optionValue, Enum.GetUnderlyingType(propertyType))));`
	`198`	`+ }`
	`199`	`+`
	`200`	`+ private bool IsValueOfIntegralNumericType(object value)`
	`201`	`+ {`
	`202`	`+ return value is byte \|\| value is sbyte`
	`203`	`+ \|\| value is ushort \|\| value is short`
	`204`	`+ \|\| value is uint \|\| value is int`
	`205`	`+ \|\| value is ulong \|\| value is long;`
`172`	`206`	`}`
`173`	`207`
`174`	`208`	`private bool IsPropertyNameMatchOption(string propertyName, string optionKey)`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public override DriverOptions DriverOptions`
`33`	`33`	`SetEdgeChromiumPrefs(options);`
`34`	`34`	`SetCapabilities(options, (name, value) => options.AddAdditionalCapability(name, value, isGlobalCapability: true));`
`35`	`35`	`SetEdgeChromiumArguments(options);`
`36`		`- SetPageLoadStratergy(options);`
	`36`	`+ SetPageLoadStrategy(options);`
`37`	`37`	`return options;`
`38`	`38`	`}`
`39`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ public override DriverOptions DriverOptions`
`28`	`28`	`var options = new EdgeOptions();`
`29`	`29`	`SetCapabilities(options);`
`30`	`30`	`SetOptionsByPropertyNames(options);`
`31`		`- SetPageLoadStratergy(options);`
	`31`	`+ SetPageLoadStrategy(options);`
`32`	`32`	`return options;`
`33`	`33`	`}`
`34`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ public override DriverOptions DriverOptions`
`40`	`40`	`SetCapabilities(options, (name, value) => options.AddAdditionalCapability(name, value, isGlobalCapability: true));`
`41`	`41`	`SetFirefoxPrefs(options);`
`42`	`42`	`SetFirefoxArguments(options);`
`43`		`- SetPageLoadStratergy(options);`
	`43`	`+ SetPageLoadStrategy(options);`
`44`	`44`	`return options;`
`45`	`45`	`}`
`46`	`46`	`}`
`@@ -54,21 +54,21 @@ private void SetFirefoxPrefs(FirefoxOptions options)`
`54`	`54`	`{`
`55`	`55`	`options.SetPreference(option.Key, DownloadDir);`
`56`	`56`	`}`
`57`		`- else if (value is bool)`
	`57`	`+ else if (value is bool boolean)`
`58`	`58`	`{`
`59`		`- options.SetPreference(option.Key, (bool) value);`
	`59`	`+ options.SetPreference(option.Key, boolean);`
`60`	`60`	`}`
`61`		`- else if (value is int)`
	`61`	`+ else if (value is int @int)`
`62`	`62`	`{`
`63`		`- options.SetPreference(option.Key, (int) value);`
	`63`	`+ options.SetPreference(option.Key, @int);`
`64`	`64`	`}`
`65`		`- else if (value is long)`
	`65`	`+ else if (value is long @long)`
`66`	`66`	`{`
`67`		`- options.SetPreference(option.Key, (long)value);`
	`67`	`+ options.SetPreference(option.Key, @long);`
`68`	`68`	`}`
`69`		`- else if (value is float)`
	`69`	`+ else if (value is float @float)`
`70`	`70`	`{`
`71`		`- options.SetPreference(option.Key, (float) value);`
	`71`	`+ options.SetPreference(option.Key, @float);`
`72`	`72`	`}`
`73`	`73`	`else if (value is string)`
`74`	`74`	`{`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public override DriverOptions DriverOptions`
`27`	`27`	`var options = new SafariOptions();`
`28`	`28`	`SetCapabilities(options);`
`29`	`29`	`SetOptionsByPropertyNames(options);`
`30`		`- SetPageLoadStratergy(options);`
	`30`	`+ SetPageLoadStrategy(options);`
`31`	`31`	`return options;`
`32`	`32`	`}`
`33`	`33`	`}`