Trivial - rename compiler flag

craigfowler · craigfowler · commit f6cb81ddfc58 · 2026-02-15T20:45:35.000Z
diff --git a/Tests/CSF.Screenplay.Selenium.Tests/Properties/LevelOfParallelismAttribute.cs b/Tests/CSF.Screenplay.Selenium.Tests/Properties/LevelOfParallelismAttribute.cs
@@ -3,6 +3,6 @@
  * test runs, with many duplicate tests.
  */
 
-#if NO_NUNIT_PARALLELISM
+#if BROWSER_STACK_ENABLED
 [assembly:LevelOfParallelism(1)]
 #endif
diff --git a/Tests/CSF.Screenplay.Selenium.Tests/TestWebappSetupAndTeardown.cs b/Tests/CSF.Screenplay.Selenium.Tests/TestWebappSetupAndTeardown.cs
@@ -19,7 +19,10 @@ public async Task StartWebAppAsync()
         webAppProcess = Process.Start("dotnet", $"run --project {GetPathToWebappProject()}");
         await WaitForWebAppToBeAvailableAsync();
 
+#if BROWSER_STACK_ENABLED
+        // If running with BrowserStack then delay because I suspect a race condition
         await Task.Delay(TimeSpan.FromSeconds(5));
+#endif
     }
 
     [OneTimeTearDown]
diff --git a/Tests/CSF.Screenplay.Selenium.Tests/Tools/BrowserStack.props b/Tests/CSF.Screenplay.Selenium.Tests/Tools/BrowserStack.props
@@ -1,6 +1,6 @@
 <Project>
   <PropertyGroup>
-    <DefineConstants>$(DefineConstants);NO_NUNIT_PARALLELISM</DefineConstants>
+    <DefineConstants>$(DefineConstants);BROWSER_STACK_ENABLED</DefineConstants>
   </PropertyGroup>
 
   <ItemGroup>

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,10 @@ public async Task StartWebAppAsync()`
`19`	`19`	`webAppProcess = Process.Start("dotnet", $"run --project {GetPathToWebappProject()}");`
`20`	`20`	`await WaitForWebAppToBeAvailableAsync();`
`21`	`21`
	`22`	`+#if BROWSER_STACK_ENABLED`
	`23`	`+ // If running with BrowserStack then delay because I suspect a race condition`
`22`	`24`	`await Task.Delay(TimeSpan.FromSeconds(5));`
	`25`	`+#endif`
`23`	`26`	`}`
`24`	`27`
`25`	`28`	`[OneTimeTearDown]`