From 22cdca77bd68450d89af80afd58d0132dd62eaf6 Mon Sep 17 00:00:00 2001 From: Naser Mansour Date: Sun, 28 Dec 2025 23:17:54 +0200 Subject: [PATCH] complete story 7.1 with qa tests --- .../gates/7.1-client-dashboard-overview.yml | 51 +++ .../story-7.1-client-dashboard-overview.md | 117 ++++++- lang/ar/client.php | 22 ++ lang/en/client.php | 22 ++ .../client/dashboard-placeholder.blade.php | 8 - .../views/livewire/client/dashboard.blade.php | 244 +++++++++++++ routes/web.php | 2 +- tests/Feature/Client/DashboardTest.php | 326 ++++++++++++++++++ 8 files changed, 773 insertions(+), 19 deletions(-) create mode 100644 docs/qa/gates/7.1-client-dashboard-overview.yml delete mode 100644 resources/views/livewire/client/dashboard-placeholder.blade.php create mode 100644 resources/views/livewire/client/dashboard.blade.php create mode 100644 tests/Feature/Client/DashboardTest.php diff --git a/docs/qa/gates/7.1-client-dashboard-overview.yml b/docs/qa/gates/7.1-client-dashboard-overview.yml new file mode 100644 index 0000000..2cb2414 --- /dev/null +++ b/docs/qa/gates/7.1-client-dashboard-overview.yml @@ -0,0 +1,51 @@ +# Quality Gate: Story 7.1 - Client Dashboard Overview +schema: 1 +story: "7.1" +story_title: "Client Dashboard Overview" +gate: PASS +status_reason: "All acceptance criteria implemented and verified by 24 passing tests. Clean code following project standards with proper security boundaries." +reviewer: "Quinn (Test Architect)" +updated: "2025-12-28T00:00:00Z" + +waiver: { active: false } + +top_issues: [] + +risk_summary: + totals: { critical: 0, high: 0, medium: 0, low: 1 } + recommendations: + must_fix: [] + monitor: + - "Consider extracting reorder()->latest() pattern to model method for cleaner reuse" + +quality_score: 100 +expires: "2026-01-11T00:00:00Z" + +evidence: + tests_reviewed: 24 + risks_identified: 0 + trace: + ac_covered: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20] + ac_gaps: [] + +nfr_validation: + security: + status: PASS + notes: "Route protected with auth+client middleware, admin blocked, data scoped to user" + performance: + status: PASS + notes: "Efficient queries with limits and eager loading, no N+1 issues" + reliability: + status: PASS + notes: "All empty states handled gracefully" + maintainability: + status: PASS + notes: "Clean Volt component following project standards, well-organized code" + +recommendations: + immediate: [] + future: + - action: "Extract reorder()->latest() to a dedicated Timeline method like latestUpdate()" + refs: ["resources/views/livewire/client/dashboard.blade.php:122"] + - action: "Add database index on booking_date if performance degrades" + refs: ["database/migrations/*consultations*"] diff --git a/docs/stories/story-7.1-client-dashboard-overview.md b/docs/stories/story-7.1-client-dashboard-overview.md index c4805ed..954e619 100644 --- a/docs/stories/story-7.1-client-dashboard-overview.md +++ b/docs/stories/story-7.1-client-dashboard-overview.md @@ -278,16 +278,16 @@ Ensure factories exist with states: - Story 7.5: New Booking Interface (link target for book button) ## Definition of Done -- [ ] Volt component created at `resources/views/livewire/client/dashboard.blade.php` -- [ ] Route registered and protected with auth middleware -- [ ] All 5 widgets display correctly with real data -- [ ] Data strictly scoped to authenticated user (security verified) -- [ ] Empty states display appropriately for each widget -- [ ] Mobile responsive (tested on 375px viewport) -- [ ] Bilingual content working (AR/EN toggle) -- [ ] All test scenarios pass -- [ ] Code formatted with `vendor/bin/pint --dirty` -- [ ] No console errors or warnings +- [x] Volt component created at `resources/views/livewire/client/dashboard.blade.php` +- [x] Route registered and protected with auth middleware +- [x] All 5 widgets display correctly with real data +- [x] Data strictly scoped to authenticated user (security verified) +- [x] Empty states display appropriately for each widget +- [x] Mobile responsive (tested on 375px viewport) +- [x] Bilingual content working (AR/EN toggle) +- [x] All test scenarios pass +- [x] Code formatted with `vendor/bin/pint --dirty` +- [x] No console errors or warnings ## Estimation **Complexity:** Medium | **Effort:** 4-5 hours @@ -297,3 +297,100 @@ Ensure factories exist with states: - Timeline detail view (Story 7.3) - Booking form functionality (Story 7.5) - Real-time updates via websockets + +--- + +## QA Results + +### Review Date: 2025-12-28 + +### Reviewed By: Quinn (Test Architect) + +### Code Quality Assessment + +The implementation is well-structured and follows project coding standards. The Volt component uses the class-based pattern correctly, queries are efficient with proper scoping to the authenticated user, and all Flux UI components are used appropriately. The bilingual support is complete with both English and Arabic translations. The component demonstrates good practices with proper separation of concerns in the `with()` method. + +### Refactoring Performed + +None required - the code quality is already satisfactory. + +### Compliance Check + +- Coding Standards: ✓ Class-based Volt pattern, Flux UI components, proper localization +- Project Structure: ✓ Files placed in correct locations, routes properly configured +- Testing Strategy: ✓ 24 comprehensive Pest tests with Volt::test() pattern +- All ACs Met: ✓ All acceptance criteria implemented and tested + +### Improvements Checklist + +- [x] All data scoped to authenticated user (verified via tests) +- [x] Empty states display appropriately for all widgets +- [x] Mobile responsive layout with card grid +- [x] Bilingual content working (AR/EN) +- [x] All 24 tests passing +- [x] Code formatted with pint --dirty +- [ ] Consider extracting the `reorder()->latest()` pattern in blade (line 122) to a dedicated model method or scope for cleaner reuse + +### Security Review + +No security concerns found: +- Route protected with `auth` and `client` middleware +- Admin users properly blocked (403 response verified by test) +- All data queries scoped to authenticated user +- No SQL injection risks (Eloquent used throughout) +- Proper authorization boundaries between clients + +### Performance Considerations + +No significant performance concerns: +- Queries are efficient with proper indexing on foreign keys +- `take(3)` limits recent updates query +- Eager loading used for timeline relationship on updates +- Consider adding database index on `booking_date` if consultation queries become slow (future optimization) + +### Files Modified During Review + +None - no modifications made. + +### Gate Status + +Gate: PASS → docs/qa/gates/7.1-client-dashboard-overview.yml + +### Recommended Status + +✓ Ready for Done - All acceptance criteria met, comprehensive test coverage, clean implementation. + +--- + +## Dev Agent Record + +### Status +**Ready for Review** + +### Agent Model Used +Claude Opus 4.5 + +### File List + +| File | Action | Purpose | +|------|--------|---------| +| `resources/views/livewire/client/dashboard.blade.php` | Created | Main client dashboard Volt component with 5 widgets | +| `routes/web.php` | Modified | Changed client dashboard route to use Volt::route() | +| `lang/en/client.php` | Modified | Added dashboard localization keys | +| `lang/ar/client.php` | Modified | Added Arabic dashboard localization keys | +| `tests/Feature/Client/DashboardTest.php` | Created | 24 feature tests for client dashboard | +| `resources/views/livewire/client/dashboard-placeholder.blade.php` | Deleted | Removed placeholder file | + +### Change Log +- Created client dashboard Volt component with Welcome Section, Upcoming Consultation Widget, Active Cases Widget, Recent Updates Widget, and Booking Status Widget +- Updated route from `Route::view()` to `Volt::route()` for proper Livewire component rendering +- Added 18 localization keys for dashboard UI in both English and Arabic +- Implemented proper data scoping to authenticated user only +- Used existing `x-layouts.app` layout (no need for separate client layout) +- Fixed query ordering issue with `reorder()` for timeline updates relationship +- All 24 tests pass covering authorization, data display, empty states, and booking status logic + +### Completion Notes +- Skeleton loaders not implemented (Livewire handles loading states automatically via wire:loading) +- Consultation model uses `booking_date`/`booking_time` columns (not `scheduled_date`/`scheduled_time` as in story spec) +- Client layout reuses existing `app.blade.php` layout which supports both admin and client contexts diff --git a/lang/ar/client.php b/lang/ar/client.php index c0ec6f3..f9fde16 100644 --- a/lang/ar/client.php +++ b/lang/ar/client.php @@ -1,6 +1,28 @@ [ + 'title' => 'لوحة التحكم', + 'welcome' => 'مرحباً، :name', + 'upcoming_consultation' => 'الاستشارة القادمة', + 'view_details' => 'عرض التفاصيل', + 'no_upcoming' => 'لا توجد استشارات قادمة', + 'book_first' => 'احجز استشارتك الأولى', + 'active_cases' => 'القضايا النشطة', + 'cases_count' => '{1} قضية نشطة|[2,10] قضايا نشطة|[11,*] قضية نشطة', + 'latest_update' => 'آخر تحديث', + 'view_all_cases' => 'عرض جميع القضايا', + 'no_cases' => 'لا توجد قضايا مسندة بعد', + 'recent_updates' => 'التحديثات الأخيرة', + 'no_updates' => 'لا توجد تحديثات حديثة', + 'booking_status' => 'حالة الحجز', + 'pending_bookings' => '{1} :count طلب حجز معلق|[2,10] :count طلبات حجز معلقة|[11,*] :count طلب حجز معلق', + 'can_book' => 'يمكنك حجز استشارة اليوم', + 'cannot_book' => 'لديك حجز بالفعل لهذا اليوم', + 'book_consultation' => 'حجز استشارة', + ], + // Timeline views 'my_cases' => 'قضاياي', 'active_cases' => 'القضايا النشطة', diff --git a/lang/en/client.php b/lang/en/client.php index cc836e3..526896a 100644 --- a/lang/en/client.php +++ b/lang/en/client.php @@ -1,6 +1,28 @@ [ + 'title' => 'Dashboard', + 'welcome' => 'Welcome, :name', + 'upcoming_consultation' => 'Upcoming Consultation', + 'view_details' => 'View Details', + 'no_upcoming' => 'No upcoming consultations', + 'book_first' => 'Book your first consultation', + 'active_cases' => 'Active Cases', + 'cases_count' => '{1} active case|[2,*] active cases', + 'latest_update' => 'Latest update', + 'view_all_cases' => 'View All Cases', + 'no_cases' => 'No cases assigned yet', + 'recent_updates' => 'Recent Updates', + 'no_updates' => 'No recent updates', + 'booking_status' => 'Booking Status', + 'pending_bookings' => '{1} :count pending booking request|[2,*] :count pending booking requests', + 'can_book' => 'You can book a consultation today', + 'cannot_book' => 'You already have a booking for today', + 'book_consultation' => 'Book Consultation', + ], + // Timeline views 'my_cases' => 'My Cases', 'active_cases' => 'Active Cases', diff --git a/resources/views/livewire/client/dashboard-placeholder.blade.php b/resources/views/livewire/client/dashboard-placeholder.blade.php deleted file mode 100644 index ac368a0..0000000 --- a/resources/views/livewire/client/dashboard-placeholder.blade.php +++ /dev/null @@ -1,8 +0,0 @@ - -
-
- {{ __('Client Dashboard') }} - {{ __('Dashboard coming soon') }} -
-
- diff --git a/resources/views/livewire/client/dashboard.blade.php b/resources/views/livewire/client/dashboard.blade.php new file mode 100644 index 0000000..6c91992 --- /dev/null +++ b/resources/views/livewire/client/dashboard.blade.php @@ -0,0 +1,244 @@ +user(); + + return [ + 'upcomingConsultation' => $user->consultations() + ->approved() + ->where('booking_date', '>=', today()) + ->orderBy('booking_date') + ->orderBy('booking_time') + ->first(), + 'activeTimelinesCount' => $user->timelines()->active()->count(), + 'latestTimeline' => $user->timelines()->active()->latest()->first(), + 'recentUpdates' => TimelineUpdate::whereHas('timeline', fn ($q) => $q->where('user_id', $user->id)) + ->latest() + ->take(3) + ->with('timeline') + ->get(), + 'pendingBookingsCount' => $user->consultations()->pending()->count(), + 'canBookToday' => ! $user->consultations() + ->whereDate('booking_date', today()) + ->whereIn('status', ['pending', 'approved']) + ->exists(), + ]; + } +}; ?> + +
+ {{-- Welcome Section --}} +
+ + {{ __('client.dashboard.welcome', ['name' => auth()->user()->full_name]) }} + + + {{ now()->locale(app()->getLocale())->translatedFormat(app()->getLocale() === 'ar' ? 'l، j F Y' : 'l, F j, Y') }} + +
+ + {{-- Widgets Grid --}} +
+ {{-- Upcoming Consultation Widget --}} +
+ + {{ __('client.dashboard.upcoming_consultation') }} + + + @if ($upcomingConsultation) +
+
+ + + @if (app()->getLocale() === 'ar') + {{ $upcomingConsultation->booking_date->format('d/m/Y') }} + @else + {{ $upcomingConsultation->booking_date->format('m/d/Y') }} + @endif + +
+
+ + + {{ \Carbon\Carbon::parse($upcomingConsultation->booking_time)->format('g:i A') }} + +
+
+ @if ($upcomingConsultation->consultation_type->value === 'free') + {{ $upcomingConsultation->consultation_type->label() }} + @else + {{ $upcomingConsultation->consultation_type->label() }} + @endif + {{ $upcomingConsultation->status->label() }} +
+
+ + {{ __('client.dashboard.view_details') }} + +
+
+ @else +
+ + {{ __('client.dashboard.no_upcoming') }} +
+ + {{ __('client.dashboard.book_first') }} + +
+
+ @endif +
+ + {{-- Active Cases Widget --}} +
+ + {{ __('client.dashboard.active_cases') }} + + + @if ($activeTimelinesCount > 0) +
+
+ {{ $activeTimelinesCount }} + {{ trans_choice('client.dashboard.cases_count', $activeTimelinesCount) }} +
+ @if ($latestTimeline) + @php + $latestUpdate = $latestTimeline->updates()->reorder()->latest()->first(); + @endphp + @if ($latestUpdate) +
+ + {{ __('client.dashboard.latest_update') }}: + + + {{ Str::limit($latestUpdate->update_text, 100) }} + +
+ @endif + @endif +
+ + {{ __('client.dashboard.view_all_cases') }} + +
+
+ @else +
+ + {{ __('client.dashboard.no_cases') }} +
+ @endif +
+ + {{-- Recent Updates Widget --}} +
+ + {{ __('client.dashboard.recent_updates') }} + + + @if ($recentUpdates->isNotEmpty()) +
+ @foreach ($recentUpdates as $update) +
+
+
+ + {{ $update->timeline->case_name }} + + + {{ $update->created_at->locale(app()->getLocale())->diffForHumans() }} + + + {{ Str::limit($update->update_text, 80) }} + +
+ +
+
+ @endforeach +
+ @else +
+ + {{ __('client.dashboard.no_updates') }} +
+ @endif +
+ + {{-- Booking Status Widget --}} +
+ + {{ __('client.dashboard.booking_status') }} + + +
+ @if ($pendingBookingsCount > 0) +
+ + + {{ trans_choice('client.dashboard.pending_bookings', $pendingBookingsCount, ['count' => $pendingBookingsCount]) }} + +
+ @endif + +
+ @if ($canBookToday) +
+ + + {{ __('client.dashboard.can_book') }} + +
+ @else +
+ + + {{ __('client.dashboard.cannot_book') }} + +
+ @endif +
+ +
+ + {{ __('client.dashboard.book_consultation') }} + +
+
+
+
+
+ diff --git a/routes/web.php b/routes/web.php index 8edb920..575a65f 100644 --- a/routes/web.php +++ b/routes/web.php @@ -125,7 +125,7 @@ Route::middleware(['auth', 'active'])->group(function () { // Client routes Route::middleware('client')->prefix('client')->name('client.')->group(function () { - Route::view('/dashboard', 'livewire.client.dashboard-placeholder') + Volt::route('/dashboard', 'client.dashboard') ->name('dashboard'); // Consultations diff --git a/tests/Feature/Client/DashboardTest.php b/tests/Feature/Client/DashboardTest.php new file mode 100644 index 0000000..378fc9e --- /dev/null +++ b/tests/Feature/Client/DashboardTest.php @@ -0,0 +1,326 @@ +individual()->create(); + + $this->actingAs($user) + ->get(route('client.dashboard')) + ->assertOk(); +}); + +test('company client can view dashboard', function () { + $user = User::factory()->company()->create(); + + $this->actingAs($user) + ->get(route('client.dashboard')) + ->assertOk(); +}); + +test('unauthenticated user cannot access dashboard', function () { + $this->get(route('client.dashboard')) + ->assertRedirect(route('login')); +}); + +test('admin cannot access client dashboard', function () { + $admin = User::factory()->admin()->create(); + + $this->actingAs($admin) + ->get(route('client.dashboard')) + ->assertForbidden(); +}); + +// Welcome Section Tests +test('dashboard shows welcome message with user name', function () { + $user = User::factory()->individual()->create(['full_name' => 'John Doe']); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee('John Doe'); +}); + +// Upcoming Consultation Widget Tests +test('dashboard shows upcoming approved consultation', function () { + $user = User::factory()->individual()->create(); + $consultation = Consultation::factory()->approved()->create([ + 'user_id' => $user->id, + 'booking_date' => today()->addDays(3), + 'booking_time' => '10:00:00', + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee('10:00 AM'); +}); + +test('dashboard shows only authenticated user consultations', function () { + $user = User::factory()->individual()->create(); + $otherUser = User::factory()->individual()->create(); + + Consultation::factory()->approved()->create([ + 'user_id' => $user->id, + 'booking_date' => today()->addDays(3), + 'booking_time' => '09:00:00', + ]); + Consultation::factory()->approved()->create([ + 'user_id' => $otherUser->id, + 'booking_date' => today()->addDays(2), + 'booking_time' => '14:00:00', + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee('9:00 AM') + ->assertDontSee('2:00 PM'); +}); + +test('dashboard shows no upcoming consultations when none exist', function () { + $user = User::factory()->individual()->create(); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.no_upcoming')); +}); + +test('dashboard does not show past approved consultations', function () { + $user = User::factory()->individual()->create(); + Consultation::factory()->approved()->create([ + 'user_id' => $user->id, + 'booking_date' => today()->subDay(), + 'booking_time' => '10:00:00', + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.no_upcoming')); +}); + +test('dashboard does not show pending consultations as upcoming', function () { + $user = User::factory()->individual()->create(); + Consultation::factory()->pending()->create([ + 'user_id' => $user->id, + 'booking_date' => today()->addDays(3), + 'booking_time' => '11:00:00', + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.no_upcoming')); +}); + +// Active Cases Widget Tests +test('dashboard shows correct active timelines count', function () { + $user = User::factory()->individual()->create(); + Timeline::factory()->active()->count(3)->create(['user_id' => $user->id]); + Timeline::factory()->archived()->count(2)->create(['user_id' => $user->id]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee('3'); +}); + +test('dashboard shows no cases empty state when user has no timelines', function () { + $user = User::factory()->individual()->create(); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.no_cases')); +}); + +test('dashboard shows latest update preview for active cases', function () { + $user = User::factory()->individual()->create(); + $timeline = Timeline::factory()->active()->create(['user_id' => $user->id]); + TimelineUpdate::factory()->create([ + 'timeline_id' => $timeline->id, + 'update_text' => 'This is the latest case update for testing purposes.', + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee('This is the latest case update'); +}); + +// Recent Updates Widget Tests +test('dashboard shows last 3 timeline updates', function () { + $user = User::factory()->individual()->create(); + $timeline = Timeline::factory()->create(['user_id' => $user->id]); + + TimelineUpdate::factory()->create([ + 'timeline_id' => $timeline->id, + 'update_text' => 'Update One', + 'created_at' => now()->subDays(4), + ]); + TimelineUpdate::factory()->create([ + 'timeline_id' => $timeline->id, + 'update_text' => 'Update Two', + 'created_at' => now()->subDays(3), + ]); + TimelineUpdate::factory()->create([ + 'timeline_id' => $timeline->id, + 'update_text' => 'Update Three', + 'created_at' => now()->subDays(2), + ]); + TimelineUpdate::factory()->create([ + 'timeline_id' => $timeline->id, + 'update_text' => 'Update Four', + 'created_at' => now()->subDay(), + ]); + TimelineUpdate::factory()->create([ + 'timeline_id' => $timeline->id, + 'update_text' => 'Update Five', + 'created_at' => now(), + ]); + + $this->actingAs($user); + + // Should see the 3 most recent updates, not the older ones + Volt::test('client.dashboard') + ->assertSee('Update Five') + ->assertSee('Update Four') + ->assertSee('Update Three') + ->assertDontSee('Update Two') + ->assertDontSee('Update One'); +}); + +test('dashboard shows no updates empty state when user has no timeline updates', function () { + $user = User::factory()->individual()->create(); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.no_updates')); +}); + +// Booking Status Widget Tests +test('canBookToday is false when user has approved booking today', function () { + $user = User::factory()->individual()->create(); + Consultation::factory()->approved()->create([ + 'user_id' => $user->id, + 'booking_date' => today(), + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.cannot_book')); +}); + +test('canBookToday is false when user has pending booking today', function () { + $user = User::factory()->individual()->create(); + Consultation::factory()->pending()->create([ + 'user_id' => $user->id, + 'booking_date' => today(), + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.cannot_book')); +}); + +test('canBookToday is true when user has no booking today', function () { + $user = User::factory()->individual()->create(); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.can_book')); +}); + +test('canBookToday is true when user has only rejected booking today', function () { + $user = User::factory()->individual()->create(); + Consultation::factory()->rejected()->create([ + 'user_id' => $user->id, + 'booking_date' => today(), + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.can_book')); +}); + +test('canBookToday is true when user has only cancelled booking today', function () { + $user = User::factory()->individual()->create(); + Consultation::factory()->cancelled()->create([ + 'user_id' => $user->id, + 'booking_date' => today(), + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.can_book')); +}); + +test('dashboard shows pending bookings count', function () { + $user = User::factory()->individual()->create(); + Consultation::factory()->pending()->count(3)->create([ + 'user_id' => $user->id, + 'booking_date' => today()->addDays(5), + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee('3'); +}); + +// Empty State Tests +test('dashboard handles empty state gracefully', function () { + $user = User::factory()->individual()->create(); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertSee(__('client.dashboard.no_upcoming')) + ->assertSee(__('client.dashboard.no_cases')) + ->assertSee(__('client.dashboard.no_updates')) + ->assertSee(__('client.dashboard.can_book')); +}); + +// Data Isolation Tests +test('dashboard does not show other users timeline updates', function () { + $user = User::factory()->individual()->create(); + $otherUser = User::factory()->individual()->create(); + + $otherTimeline = Timeline::factory()->create(['user_id' => $otherUser->id]); + TimelineUpdate::factory()->create([ + 'timeline_id' => $otherTimeline->id, + 'update_text' => 'Other user secret update', + ]); + + $this->actingAs($user); + + Volt::test('client.dashboard') + ->assertDontSee('Other user secret update'); +}); + +test('dashboard does not count other users active timelines', function () { + $user = User::factory()->individual()->create(); + $otherUser = User::factory()->individual()->create(); + + Timeline::factory()->active()->create(['user_id' => $user->id]); + Timeline::factory()->active()->count(5)->create(['user_id' => $otherUser->id]); + + $this->actingAs($user); + + // User should only see their 1 active timeline, not the 5 from other user + Volt::test('client.dashboard') + ->assertSee('1'); +});