Optimize forgotten tests to parallel

Co-authored-by: ka3de <danijs12@hotmail.com>

common/connection_test.go

@@ -19,9 +19,13 @@ import (
 )
 
 func TestConnection(t *testing.T) {
+	t.Parallel()
+
 	server := ws.NewServer(t, ws.WithEchoHandler("/echo"))
 
 	t.Run("connect", func(t *testing.T) {
+		t.Parallel()
+
 		ctx := context.Background()
 		url, _ := url.Parse(server.ServerHTTP.URL)
 		wsURL := fmt.Sprintf("ws://%s/echo", url.Host)
@@ -33,9 +37,13 @@ func TestConnection(t *testing.T) {
 }
 
 func TestConnectionClosureAbnormal(t *testing.T) {
+	t.Parallel()
+
 	server := ws.NewServer(t, ws.WithClosureAbnormalHandler("/closure-abnormal"))
 
 	t.Run("closure abnormal", func(t *testing.T) {
+		t.Parallel()
+
 		ctx := context.Background()
 		url, _ := url.Parse(server.ServerHTTP.URL)
 		wsURL := fmt.Sprintf("ws://%s/closure-abnormal", url.Host)
@@ -50,9 +58,13 @@ func TestConnectionClosureAbnormal(t *testing.T) {
 }
 
 func TestConnectionSendRecv(t *testing.T) {
+	t.Parallel()
+
 	server := ws.NewServer(t, ws.WithCDPHandler("/cdp", ws.CDPDefaultHandler, nil))
 
 	t.Run("send command with empty reply", func(t *testing.T) {
+		t.Parallel()
+
 		ctx := context.Background()
 		url, _ := url.Parse(server.ServerHTTP.URL)
 		wsURL := fmt.Sprintf("ws://%s/cdp", url.Host)
@@ -67,6 +79,8 @@ func TestConnectionSendRecv(t *testing.T) {
 }
 
 func TestConnectionCreateSession(t *testing.T) {
+	t.Parallel()
+
 	cmdsReceived := make([]cdproto.MethodType, 0)
 	handler := func(conn *websocket.Conn, msg *cdproto.Message, writeCh chan cdproto.Message, done chan struct{}) {
 		if msg.SessionID == "" && msg.Method != "" {
@@ -116,6 +130,8 @@ func TestConnectionCreateSession(t *testing.T) {
 	server := ws.NewServer(t, ws.WithCDPHandler("/cdp", handler, &cmdsReceived))
 
 	t.Run("create session for target", func(t *testing.T) {
+		t.Parallel()
+
 		ctx := context.Background()
 		url, _ := url.Parse(server.ServerHTTP.URL)
 		wsURL := fmt.Sprintf("ws://%s/cdp", url.Host)

common/context_test.go

@@ -9,6 +9,8 @@ import (
 )
 
 func TestContextWithDoneChan(t *testing.T) {
+	t.Parallel()
+
 	done := make(chan struct{})
 	ctx := contextWithDoneChan(context.Background(), done)
 	close(done)

common/page_test.go

@@ -12,6 +12,8 @@ import (
 // tests. Since we don't yet have any of them, it makes sense to keep
 // this test.
 func TestPageLocator(t *testing.T) {
+	t.Parallel()
+
 	const (
 		wantMainFrameID = "1"
 		wantSelector    = "span"

tests/element_handle_test.go

@@ -299,8 +299,7 @@ func TestElementHandleIsChecked(t *testing.T) {
 }
 
 func TestElementHandleQueryAll(t *testing.T) {
-	// tests are faster than parallel execution.
-	// do not make this test parallel.
+	t.Parallel()
 
 	const (
 		wantLiLen = 2
@@ -316,6 +315,8 @@ func TestElementHandleQueryAll(t *testing.T) {
   	`, nil)
 
 	t.Run("element_handle", func(t *testing.T) {
+		t.Parallel()
+
 		el, err := p.Query("#aul")
 		require.NoError(t, err)
 
@@ -325,12 +326,16 @@ func TestElementHandleQueryAll(t *testing.T) {
 		assert.Equal(t, wantLiLen, len(els))
 	})
 	t.Run("page", func(t *testing.T) {
+		t.Parallel()
+
 		els, err := p.QueryAll(query)
 		require.NoError(t, err)
 
 		assert.Equal(t, wantLiLen, len(els))
 	})
 	t.Run("frame", func(t *testing.T) {
+		t.Parallel()
+
 		els, err := p.MainFrame().QueryAll(query)
 		require.NoError(t, err)
 		assert.Equal(t, wantLiLen, len(els))

tests/locator_test.go

@@ -434,6 +434,8 @@ func TestLocatorElementState(t *testing.T) {
 }
 
 func TestLocatorPress(t *testing.T) {
+	t.Parallel()
+
 	tb := newTestBrowser(t)
 
 	p := tb.NewPage(nil)