Skip to content

docs(e2e): record why roadie-backstage-plugins has no app-next lane #3327

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Closed
gustavolira wants to merge 1 commit into from
Closed

docs(e2e): record why roadie-backstage-plugins has no app-next lane #3327

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/nfs-e2e-triage.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ lane means anything.
| `acr` | 2 | 1 | ✅ | guest | **svc** | real ACR (`rhdhqetest.azurecr.io`) + a GitHub-hosted entity | oci | 0 | ready |
| `quay` | 1 | 3 | — | guest | **svc** | real quay.io repo + security scan | oci | 0 | ready |
| `github` | 2 | 2 | — | github | **svc** | real GitHub Actions runs + issues | baked-in | 0 | ready (5 pkgs) |
| `roadie-backstage-plugins` | 2 | 6 | — | github, guest | **svc** | real GitHub PR data; outbound HTTP | oci | 0 | **all 6 no NFS entry point** (3 do expose `alpha`) |
| `roadie-backstage-plugins` | 2 | 6 | **blocked** | github, guest | **svc** | real GitHub PR data; outbound HTTP | oci | 0 | **all 6 no NFS entry point**; the e2e-tested `github-pull-requests` has an `alpha.tsx` but registers only an `ApiBlueprint`, no UI extension — see RHIDP-16301 |
| `bulk-import` | 2 | 9 | — | github | **svc** | real GitHub repos + generated PRs | baked-in | 17 | ready |
| `quickstart` | 1 | 2 | — | keycloak | **ctr** | Keycloak (test 1 is guest-only) | oci | 0 | ready |
| `global-header` | 2 | 10 | — | keycloak | **ctr** | Keycloak | mixed | 22 | ready |
Expand Down
27 changes: 27 additions & 0 deletions workspaces/roadie-backstage-plugins/e2e-tests/playwright.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,32 @@
import { defineConfig } from "@red-hat-developer-hub/e2e-test-utils/playwright-config";

/**
* Roadie plugins e2e test configuration.
*
* There is deliberately no `-app-next` project here. Both lanes are blocked upstream,
* for two different reasons, and both were verified at the ref this workspace pins
* (`source.json` → RoadieHQ/roadie-backstage-plugins@c1b84e94) as well as on upstream
* `main`, 62 commits ahead, where the relevant file is unchanged:
*
* 1. `backstage-plugin-github-pull-requests` — the package does have an NFS surface
* (`src/alpha.tsx`, `createFrontendPlugin`), but it registers
* `extensions: [githubPullRequestsApi]` — an `ApiBlueprint` and nothing else. No
* `EntityContentBlueprint`, so the "Pull/Merge Requests" entity tab this spec clicks
* cannot exist under the new frontend system. The NFS surface contributes an API, no
* UI. The package also declares no `backstage.features` and has no `exports` map.
*
* 2. `scaffolder-backend-module-http-request` — a backend module, so NFS does not apply
* to it at all. Its spec asserts `verifyHeading("Self-service")`, which is a global
* header element, and `rhdh:packages/app-next` ships no global header.
*
* Adding a lane would produce a suite that fails on a missing tab, and the failure would
* read as a wiring bug rather than as the absence of an upstream extension. Per the
* support-level rules in rhdh:docs/testing-requirements-matrix.md these packages are
* `community`, which requires no E2E at all — so the fix is upstream contribution or
* nothing, not a lane here.
*
* Tracked in RHIDP-16301.
*/
export default defineConfig({
projects: [
{
Expand Down
Loading