From 91caf7b08ee02a70940320aedf27c1580ead2324 Mon Sep 17 00:00:00 2001
From: Spencer Hamm <spencer_hamm@intuit.com>
Date: Tue, 3 Feb 2026 23:49:43 -0600
Subject: [PATCH 1/2] cleanup tests and remove async act

---
 .../manager/__tests__/managed-player.test.tsx | 59 ++++++++++---------
 1 file changed, 31 insertions(+), 28 deletions(-)

diff --git a/react/player/src/manager/__tests__/managed-player.test.tsx b/react/player/src/manager/__tests__/managed-player.test.tsx
index b35d9b170..b8a6926f7 100644
--- a/react/player/src/manager/__tests__/managed-player.test.tsx
+++ b/react/player/src/manager/__tests__/managed-player.test.tsx
@@ -83,6 +83,7 @@ describe.each([
 
     const onComplete = vitest.fn();
     const onError = vitest.fn();
+    const onStartedFlow = vitest.fn();
 
     const container = render(
       <Suspense fallback="loading">
@@ -91,6 +92,7 @@ describe.each([
           plugins={[new SimpleAssetPlugin(), new MetricsCorePlugin()]}
           onComplete={onComplete}
           onError={onError}
+          onStartedFlow={onStartedFlow}
         />
       </Suspense>,
       { legacyRoot },
@@ -98,22 +100,26 @@ describe.each([
 
     expect(manager.next).toBeCalledWith(undefined);
     const view = await container.findByTestId("flow-1");
+    expect(onStartedFlow).toBeCalledTimes(1);
     expect(view).toBeInTheDocument();
+    const nextButton = await container.findByText("Continue");
 
-    await act(async () => {
-      const nextButton = await container.findByText("Continue");
+    act(() => {
       nextButton.click();
     });
 
-    expect(manager.next).toBeCalledTimes(2);
-
     const view2 = await container.findByTestId("flow-2");
     expect(view2).toBeInTheDocument();
+    expect(manager.next).toBeCalledTimes(2);
+    expect(onStartedFlow).toBeCalledTimes(2);
 
-    await act(async () => {
-      const nextButton = await container.findByText("Continue");
-      nextButton.click();
+    const nextButton2 = await container.findByText("Continue");
+
+    act(() => {
+      nextButton2.click();
     });
+
+    await container.findByText("loading");
     const getRequestTime = (RequestTimeWebPlugin as any).mock.calls[0][0];
     expect(getRequestTime()).toBeDefined();
     expect(onComplete).toBeCalled();
@@ -183,21 +189,23 @@ describe.each([
     expect(manager.next).toBeCalledWith(undefined);
     const view = await screen.findByTestId("flow-1");
     expect(view).toBeInTheDocument();
+    const nextButton = await screen.findByText("Continue");
 
-    await act(async () => {
-      const nextButton = await screen.findByText("Continue");
+    act(() => {
       nextButton.click();
     });
 
-    expect(manager.next).toBeCalledTimes(2);
-
     const view2 = await screen.findByTestId("flow-2");
     expect(view2).toBeInTheDocument();
+    expect(manager.next).toBeCalledTimes(2);
 
-    await act(async () => {
-      const nextButton = await screen.findByText("Continue");
-      nextButton.click();
+    const nextButton2 = await screen.findByText("Continue");
+
+    act(() => {
+      nextButton2.click();
     });
+
+    await screen.findByText("loading");
     expect(onComplete).toBeCalledWith(
       expect.objectContaining({
         status: "completed",
@@ -240,9 +248,7 @@ describe.each([
 
     const onComplete = vitest.fn();
     const onError = vitest.fn();
-    /**
-     *
-     */
+
     const MyFallback = (props: FallbackProps) => (
       <div>
         <button type="button" onClick={props?.retry}>
@@ -336,9 +342,6 @@ describe.each([
     const onComplete = vitest.fn();
     const onError = vitest.fn();
 
-    /**
-     *
-     */
     const MyFallback = (props: FallbackProps) => (
       <div>
         <button type="button" onClick={props?.retry}>
@@ -366,25 +369,24 @@ describe.each([
     expect(manager.next).toBeCalledWith(undefined);
     const view = await screen.findByTestId("flow-1");
     expect(view).toBeInTheDocument();
+    const nextButton = await screen.findByText("Continue");
 
-    await act(async () => {
-      const nextButton = await screen.findByText("Continue");
+    act(() => {
       nextButton.click();
     });
 
-    await waitFor(() => expect(manager.next).toBeCalledTimes(2));
-
     const view2 = await screen.findByTestId("flow-2");
     expect(view2).toBeInTheDocument();
+    expect(manager.next).toBeCalledTimes(2);
 
-    await act(async () => {
+    act(() => {
       view2.click();
     });
 
     const retryButton = await screen.findByText("Retry");
     expect(retryButton).toBeInTheDocument();
 
-    await act(async () => {
+    act(() => {
       retryButton.click();
     });
 
@@ -393,17 +395,18 @@ describe.each([
     const view3 = await screen.findByTestId("flow-2");
     expect(view3).toBeInTheDocument();
 
-    await act(async () => {
+    act(() => {
       view3.click();
     });
 
     const resetButton = await screen.findByText("Retry");
     expect(resetButton).toBeInTheDocument();
 
-    await act(async () => {
+    act(() => {
       resetButton.click();
     });
 
+    await screen.findByText("loading");
     expect(manager.next).toBeCalledTimes(4);
     expect(onError).toBeCalledTimes(2);
 

From 6d78f03eb23e92b97bb97768b18a981377d5b4d8 Mon Sep 17 00:00:00 2001
From: Spencer Hamm <spencer_hamm@intuit.com>
Date: Thu, 19 Feb 2026 12:18:38 -0600
Subject: [PATCH 2/2] fix lint issue

---
 .../player/src/manager/__tests__/managed-player.test.tsx  | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/react/player/src/manager/__tests__/managed-player.test.tsx b/react/player/src/manager/__tests__/managed-player.test.tsx
index b8a6926f7..f6b5c6f97 100644
--- a/react/player/src/manager/__tests__/managed-player.test.tsx
+++ b/react/player/src/manager/__tests__/managed-player.test.tsx
@@ -1,13 +1,7 @@
 import { test, expect, vitest, describe, beforeEach } from "vitest";
 import React, { Suspense } from "react";
 import { makeFlow } from "@player-ui/make-flow";
-import {
-  render,
-  act,
-  configure,
-  waitFor,
-  screen,
-} from "@testing-library/react";
+import { render, act, configure, screen } from "@testing-library/react";
 import { userEvent } from "@testing-library/user-event";
 import {
   MetricsCorePlugin,