Skip to content

Commit a6ad745

Browse files
committed
Fix some tests #1095
1 parent b4d6e13 commit a6ad745

File tree

2 files changed

+5
-14
lines changed

2 files changed

+5
-14
lines changed

browser/e2e/tests/filePicker.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -173,8 +173,8 @@ test.describe('File Picker', () => {
173173

174174
await page.getByRole('button', { name: 'Save' }).click();
175175
await expect(page.getByText('New robot')).not.toBeVisible();
176-
await expect(page.getByText('testFile3.txt').nth(1)).toBeVisible();
177-
await page.getByText('testFile3.txt').nth(1).click();
176+
await expect(page.getByText('testFile3.txt').nth(0)).toBeVisible();
177+
await page.getByText('testFile3.txt').nth(0).click();
178178

179179
// For some reason playwright will only find text with quotes in them when using a regex instead of string.
180180
await expect(page.getByText(/It's a secret to everybody/)).toBeVisible();

browser/e2e/tests/test-utils.ts

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,6 @@ export const testFilePath = (filename: string) => {
1313
const fixturesFolder = __dirname + '/fixtures';
1414

1515
return `${fixturesFolder}/${filename}`;
16-
17-
// const processPath = process.cwd();
18-
// if (processPath.endsWith('tests')) {
19-
// return `${processPath}/fixtures/${filename}`;
20-
// } else if (processPath.endsWith('e2e')) {
21-
// return `${processPath}/tests/fixtures/${filename}`;
22-
// } else if (processPath.endsWith('browser')) {
23-
// return `${processPath}/e2e/tests/fixtures/${filename}`;
24-
// } else {
25-
// return `${processPath}/browser/e2e/tests/fixtures/${filename}`;
26-
// }
2716
};
2817

2918
export const timestamp = () => new Date().toLocaleTimeString();
@@ -199,7 +188,9 @@ export async function openAtomic(page: Page) {
199188
export async function editProfileAndCommit(page: Page) {
200189
await openAgentPage(page);
201190
await page.click('text=Edit profile');
202-
await page.getByRole('button', { name: 'Advanced' }).click();
191+
const advancedButton = await page.getByRole('button', { name: 'advanced' });
192+
await advancedButton.scrollIntoViewIfNeeded();
193+
await advancedButton.click();
203194
await expect(page.locator('text=add another property')).toBeVisible();
204195
const username = `Test user edited at ${new Date().toLocaleDateString()}`;
205196
await page.getByLabel('Name').fill(username);

0 commit comments

Comments
 (0)