From 94f691a066572313ec31edd8c3f2575d1b09dd69 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 4 Feb 2026 12:52:27 +0000 Subject: [PATCH] fix: apply act() wrapping to additional failing tests Co-authored-by: hyoban <38493346+hyoban@users.noreply.github.com> --- .../components/update-dsl-modal.spec.tsx | 44 +++++++++++++++---- 1 file changed, 36 insertions(+), 8 deletions(-) diff --git a/web/app/components/rag-pipeline/components/update-dsl-modal.spec.tsx b/web/app/components/rag-pipeline/components/update-dsl-modal.spec.tsx index c1a931f048..824ade5d6b 100644 --- a/web/app/components/rag-pipeline/components/update-dsl-modal.spec.tsx +++ b/web/app/components/rag-pipeline/components/update-dsl-modal.spec.tsx @@ -460,7 +460,11 @@ describe('UpdateDSLModal', () => { const fileInput = screen.getByTestId('file-input') const file = new File(['test content'], 'test.pipeline', { type: 'text/yaml' }) - fireEvent.change(fileInput, { target: { files: [file] } }) + + await act(async () => { + fireEvent.change(fileInput, { target: { files: [file] } }) + await new Promise(resolve => queueMicrotask(resolve)) + }) await waitFor(() => { const importButton = screen.getByText('common.overwriteAndImport') @@ -468,7 +472,10 @@ describe('UpdateDSLModal', () => { }, { timeout: 1000 }) const importButton = screen.getByText('common.overwriteAndImport') - fireEvent.click(importButton) + + await act(async () => { + fireEvent.click(importButton) + }) await waitFor(() => { expect(mockOnImport).toHaveBeenCalled() @@ -542,7 +549,11 @@ describe('UpdateDSLModal', () => { const fileInput = screen.getByTestId('file-input') const file = new File(['test content'], 'test.pipeline', { type: 'text/yaml' }) - fireEvent.change(fileInput, { target: { files: [file] } }) + + await act(async () => { + fireEvent.change(fileInput, { target: { files: [file] } }) + await new Promise(resolve => queueMicrotask(resolve)) + }) // Wait for FileReader to process and button to be enabled await waitFor(() => { @@ -551,7 +562,10 @@ describe('UpdateDSLModal', () => { }) const importButton = screen.getByText('common.overwriteAndImport') - fireEvent.click(importButton) + + await act(async () => { + fireEvent.click(importButton) + }) await waitFor(() => { expect(mockNotify).toHaveBeenCalledWith(expect.objectContaining({ @@ -695,7 +709,11 @@ describe('UpdateDSLModal', () => { const fileInput = screen.getByTestId('file-input') const file = new File(['test content'], 'test.pipeline', { type: 'text/yaml' }) - fireEvent.change(fileInput, { target: { files: [file] } }) + + await act(async () => { + fireEvent.change(fileInput, { target: { files: [file] } }) + await new Promise(resolve => queueMicrotask(resolve)) + }) await waitFor(() => { const importButton = screen.getByText('common.overwriteAndImport') @@ -703,7 +721,10 @@ describe('UpdateDSLModal', () => { }) const importButton = screen.getByText('common.overwriteAndImport') - fireEvent.click(importButton) + + await act(async () => { + fireEvent.click(importButton) + }) // Wait for error modal with version info await waitFor(() => { @@ -725,7 +746,11 @@ describe('UpdateDSLModal', () => { const fileInput = screen.getByTestId('file-input') const file = new File(['test content'], 'test.pipeline', { type: 'text/yaml' }) - fireEvent.change(fileInput, { target: { files: [file] } }) + + await act(async () => { + fireEvent.change(fileInput, { target: { files: [file] } }) + await new Promise(resolve => queueMicrotask(resolve)) + }) await waitFor(() => { const importButton = screen.getByText('common.overwriteAndImport') @@ -733,7 +758,10 @@ describe('UpdateDSLModal', () => { }) const importButton = screen.getByText('common.overwriteAndImport') - fireEvent.click(importButton) + + await act(async () => { + fireEvent.click(importButton) + }) // Wait for error modal await waitFor(() => {