Add Playwright e2e coverage + comment-level video upload workflow #40

Merged

shaun merged 7 commits from ai-agent/38-allow-administrators-to-create-local-login-users into main 2026-02-22 21:20:20 +00:00

Conversation 0 Commits 7 Files Changed 22 +993 -18

shaun commented 2026-02-22 21:17:09 +00:00

Owner

Copy Link

Summary

• add Playwright e2e auth setup + seeded test DB flow
• fix Playwright startup order so DB seed runs before web server uses SQLite
• add full trip-planning e2e scenario (future trip, travellers, destination, flight, lodging, upcoming verification)
• configure Auth.js custom sign-in page for base-path routing
• add repo guidance in AGENTS.md / CLAUDE.md requiring e2e for user-facing changes
• document and validate comment-level Gitea video attachment workflow (create comment, upload to comment assets endpoint)

Validation

• bunx playwright test e2e/auth.test.ts --project=chromium\n- bunx playwright test e2e/trip-planning.test.ts --project=chromium
• PW_VIDEO_MODE=on PW_TRACE_MODE=off bunx playwright test e2e/trip-planning.test.ts --project=chromium

Issue

• relates to Allow administrators to create local login users (#38)

## Summary - add Playwright e2e auth setup + seeded test DB flow - fix Playwright startup order so DB seed runs before web server uses SQLite - add full trip-planning e2e scenario (future trip, travellers, destination, flight, lodging, upcoming verification) - configure Auth.js custom sign-in page for base-path routing - add repo guidance in AGENTS.md / CLAUDE.md requiring e2e for user-facing changes - document and validate comment-level Gitea video attachment workflow (create comment, upload to comment assets endpoint) ## Validation - bunx playwright test e2e/auth.test.ts --project=chromium\n- bunx playwright test e2e/trip-planning.test.ts --project=chromium - PW_VIDEO_MODE=on PW_TRACE_MODE=off bunx playwright test e2e/trip-planning.test.ts --project=chromium ## Issue - relates to #38

shaun added 7 commits 2026-02-22 21:17:10 +00:00

admin) manage local login users 5c7bae61b5

fix(admin/users): convert Svelte 4 event syntax to Svelte 5 ... 345da2ef7a

Replace `on:keydown` with `onkeydown` and `onsubmit|preventDefault`
with inline handlers calling `e.preventDefault()` to fix compilation
errors caused by mixing old and new Svelte event syntax.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(login): pass callbackUrl so local auth redirects to app after sign-in ... 348aac996d

After a successful Credentials sign-in, Auth.js was redirecting to its
own built-in /auth/signin page because no callbackUrl was provided.
Add a callbackUrl hidden field (pointing to the app root) so Auth.js
redirects back to the application after local login completes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(login): post local credentials to /auth/callback/local not /signin/local ... 67648a5ba4

The Credentials provider in Auth.js processes credentials at the
/callback/:provider endpoint, not /signin/:provider. Posting to
/signin/local hits a code path that just redirects back to the Auth.js
built-in sign-in page. Changing to /callback/local routes the form
submission to the correct handler.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(admin/users): show all auth sources when a user has multiple credentials ... f4c5ee1f8f

listUsers() now LEFT JOINs local_credentials and returns has_local_credentials.
The auth source cell renders both the primary auth_source badge and a secondary
Local badge when an OIDC user also has local credentials set.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

test(login): update test to expect /callback/local and assert callbackUrl ... 4b06ecfa22

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

trips) add playwright e2e flows and issue video upload guidance 595b805722

shaun merged commit fe289f0895 into main 2026-02-22 21:20:20 +00:00

shaun deleted branch ai-agent/38-allow-administrators-to-create-local-login-users 2026-02-22 21:20:21 +00:00

shaun referenced this issue from a commit 2026-02-22 21:20:21 +00:00

Add Playwright e2e coverage + comment-level video upload workflow (#40)

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

ai-agent

Delegate this work to an AI Agent

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

ai-agent cicd shaun

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: campbellwireless/trips#40