From 28cef1e75fb15991c458431fed160c8cbe9dfccc Mon Sep 17 00:00:00 2001 From: Daniel Date: Fri, 11 Sep 2026 23:10:28 +0200 Subject: [PATCH] fix: one label for picking a session back up Start for an unsat session and Resume for a started one is a distinction the learner does not have to make. The session exists the moment it is created, so picking it up is resuming it whether or not a question has been answered yet. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01TqXevQJhxFrM7jJg82cgZN --- frontend/src/pages/AnalysisSessionPage.jsx | 5 ++++- frontend/src/pages/AnalysisSessionPage.test.jsx | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/frontend/src/pages/AnalysisSessionPage.jsx b/frontend/src/pages/AnalysisSessionPage.jsx index 1e68303..f73a2ed 100644 --- a/frontend/src/pages/AnalysisSessionPage.jsx +++ b/frontend/src/pages/AnalysisSessionPage.jsx @@ -199,8 +199,11 @@ export default function AnalysisSessionPage() { decision as far as possible from the result it follows from. */}
{data.not_started ? ( + // One label either way. The session exists the moment it is + // made, so picking it up is resuming it whether or not a + // question has been answered yet. - Start session + Resume session ) : ( <> diff --git a/frontend/src/pages/AnalysisSessionPage.test.jsx b/frontend/src/pages/AnalysisSessionPage.test.jsx index 7c12699..c6a4698 100644 --- a/frontend/src/pages/AnalysisSessionPage.test.jsx +++ b/frontend/src/pages/AnalysisSessionPage.test.jsx @@ -72,7 +72,7 @@ describe('a session nobody has sat', () => { it('offers the sitting, and nothing that makes no sense yet', async () => { mountQuiz() - expect(await screen.findByRole('link', { name: 'Start session' })) + expect(await screen.findByRole('link', { name: 'Resume session' })) .toHaveAttribute('href', '/study/3?start=1') expect(screen.queryByRole('link', { name: 'Review answers' })).not.toBeInTheDocument() expect(screen.queryByRole('button', { name: 'Delete session' })).not.toBeInTheDocument()