fix(e2e): use pnpm-capable workflow branch (PRI-634) #25

Closed
privilegedescalation-engineer[bot] wants to merge 6 commits from hugh/fix-argocd-final into main
privilegedescalation-engineer[bot] commented 2026-05-05 12:20:55 +00:00 (Migrated from github.com)

Summary

Updates E2E workflow to use @hugh/add-pnpm-support-plugin-e2e which has pnpm support via corepack and configurable e2e-namespace.

cc @cpfarhood

## Summary Updates E2E workflow to use `@hugh/add-pnpm-support-plugin-e2e` which has pnpm support via corepack and configurable `e2e-namespace`. cc @cpfarhood
privilegedescalation-engineer[bot] commented 2026-05-05 17:31:04 +00:00 (Migrated from github.com)

UAT Review: APPROVED

Review type: Code-review UAT (CTO directive — no browser UI component)

What changed: e2e.yaml workflow now calls plugin-e2e.yaml@hugh/add-pnpm-support-plugin-e2e (PR #144 branch) instead of @main. Auth setup simplified to token-only. Added basic.spec.ts smoke test.

UAT verification:

  • Workflow caller: uses pnpm-capable workflow branch
  • Auth setup: simplified token-based auth
  • Basic smoke test: verifies plugin page loads

Note: PR #25 depends on PR #144 (critical path). UAT approval contingent on #144 merge.

Acceptance criteria: ArgoCD plugin E2E uses pnpm-capable workflow.

## UAT Review: APPROVED **Review type:** Code-review UAT (CTO directive — no browser UI component) **What changed:** `e2e.yaml` workflow now calls `plugin-e2e.yaml@hugh/add-pnpm-support-plugin-e2e` (PR #144 branch) instead of `@main`. Auth setup simplified to token-only. Added basic.spec.ts smoke test. **UAT verification:** - Workflow caller: uses pnpm-capable workflow branch ✅ - Auth setup: simplified token-based auth ✅ - Basic smoke test: verifies plugin page loads ✅ **Note:** PR #25 depends on PR #144 (critical path). UAT approval contingent on #144 merge. **Acceptance criteria:** ArgoCD plugin E2E uses pnpm-capable workflow. ✅
privilegedescalation-engineer[bot] commented 2026-05-05 19:13:37 +00:00 (Migrated from github.com)

Closing — superseded by #29 (canonical E2E consolidation PR). E2E infra changes have been consolidated into a single PR per repo per PRI-779.

Closing — superseded by #29 (canonical E2E consolidation PR). E2E infra changes have been consolidated into a single PR per repo per PRI-779.

Pull request closed

Sign in to join this conversation.