Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

e2e-test: add env var to indicate PW tests #6251

Merged
merged 2 commits into from
Feb 10, 2025
Merged

e2e-test: add env var to indicate PW tests #6251

merged 2 commits into from
Feb 10, 2025

Conversation

midleman
Copy link
Contributor

Summary

Adding process.env.PW_TEST to indicate whether Playwright tests are running. This variable is automatically set to "1" when Playwright executes tests, allowing conditional logic in the test environment.

QA Notes

No functional changes to the tests or application behavior. This is an internal enhancement to make environment detection more explicit.

@midleman midleman marked this pull request as ready for review February 10, 2025 11:44
Copy link

github-actions bot commented Feb 10, 2025

E2E Tests 🚀
This PR will run tests tagged with: @:critical

readme  valid tags

@midleman midleman changed the title e2e-test: add env var to indicated PW tests e2e-test: add env var to indicate PW tests Feb 10, 2025
Copy link
Contributor

@testlabauto testlabauto left a comment

Choose a reason for hiding this comment

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

LGTM!

@midleman midleman merged commit 2486039 into main Feb 10, 2025
9 checks passed
@midleman midleman deleted the mi/pw-test-env branch February 10, 2025 15:08
@github-actions github-actions bot locked and limited conversation to collaborators Feb 10, 2025
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants