Skip to content

fix(streams): add data-testids to dashboard + search (FlowwStar/FlowStar#379, #381)#396

Open
jdjioe5-cpu wants to merge 1 commit into
FlowwStar:mainfrom
jdjioe5-cpu:fix/issue-379-381-data-testids
Open

fix(streams): add data-testids to dashboard + search (FlowwStar/FlowStar#379, #381)#396
jdjioe5-cpu wants to merge 1 commit into
FlowwStar:mainfrom
jdjioe5-cpu:fix/issue-379-381-data-testids

Conversation

@jdjioe5-cpu

Copy link
Copy Markdown

Closes #379 and #381. Adds data-testid="streams-search-input" + per-stat testids via slugStatLabel().

…#379 + FlowwStar#381)

Closes FlowwStar#379 and FlowwStar#381.

### Issue FlowwStar#381 — streams search input
- app/app/streams/page.tsx:101: add data-testid="streams-search-input".
  Independent of placeholder copy.

### Issue FlowwStar#379 — dashboard stat cards
- components/streams/dashboard-stats.tsx:124: add a per-stat data-testid
  derived from the label via a `slugStatLabel` helper
  (e.g. `stat-total-streaming-value`, `stat-available-to-withdraw`,
  `stat-receiving`, `stat-sending`).

No behavior change. Both additions are stable DOM selectors for
e2e tests.
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.

good first issue: add data-testid to dashboard stat cards

1 participant