chihlasm/resolutionflow
1
0
Fork 0
Code Issues 23 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: align CI tests with current UI text and database naming

Browse Source 
Backend conftest.py was hardcoded to patherly_test but CI creates
resolutionflow_test — now reads DATABASE_URL env var first. E2E tests
had stale placeholder text, heading, and landing page assertions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
chihlasm
2026-03-27 17:02:19 +00:00
parent 9150972de9
commit 9061c91c89
5 changed files with 14 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
backend/tests/conftest.py
View File

@@ -24,8 +24,11 @@ settings.REQUIRE_INVITE_CODE = False
# otherwise fall back to localhost for local development.
import os
TEST_DATABASE_URL = os.environ.get(
"DATABASE_TEST_URL",
"postgresql+asyncpg://postgres:postgres@localhost:5432/patherly_test",
"DATABASE_URL",
os.environ.get(
"DATABASE_TEST_URL",
"postgresql+asyncpg://postgres:postgres@localhost:5432/patherly_test",
),
)

10
frontend/e2e/command-palette.spec.ts
View File

@@ -15,7 +15,7 @@ test.describe('command palette smoke tests', () => {
await page.keyboard.press('Control+k')
// Should show the palette modal with search input
await expect(page.getByPlaceholder('Search flows, ask a question, navigate')).toBeVisible({ timeout: 3000 })
await expect(page.getByPlaceholder('Search flows, sessions, tags... or describe an issue to troubleshoot')).toBeVisible({ timeout: 3000 })
// Empty state should show quick actions — the palette label renders uppercase via CSS
// Use the palette container to scope the check
@@ -24,7 +24,7 @@ test.describe('command palette smoke tests', () => {
// Close with Escape
await page.keyboard.press('Escape')
await expect(page.getByPlaceholder('Search flows, ask a question, navigate')).not.toBeVisible()
await expect(page.getByPlaceholder('Search flows, sessions, tags... or describe an issue to troubleshoot')).not.toBeVisible()
})
test('searches and shows AI Assistant option', async ({ page }) => {
@@ -39,7 +39,7 @@ test.describe('command palette smoke tests', () => {
await page.keyboard.press('Control+k')
const input = page.getByPlaceholder('Search flows, ask a question, navigate')
const input = page.getByPlaceholder('Search flows, sessions, tags... or describe an issue to troubleshoot')
await expect(input).toBeVisible()
await input.fill('PW Palette Search')
@@ -57,7 +57,7 @@ test.describe('command palette smoke tests', () => {
await page.keyboard.press('Control+k')
const input = page.getByPlaceholder('Search flows, ask a question, navigate')
const input = page.getByPlaceholder('Search flows, sessions, tags... or describe an issue to troubleshoot')
await expect(input).toBeVisible()
await input.fill('analytics')
@@ -77,7 +77,7 @@ test.describe('command palette smoke tests', () => {
await page.keyboard.press('Control+k')
const input = page.getByPlaceholder('Search flows, ask a question, navigate')
const input = page.getByPlaceholder('Search flows, sessions, tags... or describe an issue to troubleshoot')
await expect(input).toBeVisible()
await input.fill('how do I fix a print spooler issue')

2
frontend/e2e/history.spec.ts
View File

@@ -24,7 +24,7 @@ test.describe('session history smoke tests', () => {
await page.goto('/sessions')
await expect(
page.getByRole('heading', { name: 'Session History' }),
page.getByRole('heading', { name: 'Sessions' }),
).toBeVisible()
await page.getByPlaceholder('Search by ticket number...').fill(ticketNumber)

2
frontend/e2e/navigation.spec.ts
View File

@@ -14,7 +14,7 @@ test.describe('authenticated navigation smoke tests', () => {
await page.goto('/sessions')
await expect(
page.getByRole('heading', { name: 'Session History' }),
page.getByRole('heading', { name: 'Sessions' }),
).toBeVisible()
})

4
frontend/e2e/public.spec.ts
View File

@@ -7,10 +7,10 @@ test.describe('public route smoke tests', () => {
await page.goto('/landing')
await expect(
page.getByRole('link', { name: 'Get Started Free' }),
page.getByRole('link', { name: 'Start Free' }),
).toBeVisible()
await expect(
page.getByText('Stop writing ticket notes.'),
page.getByText('Resolve tickets faster.'),
).toBeVisible()
})