chore(ci): remove unused Xvfb from E2E test runner [release-1.10]

[zdrapela]

Summary

• Cherry-pick of chore(ci): remove unused Xvfb from E2E test runner #5188 for release-1.10
• Remove unused Xvfb/DISPLAY from testing::run_tests (testing.sh only)

Jira: RHIDP-15894

Test plan

• Release-branch e2e job reaches Playwright without Missing X server / DISPLAY errors
• Job log has no _XSERVTransmkdir / xkbcomp XF86* warnings

[rhdh-qodo-merge]

PR Summary by Qodo

CI: drop unused Xvfb/DISPLAY setup from E2E Playwright runner (release-1.10)

⚙️ Configuration changes 🕐 Less than 10 minutes

AI Description

• Remove Xvfb startup/shutdown and DISPLAY export from the E2E test runner.
• Keep Playwright running headless in CI while reducing noisy X server warnings.

Diagram

graph TD
  A["CI E2E job"] --> B["testing.sh"] --> C["Playwright (headless)"] --> D["Test results artifacts"]

  subgraph Legend
    direction LR
    _job["CI job"] ~~~ _script["Script"] ~~~ _tool["Test tool"] ~~~ _art["Artifacts"]
  end

Loading

High-Level Assessment

Current approach is optimal: since Playwright runs headless in CI, removing Xvfb/DISPLAY reduces log noise without changing test behavior. A conditional Xvfb setup (only when headed mode is enabled) was considered but adds complexity not needed for this pipeline.

Files changed (1) +0 / -5

Other (1) +0 / -5

testing.shRemove Xvfb/DISPLAY setup and teardown from testing::run_tests +0/-5

Remove Xvfb/DISPLAY setup and teardown from testing::run_tests

• Deletes the background Xvfb start and DISPLAY export before running Playwright, and removes the corresponding pkill cleanup. This keeps the E2E runner aligned with headless Playwright execution and avoids X server-related log warnings.

.ci/pipelines/lib/testing.sh

[rhdh-qodo-merge]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

_{To customize comments, go to the Qodo configuration screen, or learn more in the docs.}

Previous review results

Review updated until commit b3aa808 ⚖️ Balanced

Results up to commit b3aa808 ⚖️ Balanced

No changes from previous review

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.60%. Comparing base (2a65941) to head (b3aa808).
⚠️ Report is 1 commits behind head on release-1.10.

Additional details and impacted files

@@                Coverage Diff                @@
##           release-1.10    #5189       +/-   ##
=================================================
+ Coverage         41.03%   69.60%   +28.57%     
=================================================
  Files               121      111       -10     
  Lines              2220     4702     +2482     
  Branches            563      536       -27     
=================================================
+ Hits                911     3273     +2362     
- Misses             1303     1428      +125     
+ Partials              6        1        -5     

Flag	Coverage Δ
install-dynamic-plugins	92.44% <ø> (?)
rhdh	38.81% <ø> (-2.23%)	⬇️

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2a65941...b3aa808. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

Image was built and published successfully. It is available at:

• quay.io/rhdh-community/rhdh:pr-5189
• quay.io/rhdh-community/rhdh:pr-5189-b3aa8087

[github-actions]

The container image build and publish workflows were skipped (either due to [skip-build] tag or no relevant changes with existing image).

[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

[rhdh-qodo-merge]

Code review by qodo was updated up to the latest commit b3aa808