Skip to content

chore(e2e): Add workflow for running admin ui tests#6525

Open
moduli wants to merge 2 commits intomainfrom
moduli-e2e-admin-ui
Open

chore(e2e): Add workflow for running admin ui tests#6525
moduli wants to merge 2 commits intomainfrom
moduli-e2e-admin-ui

Conversation

@moduli
Copy link
Copy Markdown
Collaborator

@moduli moduli commented Mar 20, 2026

Description

This PR adds a workflow to run Admin UI e2e tests against the binary built in CI. This is to assist with release testing.

Here's an example run: https://github.com/hashicorp/boundary/actions/runs/23872207204/job/69607768896 (I modified the trigger rules in this run to trigger on PR changes, but the workflow is intended to only trigger on release branches).

https://hashicorp.atlassian.net/browse/ICU-18378

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've documented the impact of any changes to security controls.
    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

@moduli moduli added the pr/no-milestone Ignores the Milestone Check label Mar 20, 2026
@moduli moduli force-pushed the moduli-e2e-admin-ui branch 10 times, most recently from c91b295 to c5ab4b0 Compare March 23, 2026 14:27
@moduli moduli force-pushed the moduli-e2e-admin-ui branch from 45f0e5f to cfeb66c Compare March 25, 2026 16:12
Comment thread internal/ui/VERSION
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The UI Version needed to be updated to include some e2e test changes in boundary-ui

@moduli moduli force-pushed the moduli-e2e-admin-ui branch 2 times, most recently from ab61abb to 086dce9 Compare April 2, 2026 15:39
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a reusable GitHub Actions workflow to provision Enos UI test infrastructure and run Boundary Admin UI Playwright e2e tests against CI-built artifacts (intended for release-branch validation).

Changes:

  • Introduces a new reusable workflow (enos-run-admin-ui.yml) that launches Enos UI scenarios and runs Admin UI e2e tests from the boundary-ui repo at the embedded UI SHA.
  • Wires the new workflow into the main build.yml pipeline for release branches.
  • Refactors Enos UI e2e scenario/module plumbing to stop running UI tests inside the Terraform module and instead use it to expose environment variables for the external test runner.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
internal/ui/VERSION Updates the embedded UI commit SHA used by Boundary.
enos/modules/test_e2e_ui/main.tf Removes local test execution wiring; module now effectively only publishes env vars via state.
enos/enos-scenario-e2e-ui-docker.hcl Drops now-unused variables previously required to run UI tests from local paths.
enos/enos-scenario-e2e-ui-aws.hcl Drops now-unused variables previously required to run UI tests from local paths.
.github/workflows/enos-run-admin-ui.yml Adds reusable workflow to provision infra, checkout boundary-ui, and run Playwright Admin UI e2e tests.
.github/workflows/build.yml Adds e2e-admin-ui job that calls the new reusable workflow on release branches.
Comments suppressed due to low confidence (1)

enos/modules/test_e2e_ui/main.tf:244

  • enos_local_exec.run_e2e_test is now hardcoded to run an empty inline command, which means this module no longer executes the UI e2e tests and test_results will always be empty. If the intent is to move test execution fully into GitHub Actions, consider renaming/adjusting the module output (or adding a flag) so consumers don’t assume this step actually runs tests.
  inline = [""]
}

output "test_results" {
  value = enos_local_exec.run_e2e_test.stdout
}

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/enos-run-admin-ui.yml Outdated
Comment thread .github/workflows/enos-run-admin-ui.yml
Comment thread .github/workflows/enos-run-admin-ui.yml
Comment thread .github/workflows/enos-run-admin-ui.yml
@moduli moduli force-pushed the moduli-e2e-admin-ui branch 2 times, most recently from d9e0c25 to 6976fa6 Compare April 2, 2026 18:04
@moduli moduli marked this pull request as ready for review April 2, 2026 18:04
@moduli moduli requested a review from a team as a code owner April 2, 2026 18:04
@moduli moduli requested review from a team and ZedLi April 2, 2026 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants