Skip to content

Commit 5fc4dde

Browse files
test: refactor specs, add ci shard for chromium.
1 parent 762eb4f commit 5fc4dde

13 files changed

Lines changed: 7237 additions & 7022 deletions

.github/workflows/playwright.yml

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,15 @@ jobs:
3434
matrix:
3535
include:
3636
- browser: chromium
37-
jobName: E2E (Playwright, chromium)
38-
workers: 2
37+
jobName: E2E (Playwright, chromium, shard 1/2)
38+
workers: 1
3939
shardIndex: 1
40-
shardTotal: 1
40+
shardTotal: 2
41+
- browser: chromium
42+
jobName: E2E (Playwright, chromium, shard 2/2)
43+
workers: 1
44+
shardIndex: 2
45+
shardTotal: 2
4146
- browser: webkit
4247
jobName: E2E (Playwright, webkit, shard 1/4)
4348
workers: 1

playwright/github-byot-ai.spec.ts

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -768,9 +768,21 @@ test('BYOT remembers selected repository across reloads', async ({ page }) => {
768768

769769
await page.getByRole('button', { name: 'Workspaces' }).click()
770770
const workspaceRepositoryFilter = page.getByLabel('Workspace repository filter')
771+
const storedContextsSelect = page.getByLabel('Stored local editor contexts')
772+
const openStoredContextButton = page.getByRole('button', {
773+
name: 'Open',
774+
exact: true,
775+
})
771776
await expect(workspaceRepositoryFilter).toBeVisible()
772777
await workspaceRepositoryFilter.selectOption('knightedcodemonkey/develop')
773778
await expect(workspaceRepositoryFilter).toHaveValue('knightedcodemonkey/develop')
779+
780+
await expect(storedContextsSelect).toBeVisible()
781+
await storedContextsSelect.selectOption({
782+
label: 'Start new context for knightedcodemonkey/develop',
783+
})
784+
await expect(openStoredContextButton).toBeEnabled()
785+
await openStoredContextButton.click()
774786
await page.getByRole('button', { name: 'Close workspaces drawer' }).click()
775787

776788
await ensureOpenPrDrawerOpen(page)

0 commit comments

Comments
 (0)