test: wait for workerFunction to be defined in worker test (#561)

* test: wait for workerFunction to be defined in worker test
* test: page workers may not immediately available in macos

---------

Co-authored-by: canstand <[email protected]>

Author: liruohrh

tests/page_test.go

@@ -258,6 +258,15 @@ func TestPageExpectWorker(t *testing.T) {
 	require.Equal(t, worker, page.Workers()[0])
 	worker = page.Workers()[0]
 	require.Contains(t, worker.URL(), "worker.js")
+	// flaky in the macos-latest of gh action
+	require.Eventually(t,
+		func() bool {
+			v, err := worker.Evaluate(`() => self["workerFunction"] ? true : false`)
+			require.NoError(t, err)
+			return v == true
+		},
+		500*time.Millisecond, 10*time.Millisecond,
+	)
 	res, err := worker.Evaluate(`() => self["workerFunction"]()`)
 	require.NoError(t, err)
 	require.Equal(t, "worker function result", res)

tests/worker_test.go

@@ -2,6 +2,7 @@ package playwright_test
 
 import (
 	"testing"
+	"time"
 
 	"github.com/playwright-community/playwright-go"
 	"github.com/stretchr/testify/require"
@@ -19,6 +20,15 @@ func TestWorkerShouldWork(t *testing.T) {
 	require.Equal(t, worker, page.Workers()[0])
 	worker = page.Workers()[0]
 	require.Contains(t, worker.URL(), "worker.js")
+	// flaky in the macos-latest of gh action
+	require.Eventually(t,
+		func() bool {
+			v, err := worker.Evaluate(`() => self["workerFunction"] ? true : false`)
+			require.NoError(t, err)
+			return v == true
+		},
+		500*time.Millisecond, 10*time.Millisecond,
+	)
 	res, err := worker.Evaluate(`() => self["workerFunction"]()`)
 	require.NoError(t, err)
 	require.Equal(t, "worker function result", res)