fail tests if the first testAllocate fails

Otherwise we abruptly stop with a panic.
chromedp · Nov 27, 2019 · d883c83 · d883c83
1 parent aabea60
commit d883c83
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/chromedp_test.go b/chromedp_test.go
@@ -96,6 +96,11 @@ func testAllocate(tb testing.TB, name string) (context.Context, context.CancelFu
 	// Start the browser exactly once, as needed.
 	allocateOnce.Do(func() { browserCtx, _ = testAllocateSeparate(tb) })
 
+	if browserCtx == nil {
+		// allocateOnce.Do failed; continuing would result in panics.
+		tb.FailNow()
+	}
+
 	// Same browser, new tab; not needing to start new chrome browsers for
 	// each test gives a huge speed-up.
 	ctx, _ := NewContext(browserCtx)