high priority low complexity testing pending testing specialist Tier 2

Acceptance Criteria

TimeWindowSelector test suite covers: all expected segment labels render (e.g., '7 days', '30 days', '90 days', 'All time'), tapping a segment fires the onChanged callback with the correct TimeWindow enum value, the active segment is visually distinct (verified via widget property or golden), and Semantics labels match expected strings for each segment
TimeWindowSelector tests cover the edge case where the widget is rendered with the first segment pre-selected and the last segment pre-selected
StatsSummaryCards test suite covers: data state renders the correct numeric value and label for each KPI card using a mocked Riverpod provider override, loading state renders a shimmer widget (assert widget type, not pixel colour), error state renders a user-friendly error message (not a raw exception string), and empty/zero data renders '0' not a blank
Each StatsSummaryCards KPI card Semantics label test asserts the full announced string (e.g., 'Total activities: 24') not just that a Semantics widget exists
All tests are deterministic and pass in CI without network calls or real Supabase connections
Test file follows project naming conventions (`*_test.dart`) and is co-located with or alongside the widget file
Total test coverage for TimeWindowSelector and StatsSummaryCards widget files reaches ≥ 90% line coverage

Technical Requirements

frameworks
Flutter
flutter_test
Riverpod (ProviderScope overrides)
data models
TimeWindow
StatsSummary
KPICard
performance requirements
Full test suite for both widgets completes in under 10 seconds
ui components
TimeWindowSelector
StatsSummaryCards
Shimmer loading widget

Execution Context

Execution Tier
Tier 2

Tier 2 - 518 tasks

Can start after Tier 1 completes

Implementation Notes

For Riverpod provider overrides in tests, create a `buildTestableWidget(Widget child, {List overrides})` helper that wraps the widget in `ProviderScope(overrides: overrides, child: MaterialApp(home: child))` — this avoids boilerplate duplication across test cases. Mock the stats summary provider to return a sealed-class AsyncValue (AsyncData, AsyncLoading, AsyncError) for the three state branches. For TimeWindowSelector interaction tests, use a `ValueChanged` mock callback (can be a simple List.add collector) rather than a full mock class. Avoid testing internal implementation details (e.g., private variables or specific widget tree depth); test observable behaviour only.

Ensure shimmer test finds the shimmer widget by type, not by colour, to stay resilient to design token changes.

Testing Requirements

All tests are Flutter widget tests (`testWidgets`). Use `ProviderScope` with `overrides` to inject mock data, loading, and error states into StatsSummaryCards without real providers. Use `tester.tap()` + `tester.pump()` for interaction tests. Use `tester.getSemantics()` for Semantics assertions.

Use `find.byType()` for shimmer detection and `find.text()` for label assertions. No golden image tests required for this task (covered in task-015 audit). Organise tests with descriptive `group()` blocks: 'TimeWindowSelector — rendering', 'TimeWindowSelector — interaction', 'TimeWindowSelector — accessibility', 'StatsSummaryCards — data state', 'StatsSummaryCards — loading state', 'StatsSummaryCards — error state', 'StatsSummaryCards — accessibility'.

Component
Stats Summary Cards
ui low
Epic Risks (4)
high impact high prob technical

fl_chart renders chart elements on a Canvas, making individual bars and data points invisible to the Flutter Semantics tree by default. Without explicit Semantics wrappers, VoiceOver and TalkBack users receive no chart information, violating the WCAG 2.2 AA requirement mandated by all three partner organizations.

Mitigation & Contingency

Mitigation: Wrap the fl_chart widget in a Semantics node with a dynamically generated textual description of the chart data (e.g., 'Bar chart: January 12, February 8, March 15 sessions'). Implement a collapsible data table alternative beneath the chart that screen readers can navigate row by row. Validate with VoiceOver on iOS and TalkBack on Android before the epic is marked complete.

Contingency: If fl_chart's Canvas rendering cannot be made accessible within the epic timeline, ship the chart hidden from the Semantics tree with ExcludeSemantics and promote the data table alternative to first-class UI so screen reader users have full access to the information. Log a tech-debt item to revisit native chart accessibility in a future sprint.

medium impact medium prob scope

Coordinators managing up to 5 chapters (NHF requirement) require the PeerMentorStatsList to display chapter affiliation labels for each row. With large chapter lists and many peer mentors, the list could become overwhelming and cause layout overflow or scrolling performance issues on lower-end Android devices.

Mitigation & Contingency

Mitigation: Implement chapter filtering as a segmented control above the list so coordinators can scope the list to one chapter at a time. Use ListView.builder (lazy rendering) rather than a Column of all rows. Profile scroll performance on a low-end Android device (Pixel 4a equivalent) with 50 peer mentors in scope during development.

Contingency: If multi-chapter display causes unacceptable performance, ship with single-chapter scope as the default view and a chapter switcher dropdown, deferring the combined cross-chapter list to a follow-up sprint.

low impact low prob technical

Summary cards and the chart widget rebuilding simultaneously on provider state change could cause a visible jank frame on slower devices, degrading perceived quality especially since this screen is intended to feel motivating and polished for gamification purposes.

Mitigation & Contingency

Mitigation: Use AnimatedSwitcher with a short fade transition (150ms) on the stats cards and chart so that data replacement feels intentional rather than jarring. Profile with Flutter DevTools on a mid-range device and ensure no frame exceeds 16ms during a time-window switch.

Contingency: If animation introduces complexity that delays delivery, ship without animation and use a loading skeleton (shimmer effect) during re-fetch instead, which is simpler and equally effective at masking the data swap.

high impact low prob security

If the role-based screen dispatch is misconfigured, a peer mentor could navigate to the coordinator stats screen and see aggregated chapter data for all peer mentors, which is a data privacy violation and a compliance risk for all three organizations.

Mitigation & Contingency

Mitigation: Implement role guard at the router level using an existing role-route-guard component so the coordinator screen route is unreachable for peer mentor roles. Add a widget test that mounts the coordinator screen with a peer mentor session token and asserts that the guard redirects to the no-access screen.

Contingency: If a bypass is found in QA, add a secondary in-screen role assertion in the coordinator screen's initState that throws an AuthorizationException and navigates to the error screen, ensuring defence in depth regardless of router configuration.