lock user stats

Author: monneyboi

poliloom-gui/src/app/evaluate/complete/page.test.tsx

@@ -0,0 +1,69 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { screen, fireEvent } from '@testing-library/react'
+import { render } from '@/test/test-utils'
+import CompletePage from './page'
+
+const mockPush = vi.fn()
+vi.mock('next/navigation', () => ({
+  useRouter: () => ({
+    push: mockPush,
+  }),
+}))
+
+const mockResetSession = vi.fn()
+const mockUseEvaluationSession = vi.fn()
+vi.mock('@/contexts/EvaluationSessionContext', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('@/contexts/EvaluationSessionContext')>()
+  return {
+    ...actual,
+    useEvaluationSession: () => mockUseEvaluationSession(),
+  }
+})
+
+describe('Complete Page', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+
+    mockUseEvaluationSession.mockReturnValue({
+      sessionGoal: 5,
+      resetSession: mockResetSession,
+    })
+  })
+
+  it('shows session complete message with politician count', () => {
+    render(<CompletePage />)
+
+    expect(screen.getByText('Session Complete!')).toBeInTheDocument()
+    expect(screen.getByText(/reviewed 5 politicians/)).toBeInTheDocument()
+  })
+
+  it('shows Start Another Round as primary action', () => {
+    render(<CompletePage />)
+
+    expect(screen.getByRole('button', { name: 'Start Another Round' })).toBeInTheDocument()
+  })
+
+  it('shows Return Home as secondary action', () => {
+    render(<CompletePage />)
+
+    expect(screen.getByRole('button', { name: 'Return Home' })).toBeInTheDocument()
+  })
+
+  it('navigates to evaluate page when clicking Start Another Round', () => {
+    render(<CompletePage />)
+
+    fireEvent.click(screen.getByRole('button', { name: 'Start Another Round' }))
+
+    expect(mockResetSession).toHaveBeenCalled()
+    expect(mockPush).toHaveBeenCalledWith('/evaluate')
+  })
+
+  it('navigates to home page when clicking Return Home', () => {
+    render(<CompletePage />)
+
+    fireEvent.click(screen.getByRole('button', { name: 'Return Home' }))
+
+    expect(mockResetSession).toHaveBeenCalled()
+    expect(mockPush).toHaveBeenCalledWith('/')
+  })
+})

poliloom-gui/src/app/evaluate/page.test.tsx

@@ -47,7 +47,7 @@ describe('Evaluate Page', () => {
       render(<EvaluatePage />)
     })
 
-    expect(screen.getByText('Finding politicians...')).toBeInTheDocument()
+    expect(screen.getByText('Gathering data...')).toBeInTheDocument()
   })
 
   it('shows all caught up message when no politicians and nothing to enrich', async () => {
@@ -92,7 +92,7 @@ describe('Evaluate Page', () => {
       render(<EvaluatePage />)
     })
 
-    expect(screen.getByText('Finding politicians...')).toBeInTheDocument()
+    expect(screen.getByText('Gathering data...')).toBeInTheDocument()
   })
 
   it('shows PoliticianEvaluation component when politician data is available', async () => {

poliloom-gui/src/app/evaluate/page.tsx

@@ -7,13 +7,13 @@ import { useEvaluationSession } from '@/contexts/EvaluationSessionContext'
 import { CenteredCard } from '@/components/ui/CenteredCard'
 import { Button } from '@/components/ui/Button'
 import { Spinner } from '@/components/ui/Spinner'
-import { useTutorial } from '@/contexts/TutorialContext'
+import { useUserProgress } from '@/contexts/UserProgressContext'
 import { useUserPreferences } from '@/contexts/UserPreferencesContext'
 
 export default function EvaluatePage() {
   const router = useRouter()
   const { currentPolitician, loading, isSessionComplete, enrichmentMeta } = useEvaluationSession()
-  const { completeBasicTutorial, completeAdvancedTutorial } = useTutorial()
+  const { completeBasicTutorial, completeAdvancedTutorial, statsUnlocked } = useUserProgress()
   const { isAdvancedMode } = useUserPreferences()
 
   // Mark tutorials complete once on mount
@@ -28,9 +28,13 @@ export default function EvaluatePage() {
   // Navigate to completion page when session goal is reached
   useEffect(() => {
     if (isSessionComplete) {
-      router.push('/evaluate/complete')
+      if (statsUnlocked) {
+        router.push('/evaluate/complete')
+      } else {
+        router.push('/evaluate/unlocked')
+      }
     }
-  }, [isSessionComplete, router])
+  }, [isSessionComplete, router, statsUnlocked])
 
   // Determine if we're in a loading state (fetching or waiting for enrichment)
   const isWaitingForData =
@@ -51,7 +55,7 @@ export default function EvaluatePage() {
           No more politicians to evaluate for your current filters. Try different filters to
           continue contributing.
         </p>
-        <Button href="/" size="large">
+        <Button href="/" size="large" fullWidth>
           Start New Session
         </Button>
       </CenteredCard>

poliloom-gui/src/app/evaluate/unlocked/page.test.tsx

@@ -0,0 +1,84 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { screen, fireEvent } from '@testing-library/react'
+import { render } from '@/test/test-utils'
+import UnlockedPage from './page'
+
+const mockPush = vi.fn()
+vi.mock('next/navigation', () => ({
+  useRouter: () => ({
+    push: mockPush,
+  }),
+}))
+
+const mockResetSession = vi.fn()
+vi.mock('@/contexts/EvaluationSessionContext', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('@/contexts/EvaluationSessionContext')>()
+  return {
+    ...actual,
+    useEvaluationSession: () => ({
+      resetSession: mockResetSession,
+    }),
+  }
+})
+
+const mockUnlockStats = vi.fn()
+vi.mock('@/contexts/UserProgressContext', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('@/contexts/UserProgressContext')>()
+  return {
+    ...actual,
+    useUserProgress: () => ({
+      unlockStats: mockUnlockStats,
+    }),
+  }
+})
+
+describe('Unlocked Page', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+  })
+
+  it('shows stats unlocked message', () => {
+    render(<UnlockedPage />)
+
+    expect(screen.getByText('Stats Unlocked!')).toBeInTheDocument()
+    expect(
+      screen.getByText(/completed your first session and unlocked the community stats page/),
+    ).toBeInTheDocument()
+  })
+
+  it('calls unlockStats on mount', () => {
+    render(<UnlockedPage />)
+
+    expect(mockUnlockStats).toHaveBeenCalled()
+  })
+
+  it('shows View Stats as primary action', () => {
+    render(<UnlockedPage />)
+
+    expect(screen.getByRole('button', { name: 'View Stats' })).toBeInTheDocument()
+  })
+
+  it('shows Start Another Round as secondary action', () => {
+    render(<UnlockedPage />)
+
+    expect(screen.getByRole('button', { name: 'Start Another Round' })).toBeInTheDocument()
+  })
+
+  it('navigates to stats page when clicking View Stats', () => {
+    render(<UnlockedPage />)
+
+    fireEvent.click(screen.getByRole('button', { name: 'View Stats' }))
+
+    expect(mockResetSession).toHaveBeenCalled()
+    expect(mockPush).toHaveBeenCalledWith('/stats')
+  })
+
+  it('navigates to evaluate page when clicking Start Another Round', () => {
+    render(<UnlockedPage />)
+
+    fireEvent.click(screen.getByRole('button', { name: 'Start Another Round' }))
+
+    expect(mockResetSession).toHaveBeenCalled()
+    expect(mockPush).toHaveBeenCalledWith('/evaluate')
+  })
+})

poliloom-gui/src/app/evaluate/unlocked/page.tsx

@@ -0,0 +1,44 @@
+'use client'
+
+import { useEffect } from 'react'
+import { useRouter } from 'next/navigation'
+import { useEvaluationSession } from '@/contexts/EvaluationSessionContext'
+import { useUserProgress } from '@/contexts/UserProgressContext'
+import { CenteredCard } from '@/components/ui/CenteredCard'
+import { Button } from '@/components/ui/Button'
+
+export default function UnlockedPage() {
+  const router = useRouter()
+  const { resetSession } = useEvaluationSession()
+  const { unlockStats } = useUserProgress()
+
+  useEffect(() => {
+    unlockStats()
+  }, [unlockStats])
+
+  const handleViewStats = () => {
+    resetSession()
+    router.push('/stats')
+  }
+
+  const handleStartAnother = () => {
+    resetSession()
+    router.push('/evaluate')
+  }
+
+  return (
+    <CenteredCard emoji="🔓" title="Stats Unlocked!">
+      <p className="mb-8">
+        Great work! You&apos;ve completed your first session and unlocked the community stats page.
+      </p>
+      <div className="flex flex-col gap-4">
+        <Button onClick={handleViewStats} size="large" fullWidth>
+          View Stats
+        </Button>
+        <Button onClick={handleStartAnother} variant="secondary" size="large" fullWidth>
+          Start Another Round
+        </Button>
+      </div>
+    </CenteredCard>
+  )
+}

poliloom-gui/src/app/layout.tsx

@@ -5,7 +5,7 @@ import { auth } from '@/auth'
 import { SessionProvider } from '@/components/SessionProvider'
 import { UserPreferencesProvider } from '@/contexts/UserPreferencesContext'
 import { EvaluationSessionProvider } from '@/contexts/EvaluationSessionContext'
-import { TutorialProvider } from '@/contexts/TutorialContext'
+import { UserProgressProvider } from '@/contexts/UserProgressContext'
 import { FetchInterceptor } from '@/components/FetchInterceptor'
 import { MobileGuard } from '@/components/layout/MobileGuard'
 import { Header } from '@/components/layout/Header'
@@ -41,10 +41,10 @@ export default async function RootLayout({
           <FetchInterceptor />
           <UserPreferencesProvider>
             <EvaluationSessionProvider>
-              <TutorialProvider>
+              <UserProgressProvider>
                 <Header />
                 <MobileGuard>{children}</MobileGuard>
-              </TutorialProvider>
+              </UserProgressProvider>
             </EvaluationSessionProvider>
           </UserPreferencesProvider>
         </SessionProvider>

poliloom-gui/src/app/page.test.tsx

@@ -37,12 +37,12 @@ vi.mock('next-auth/react', () => ({
   signIn: (...args: unknown[]) => mockSignIn(...args),
 }))
 
-const mockUseTutorial = vi.fn()
-vi.mock('@/contexts/TutorialContext', async (importOriginal) => {
-  const actual = await importOriginal<typeof import('@/contexts/TutorialContext')>()
+const mockUseUserProgress = vi.fn()
+vi.mock('@/contexts/UserProgressContext', async (importOriginal) => {
+  const actual = await importOriginal<typeof import('@/contexts/UserProgressContext')>()
   return {
     ...actual,
-    useTutorial: () => mockUseTutorial(),
+    useUserProgress: () => mockUseUserProgress(),
   }
 })
 
@@ -60,12 +60,13 @@ describe('Home Page (Filter Selection)', () => {
     vi.clearAllMocks()
 
     // Default tutorial state - not completed
-    mockUseTutorial.mockReturnValue({
+    mockUseUserProgress.mockReturnValue({
       hasCompletedBasicTutorial: false,
       hasCompletedAdvancedTutorial: false,
+      statsUnlocked: false,
       completeBasicTutorial: vi.fn(),
       completeAdvancedTutorial: vi.fn(),
-      resetTutorial: vi.fn(),
+      unlockStats: vi.fn(),
     })
 
     // Default user preferences - basic mode
@@ -162,12 +163,13 @@ describe('Home Page (Filter Selection)', () => {
       status: 'loading',
     })
 
-    mockUseTutorial.mockReturnValue({
+    mockUseUserProgress.mockReturnValue({
       hasCompletedBasicTutorial: true,
       hasCompletedAdvancedTutorial: false,
+      statsUnlocked: false,
       completeBasicTutorial: vi.fn(),
       completeAdvancedTutorial: vi.fn(),
-      resetTutorial: vi.fn(),
+      unlockStats: vi.fn(),
     })
 
     // Basic mode (default from beforeEach)
@@ -186,12 +188,13 @@ describe('Home Page (Filter Selection)', () => {
       status: 'loading',
     })
 
-    mockUseTutorial.mockReturnValue({
+    mockUseUserProgress.mockReturnValue({
       hasCompletedBasicTutorial: true,
       hasCompletedAdvancedTutorial: false,
+      statsUnlocked: false,
       completeBasicTutorial: vi.fn(),
       completeAdvancedTutorial: vi.fn(),
-      resetTutorial: vi.fn(),
+      unlockStats: vi.fn(),
     })
 
     mockUseUserPreferences.mockReturnValue({
@@ -221,12 +224,13 @@ describe('Home Page (Filter Selection)', () => {
       status: 'loading',
     })
 
-    mockUseTutorial.mockReturnValue({
+    mockUseUserProgress.mockReturnValue({
       hasCompletedBasicTutorial: true,
       hasCompletedAdvancedTutorial: true,
+      statsUnlocked: true,
       completeBasicTutorial: vi.fn(),
       completeAdvancedTutorial: vi.fn(),
-      resetTutorial: vi.fn(),
+      unlockStats: vi.fn(),
     })
 
     mockUseUserPreferences.mockReturnValue({

poliloom-gui/src/app/page.tsx

@@ -5,7 +5,7 @@ import { Button } from '@/components/ui/Button'
 import { Toggle } from '@/components/ui/Toggle'
 import { MultiSelect, MultiSelectOption } from '@/components/entity/MultiSelect'
 import { useUserPreferences } from '@/contexts/UserPreferencesContext'
-import { useTutorial } from '@/contexts/TutorialContext'
+import { useUserProgress } from '@/contexts/UserProgressContext'
 import { useMemo } from 'react'
 import { PreferenceType, WikidataEntity } from '@/types'
 
@@ -20,7 +20,7 @@ export default function Home() {
     isAdvancedMode,
     setAdvancedMode,
   } = useUserPreferences()
-  const { hasCompletedBasicTutorial, hasCompletedAdvancedTutorial } = useTutorial()
+  const { hasCompletedBasicTutorial, hasCompletedAdvancedTutorial } = useUserProgress()
 
   // Determine where to route the user based on tutorial completion and advanced mode
   const { ctaHref, ctaText } = useMemo(() => {