Skip to content

test: add e2e tests for dashboard hub - #370

Merged
karelhala merged 1 commit into
RedHatInsights:masterfrom
OlhaTmlk:e2e-tests
Aug 18, 2026
Merged

test: add e2e tests for dashboard hub#370
karelhala merged 1 commit into
RedHatInsights:masterfrom
OlhaTmlk:e2e-tests

Conversation

@OlhaTmlk

@OlhaTmlk OlhaTmlk commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Description

Adds comprehensive E2E test coverage for the Dashboard Hub feature. All tests follow a create/test/cleanup pattern to leave the environment in its original state.

New test suites added:

editing-dashboard.spec.ts (6 new tests):

  • Dashboard Hub - Page Rendering: hub heading, create button, table with Homepage/Name/Last Modified columns
  • Dashboard Hub - Create Blank Dashboard: create from blank → verify in table → cleanup delete
  • Dashboard Hub - Duplicate Dashboard: duplicate via table action → verify notification + table entry → cleanup
  • Dashboard Hub - Delete Dashboard: create → navigate → delete via kebab → verify removed from hub
  • Dashboard Hub - Navigation: hub → dashboard → breadcrumb back to hub
  • Generic Dashboard Page - Rendering: header controls (title, edit name, add widgets, kebab), breadcrumb, and kebab menu items (Set as homepage, Duplicate, Copy config string, Delete)

widget-layout.spec.ts (4 new tests, 1 replaced):

  • Empty Dashboard: replaced mock-based test with real create/set-homepage/test/restore/cleanup flow that verifies auto-open drawer on empty dashboard
  • Extracted symbolic timeout constants (PAGE_LOAD_TIMEOUT_MS, WIDGET_LOAD_TIMEOUT_MS, DRAWER_TIMEOUT_MS)

Total: 22 E2E tests (11 per file)


Checklist ☑️

  • PR only fixes one issue or story
  • Change reviewed for extraneous code
  • UI best practices adhered to
  • All PR checks pass locally (build, lint, test, E2E)

@OlhaTmlk
OlhaTmlk force-pushed the e2e-tests branch 5 times, most recently from 88a874d to da574ea Compare August 18, 2026 06:34
@OlhaTmlk

Copy link
Copy Markdown
Contributor Author

/retest

@red-hat-konflux

Copy link
Copy Markdown
Contributor

All PipelineRuns for this commit have already succeeded. Use /retest <pipeline-name> to re-run a specific pipeline or /test to re-run all pipelines.

@OlhaTmlk

Copy link
Copy Markdown
Contributor Author

/retest

@OlhaTmlk
OlhaTmlk marked this pull request as ready for review August 18, 2026 09:27
@OlhaTmlk
OlhaTmlk requested a review from a team as a code owner August 18, 2026 09:27
@karelhala
karelhala merged commit 934831d into RedHatInsights:master Aug 18, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants