Remove waitForEvent helper function

This function is no longer being used.
Also removing a todo comment which no longer is needed since these
tests have been implemented in this PR.

Author: ankur22

common/helpers.go

@@ -198,26 +198,6 @@ func createWaitForEventPredicateHandler(
 	return ch, evCancelFn
 }
 
-func waitForEvent(
-	ctx context.Context,
-	emitter EventEmitter, events []string,
-	predicateFn func(data any) bool,
-	timeout time.Duration,
-) (any, error) { //nolint:unparam
-	ch, evCancelFn := createWaitForEventHandler(ctx, emitter, events, predicateFn)
-	defer evCancelFn() // Remove event handler
-
-	select {
-	case <-ctx.Done():
-	case <-time.After(timeout):
-		return nil, fmt.Errorf("%w after %s", ErrTimedOut, timeout)
-	case evData := <-ch:
-		return evData, nil
-	}
-
-	return nil, nil
-}
-
 // panicOrSlowMo panics if err is not nil, otherwise applies slow motion.
 func panicOrSlowMo(ctx context.Context, err error) {
 	if err != nil {

tests/page_test.go

@@ -662,8 +662,6 @@ func TestPageWaitForFunction(t *testing.T) {
 func TestPageWaitForLoadState(t *testing.T) {
 	t.Parallel()
 
-	// TODO: Add happy path tests once WaitForLoadState is not racy.
-
 	t.Run("err_wrong_event", func(t *testing.T) {
 		t.Parallel()