feat: improve error handling in setup methods for Browser, Context, and Page for TUnit.Playwright (#2746)

Author: thomhurst

.github/workflows/dotnet.yml

@@ -57,9 +57,12 @@ jobs:
         if: matrix.os == 'ubuntu-latest'
         uses: docker/setup-docker-action@v4.3.0
 
-      - name: Install PlaywrightDependencies
+      - name: Install Playwright Dependencies
         run: npx playwright install-deps
 
+      - name: Install Playwright Browsers
+        run: npx playwright install
+
       - name: Build
         run: dotnet build -c Release
 

TUnit.Playwright/BrowserTest.cs

@@ -29,6 +29,11 @@ public async Task<IBrowserContext> NewContext(BrowserNewContextOptions options)
     [Before(HookType.Test, "", 0)]
     public async Task BrowserSetup()
     {
+        if (BrowserType == null)
+        {
+            throw new InvalidOperationException($"BrowserType is not initialized. This may indicate that {nameof(PlaywrightTest)}.{nameof(Playwright)} is not initialized or {nameof(PlaywrightTest)}.{nameof(PlaywrightSetup)} did not execute properly.");
+        }
+        
         var service = await BrowserService.Register(this, BrowserType, _options).ConfigureAwait(false);
         Browser = service.Browser;
     }

TUnit.Playwright/ContextTest.cs

@@ -19,6 +19,11 @@ public virtual BrowserNewContextOptions ContextOptions(TestContext testContext)
     [Before(HookType.Test, "", 0)]
     public async Task ContextSetup(TestContext testContext)
     {
+        if (Browser == null)
+        {
+            throw new InvalidOperationException($"Browser is not initialized. This may indicate that {nameof(BrowserTest)}.{nameof(BrowserSetup)} did not execute properly.");
+        }
+        
         Context = await NewContext(ContextOptions(testContext)).ConfigureAwait(false);
     }
 }

TUnit.Playwright/PageTest.cs

@@ -10,6 +10,11 @@ public class PageTest : ContextTest
     [Before(HookType.Test, "", 0)]
     public async Task PageSetup()
     {
+        if (Context == null)
+        {
+            throw new InvalidOperationException($"Browser context is not initialized. This may indicate that {nameof(ContextTest)}.{nameof(ContextSetup)} did not execute properly.");
+        }
+        
         Page = await Context.NewPageAsync().ConfigureAwait(false);
     }
 }