Fix browser connection goroutine leaks in tests (#5647)

TestBrowserTypeConnect and TestMultiConnectToSingleBrowser create
browser connections via bt.Connect() but never close them. Since
both contexts are context.Background(), the child goroutines
(Connection, Session, BaseEventEmitter.syncAll, etc.) live forever.

Add t.Cleanup(b.Close) to ensure browsers created via Connect are
properly closed when the test finishes.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: ankur22

internal/js/modules/k6/browser/tests/browser_test.go

@@ -329,13 +329,15 @@ func TestMultiConnectToSingleBrowser(t *testing.T) {
 
 	b1, err := tb.browserType.Connect(context.Background(), ctx, tb.wsURL)
 	require.NoError(t, err)
+	t.Cleanup(b1.Close)
 	bctx1, err := b1.NewContext(nil)
 	require.NoError(t, err)
 	p1, err := bctx1.NewPage()
 	require.NoError(t, err, "failed to create page #1")
 
 	b2, err := tb.browserType.Connect(context.Background(), ctx, tb.wsURL)
 	require.NoError(t, err)
+	t.Cleanup(b2.Close)
 	bctx2, err := b2.NewContext(nil)
 	require.NoError(t, err)
 

internal/js/modules/k6/browser/tests/browser_type_test.go

@@ -24,6 +24,7 @@ func TestBrowserTypeConnect(t *testing.T) {
 
 	b, err := bt.Connect(context.Background(), context.Background(), tb.wsURL)
 	require.NoError(t, err)
+	t.Cleanup(b.Close)
 	_, err = b.NewPage(nil)
 	require.NoError(t, err)
 }