fix(#3694): bulk-import E2E NFS sidebar nav and orchestrator skip

[fullsend-ai-coder]

Replace uiHelper.openSidebar("Catalog") with page.goto("/catalog") in the "Verify Added Repositories" test to avoid NFS global header overlay interception in bulk-import-app-next. Remove redundant openSidebar("Catalog") before page.goto in "Catalog-imported repo" test. Add test.skip guard for nightly mode in orchestrator "should interact with plugin features" test for product bug (TypeError at router.cjs.js:616).

Re-applies validated fix from PR #3337, closed without merge due to pipeline issues.

---

Closes #3694

Post-script verification

• Branch is not main/master (agent/3694-bulk-import-e2e-fixes)
• Secret scan passed (gitleaks — 867ccab30b5496862b752cb772d6a814f3216109..HEAD)
• Pre-commit hooks passed (authoritative run on runner)
• Tests ran inside sandbox

[openshift-ci]

Hi @fullsend-ai-coder[bot]. Thanks for your PR.

I'm waiting for a redhat-developer member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[fullsend-ai-review]

🤖 Finished Review · ✅ Success · Started 11:55 AM UTC · Completed 12:05 PM UTC

Commit: cd48aee · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Cost: $2.01

[fullsend-ai-review]

Review

Findings

Low

• [naming-conventions] workspaces/bulk-import/e2e-tests/tests/specs/bulk-import.spec.ts:238 — The new page.goto("/catalog") call introduces the only hardcoded route string among all page.goto() calls in the bulk-import e2e test suite. Every other page.goto() call uses a named constant or helper function (CATALOG_IMPORT_ROUTE, BULK_IMPORT_ROUTE, catalogDefaultComponentPath(), bulkImportImportHistoryPath()). However, the code it replaces (uiHelper.openSidebar("Catalog")) also used a hardcoded string, and /catalog is a core Backstage route unlikely to change.
  Remediation: Define a CATALOG_ROUTE constant in workspaces/bulk-import/e2e-tests/support/constants/catalog.ts (alongside the existing CATALOG_IMPORT_ROUTE) and import/use it here.

---

Labels: PR modifies E2E test files in an existing workspace to fix test failures.

[fullsend-ai-review]

[low] naming-conventions

The new page.goto("/catalog") call introduces the only hardcoded route string among all page.goto() calls in the bulk-import e2e test suite. Every other page.goto() call uses a named constant or helper function (CATALOG_IMPORT_ROUTE, BULK_IMPORT_ROUTE, catalogDefaultComponentPath(), bulkImportImportHistoryPath()). However, the code it replaces (uiHelper.openSidebar("Catalog")) also used a hardcoded string, and /catalog is a core Backstage route unlikely to change.

Suggested fix: Define a CATALOG_ROUTE constant in workspaces/bulk-import/e2e-tests/support/constants/catalog.ts (alongside the existing CATALOG_IMPORT_ROUTE) and import/use it here.

[openshift-ci]

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.