Minor tech debt fixes

craigfowler · craigfowler · commit c9578cc62a5e · 2026-01-27T22:22:15.000Z
diff --git a/CSF.Screenplay.Selenium/Tasks/EnterTheDate.cs b/CSF.Screenplay.Selenium/Tasks/EnterTheDate.cs
@@ -32,6 +32,7 @@ namespace CSF.Screenplay.Selenium.Actions
     public class EnterTheDate : IPerformable, ICanReport
     {
         const string nonNumericPattern = @"\D";
+        
         static readonly Regex nonNumeric = new Regex(nonNumericPattern,
                                                      RegexOptions.Compiled | RegexOptions.IgnoreCase | RegexOptions.CultureInvariant,
                                                      TimeSpan.FromMilliseconds(100));
diff --git a/Tests/CSF.Screenplay.Selenium.Tests/BrowserStack/BrowserStackDriverFactory.cs b/Tests/CSF.Screenplay.Selenium.Tests/BrowserStack/BrowserStackDriverFactory.cs
@@ -34,7 +34,7 @@ public WebDriverAndOptions GetWebDriver(WebDriverCreationOptions options, Action
         return new (driver, driverOptions);
     }
 
-    DriverOptions GetDriverOptions()
+    static DriverOptions GetDriverOptions()
     {
         var browserName = GetBrowserName();
         return browserName switch

Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,7 @@ namespace CSF.Screenplay.Selenium.Actions`
`32`	`32`	`public class EnterTheDate : IPerformable, ICanReport`
`33`	`33`	`{`
`34`	`34`	`const string nonNumericPattern = @"\D";`
	`35`	`+`
`35`	`36`	`static readonly Regex nonNumeric = new Regex(nonNumericPattern,`
`36`	`37`	`RegexOptions.Compiled \| RegexOptions.IgnoreCase \| RegexOptions.CultureInvariant,`
`37`	`38`	`TimeSpan.FromMilliseconds(100));`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public WebDriverAndOptions GetWebDriver(WebDriverCreationOptions options, Action`
`34`	`34`	`return new (driver, driverOptions);`
`35`	`35`	`}`
`36`	`36`
`37`		`- DriverOptions GetDriverOptions()`
	`37`	`+ static DriverOptions GetDriverOptions()`
`38`	`38`	`{`
`39`	`39`	`var browserName = GetBrowserName();`
`40`	`40`	`return browserName switch`