fix(e2e): remove page.reload() causing orchestrator RBAC test timeouts (#4494)

The page.reload() before goToPageUrl("/orchestrator") creates a race
condition: the reload reloads the current page (which may be a workflow
detail page or dashboard from a previous test), and the subsequent
navigation can fail to complete, causing the "Workflows" heading
to never appear and the test to timeout after 20s.

Since goToPageUrl() calls page.goto() which navigates directly to the
URL, the reload is unnecessary and harmful. Removing it from all 7
occurrences in the test file.

Ref: https://issues.redhat.com/browse/RHDHBUGS-2671

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 2 people

e2e-tests/playwright/e2e/plugins/orchestrator/orchestrator-rbac.spec.ts

@@ -115,7 +115,6 @@ test.describe.serial("Test Orchestrator RBAC", () => {
     });
 
     test("Test global orchestrator workflow access is allowed", async () => {
-      await page.reload();
       await uiHelper.goToPageUrl("/orchestrator");
       await uiHelper.verifyHeading("Workflows");
 
@@ -264,7 +263,6 @@ test.describe.serial("Test Orchestrator RBAC", () => {
     });
 
     test("Test global orchestrator workflow read-only access - Run button disabled", async () => {
-      await page.reload();
       await uiHelper.goToPageUrl("/orchestrator");
       await uiHelper.verifyHeading("Workflows");
 
@@ -421,7 +419,6 @@ test.describe.serial("Test Orchestrator RBAC", () => {
     });
 
     test("Test global orchestrator workflow denied access - no workflows visible", async () => {
-      await page.reload();
       await uiHelper.goToPageUrl("/orchestrator");
       await uiHelper.verifyHeading("Workflows");
 
@@ -559,7 +556,6 @@ test.describe.serial("Test Orchestrator RBAC", () => {
     });
 
     test("Test individual workflow denied access - no workflows visible", async () => {
-      await page.reload();
       await uiHelper.goToPageUrl("/orchestrator");
       await uiHelper.verifyHeading("Workflows");
 
@@ -705,7 +701,6 @@ test.describe.serial("Test Orchestrator RBAC", () => {
     });
 
     test("Test individual workflow read-write access - only Greeting workflow visible and runnable", async () => {
-      await page.reload();
       await uiHelper.goToPageUrl("/orchestrator");
       await uiHelper.verifyHeading("Workflows");
 
@@ -859,7 +854,6 @@ test.describe.serial("Test Orchestrator RBAC", () => {
     });
 
     test("Test individual workflow read-only access - only Greeting workflow visible, Run button disabled", async () => {
-      await page.reload();
       await uiHelper.goToPageUrl("/orchestrator");
       await uiHelper.verifyHeading("Workflows");
 
@@ -1135,7 +1129,6 @@ test.describe.serial("Test Orchestrator RBAC", () => {
     });
 
     test("rhdh-qe user runs greeting workflow and captures instance ID", async () => {
-      await page.reload();
       await uiHelper.goToPageUrl("/orchestrator");
       await uiHelper.verifyHeading("Workflows");