Skip to content

Strengthen Playwright auth path handling#11579

Closed
mikeldking wants to merge 2 commits intomainfrom
mikeldking/make-playwright-scripts-directory-ag
Closed

Strengthen Playwright auth path handling#11579
mikeldking wants to merge 2 commits intomainfrom
mikeldking/make-playwright-scripts-directory-ag

Conversation

@mikeldking
Copy link
Collaborator

Summary

  • centralize the Playwright auth constants in app/tests/constants.ts and expose uppercase snake case paths so all scripts reuse the same values
  • make the Playwright config aware of its true directory, resolve the tests dir and storage state from that location, and run the dev server command via pnpm --dir so tests can execute from anywhere
  • update auth.setup.ts, member-access.spec.ts, and viewer-access.spec.ts to import the shared constants instead of hard-coding paths

Testing

  • Not run (not requested)

@mikeldking mikeldking requested a review from a team as a code owner February 19, 2026 00:37
@github-project-automation github-project-automation bot moved this to 📘 Todo in phoenix Feb 19, 2026
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Feb 19, 2026
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@github-project-automation github-project-automation bot moved this from 📘 Todo to 👍 Approved in phoenix Feb 19, 2026
@mikeldking mikeldking closed this Feb 27, 2026
@github-project-automation github-project-automation bot moved this from 👍 Approved to ✅ Done in phoenix Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants