fix(e2e): fix flaky Widget Lab and graph chart tests

[alfredo1996]

Summary

• Widget Lab template card buttons had mismatched aria-label values ("Edit"/"Delete") vs E2E selectors ("Edit template"/"Delete template")
• Graph chart test had a 5s timeout for dialog dismiss that was too short for Radix animation — increased to 10s

Fixes

• app/src/app/(dashboard)/widget-lab/page.tsx: aria-label="Edit" → "Edit template", aria-label="Delete" → "Delete template"
• app/e2e/charts.spec.ts:578: dialog dismiss timeout 5s → 10s

Test plan

• E2E shard 1/5: graph chart test should pass
• E2E shard 5/5: all 3 widget-lab tests should pass

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes
  • Enhanced accessibility labels on widget template action buttons to provide clearer descriptions for screen reader users.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Two minor adjustments were made: an e2e test timeout for dialog hiding was increased from 5,000ms to 10,000ms to reduce flakiness, and accessibility labels were clarified in the Widget Lab page for edit and delete template buttons.

Changes

Cohort / File(s)	Summary
E2E Test Timeout app/e2e/charts.spec.ts	Increased dialog hide timeout from 5,000ms to 10,000ms in "graph chart — added widget renders on dashboard" test to reduce flakiness.
Widget Lab Accessibility app/src/app/(dashboard)/widget-lab/page.tsx	Clarified aria-labels for template action buttons: "Edit template" and "Delete template" replace generic "Edit" and "Delete" labels.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• alfredo1996/neoboard#91: Modifies the same Widget Lab component where aria-label changes were applied.
• alfredo1996/neoboard#296: Updates widget-lab UI with template action buttons in the same file receiving aria-label improvements.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main changes: fixing flaky tests in Widget Lab and graph chart E2E tests through aria-label corrections and timeout adjustments.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/e2e-flaky-tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[alfredo1996]

Superseded by #414 (combined release 1.1 fixes)

[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