Skip to content

[Issue #11704] support e2e feature envs - #11804

Open
doug-s-nava wants to merge 2 commits into
mainfrom
dschrashun/11704-support-e2e-feature-envs
Open

[Issue #11704] support e2e feature envs#11804
doug-s-nava wants to merge 2 commits into
mainfrom
dschrashun/11704-support-e2e-feature-envs

Conversation

@doug-s-nava

@doug-s-nava doug-s-nava commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

Work for #11704

Changes proposed

Covers parts of the ticket not covered in #11800, adding support for e2e tests to run against grantee and grantor environments. Involves a few refactors to:

  • simplify environment based logic within tests
  • make sure that logic does not assume that any non-staging targets are local (switch polarity on if statements from === 'staging' to !== 'local' where applicable)

Note, depends on #11800, don't review til that has merged

Context for reviewers

Note that not everything is fully configured for tests to pass in these environments, see #11806

Validation steps

  1. VERIFY: testing run against grantee1 runs successfully (all tests don't need to pass, but need to ensure that the tests are properly targeting the environment), see https://github.com/HHS/simpler-grants-gov/actions/runs/30828478001

@doug-s-nava
doug-s-nava force-pushed the dschrashun/11704-support-e2e-feature-envs branch from e301a61 to 5a0ec1a Compare August 3, 2026 15:57
doug-s-nava added a commit that referenced this pull request Aug 4, 2026
…#11800)

Introduces `PLAYWRIGHT_API_URL` and `PLAYWRIGHT_BASE_URL` environment variables into the playwright system and CI to allow for directly setting these values without any hard coding. This change is needed to
support running tests from the mgmt repo after splitting code.

This piece of the work does not address the parts of the ticket oriented towards supporting grantee / grantor test environments. Those pieces are in #11804
@doug-s-nava
doug-s-nava force-pushed the dschrashun/11704-support-e2e-feature-envs branch from 5a0ec1a to 9a90dec Compare August 4, 2026 14:57
@doug-s-nava
doug-s-nava marked this pull request as ready for review August 4, 2026 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant