ros/automaker
1
0
Fork 0
mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-01-30 06:12:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #281 from tony-nekola-silk/fix/flaky-context-tests

Browse Source 
fix: add retry mechanisms to context test helpers for flaky test stability
This commit is contained in:
Web Dev Cody
2025-12-27 15:50:24 -05:00
committed by GitHub
parent 4eae231166 b65fccbcf7
commit f7a0365bee
3 changed files with 27 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
apps/ui/tests/context/context-file-management.spec.ts
View File

@@ -50,7 +50,8 @@ test.describe('Context File Management', () => {
{ timeout: 5000 }
);
await waitForContextFile(page, 'test-context.md', 10000);
await waitForNetworkIdle(page);
await waitForContextFile(page, 'test-context.md');
const fileButton = await getByTestId(page, 'context-file-test-context.md');
await expect(fileButton).toBeVisible();

2
apps/ui/tests/context/delete-context-file.spec.ts
View File

@@ -53,7 +53,7 @@ test.describe('Delete Context File', () => {
);
// Wait for the file to appear in the list
await waitForContextFile(page, fileName, 10000);
await waitForContextFile(page, fileName);
// Select the file
await selectContextFile(page, fileName);

44
apps/ui/tests/utils/views/context.ts
View File

@@ -114,47 +114,51 @@ export async function toggleContextPreviewMode(page: Page): Promise<void> {
/**
* Wait for a specific file to appear in the context file list
* Uses retry mechanism to handle race conditions with API/UI updates
*/
export async function waitForContextFile(
page: Page,
filename: string,
timeout: number = 10000
timeout: number = 15000
): Promise<void> {
const locator = page.locator(`[data-testid="context-file-${filename}"]`);
await locator.waitFor({ state: 'visible', timeout });
await expect(async () => {
const locator = page.locator(`[data-testid="context-file-${filename}"]`);
await expect(locator).toBeVisible();
}).toPass({ timeout, intervals: [500, 1000, 2000] });
}
/**
* Click a file in the list and wait for it to be selected (toolbar visible)
* Uses JavaScript click to ensure React event handler fires
* Uses retry mechanism to handle race conditions where element is visible but not yet interactive
*/
export async function selectContextFile(
page: Page,
filename: string,
timeout: number = 10000
timeout: number = 15000
): Promise<void> {
const fileButton = await getByTestId(page, `context-file-${filename}`);
await fileButton.waitFor({ state: 'visible', timeout });
// Use JavaScript click to ensure React onClick handler fires
await fileButton.evaluate((el) => (el as HTMLButtonElement).click());
// Wait for the file to be selected (toolbar with delete button becomes visible)
const deleteButton = await getByTestId(page, 'delete-context-file');
await expect(deleteButton).toBeVisible({
timeout,
});
// Retry click + wait for delete button to handle timing issues
await expect(async () => {
// Use JavaScript click to ensure React onClick handler fires
await fileButton.evaluate((el) => (el as HTMLButtonElement).click());
// Wait for the file to be selected (toolbar with delete button becomes visible)
const deleteButton = await getByTestId(page, 'delete-context-file');
await expect(deleteButton).toBeVisible();
}).toPass({ timeout, intervals: [500, 1000, 2000] });
}
/**
* Wait for file content panel to load (either editor, preview, or image)
* Uses retry mechanism to handle race conditions with file selection
*/
export async function waitForFileContentToLoad(page: Page): Promise<void> {
// Wait for either the editor, preview, or image to appear
await page.waitForSelector(
'[data-testid="context-editor"], [data-testid="markdown-preview"], [data-testid="image-preview"]',
{ timeout: 10000 }
);
export async function waitForFileContentToLoad(page: Page, timeout: number = 15000): Promise<void> {
await expect(async () => {
const contentLocator = page.locator(
'[data-testid="context-editor"], [data-testid="markdown-preview"], [data-testid="image-preview"]'
);
await expect(contentLocator).toBeVisible();
}).toPass({ timeout, intervals: [500, 1000, 2000] });
}
/**