fix(aluno): finish-button hint + duration clamp (v2, rebased on #182) (#184)

* fix(aluno): hint when finish button disabled by zero checks

CardTreino footer ganha <p> condicional data-testid finish-hint
quando completedCount === 0 e sem feedback loading. Sinaliza
pre-condicao (marque exercicios) para o botao Finalizar e Avaliar
Treino que parecia morto quando disabled.

Test: stateful mock useWorkoutTracker para toggle. 15/15 card-treino,
1134/1134 suite.

Co-Authored-By: Claude <noreply@anthropic.com>

* fix(workout): clamp duracaoMinutos to min 1 for Zod schema

HistoricoTreinoBaseSchema requires duracaoMinutos >= 1. Quick workouts
(<30s) rounded to 0, failing save and surfacing "ocorreu um erro
inesperado ao salvar no histórico" toast.

Co-Authored-By: Claude <noreply@anthropic.com>

* fix(aluno): address cubic P2 — aria-live, duration test, simpler mock

- card-treino.tsx: finish-hint gains aria-live="polite" so AT announces
  reappearance when user unchecks last completed exercise.
- WorkoutSession.test.tsx: new test asserting onFinish receives
  duracaoMinutos >= 1 even on sub-minute session (regression guard for
  the Math.max(1, ...) clamp).
- card-treino.test.tsx: simplify "hides finish hint" test per cubic —
  set mockReturnValue directly to { 'ex-1': true } + rerender; drop
  mutable let, callback re-mock, fireEvent.click, silent no-op path.

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Emiya Kiritsugu <emiyakiritsugu3@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>

Author: EmiyaKiritsugu3

src/components/WorkoutSession.test.tsx

@@ -195,6 +195,25 @@ describe('WorkoutSession', () => {
     });
   });
 
+  it('clamps duracaoMinutos to at least 1 on a very short session', async () => {
+    const { generateWorkoutFeedback } = await import('@/ai/flows/workout-feedback-flow');
+    vi.mocked(generateWorkoutFeedback).mockResolvedValue({
+      title: 'Treino Concluído!',
+      message: 'Excelente trabalho!',
+    });
+
+    render(<WorkoutSession treino={mockTreino} onFinish={mockOnFinish} onCancel={mockOnCancel} />);
+    // Finish immediately — elapsed < 1 min, raw Math.round would yield 0
+    fireEvent.click(screen.getByText(/Próximo/));
+    fireEvent.click(screen.getByText(/Finalizar Treino/));
+
+    await waitFor(() => {
+      expect(mockOnFinish).toHaveBeenCalled();
+    });
+    const historico = mockOnFinish.mock.calls[0][0];
+    expect(historico.duracaoMinutos).toBeGreaterThanOrEqual(1);
+  });
+
   it('shows loading state while generating feedback', async () => {
     const { generateWorkoutFeedback } = await import('@/ai/flows/workout-feedback-flow');
     let resolveFeedback!: (v: { title: string; message: string }) => void;

src/components/WorkoutSession.tsx

@@ -170,7 +170,10 @@ export function WorkoutSession({ treino, onFinish, onCancel }: Readonly<WorkoutS
     if (isFinishing) return;
     setIsFinishing(true);
     const endTime = new Date();
-    const duracaoMinutos = Math.round((endTime.getTime() - startTime.getTime()) / 60000);
+    const duracaoMinutos = Math.max(
+      1,
+      Math.round((endTime.getTime() - startTime.getTime()) / 60000)
+    );
 
     const historico: Omit<HistoricoTreino, 'id' | 'alunoId'> = {
       treinoId: treino.id,

src/components/dashboard/aluno/card-treino.test.tsx

@@ -212,6 +212,47 @@ describe('CardTreino', () => {
     expect(button?.disabled).toBe(true);
   });
 
+  it('shows finish hint when no exercises checked', () => {
+    render(
+      <CardTreino
+        treino={mockTreino}
+        onFinishTraining={mockOnFinishTraining}
+        isFeedbackLoading={false}
+        onViewExercicio={mockOnViewExercicio}
+      />
+    );
+    expect(screen.getByTestId('finish-hint')).toBeTruthy();
+    expect(
+      screen.getByText('Marque os exercícios concluídos acima para finalizar o treino.')
+    ).toBeTruthy();
+  });
+
+  it('hides finish hint when an exercise is checked', () => {
+    const { rerender } = render(
+      <CardTreino
+        treino={mockTreino}
+        onFinishTraining={mockOnFinishTraining}
+        isFeedbackLoading={false}
+        onViewExercicio={mockOnViewExercicio}
+      />
+    );
+    expect(screen.getByTestId('finish-hint')).toBeTruthy();
+
+    mockUseWorkoutTracker.mockReturnValue({
+      checkedExercises: { 'ex-1': true },
+      handleCheckChange: vi.fn(),
+    });
+    rerender(
+      <CardTreino
+        treino={mockTreino}
+        onFinishTraining={mockOnFinishTraining}
+        isFeedbackLoading={false}
+        onViewExercicio={mockOnViewExercicio}
+      />
+    );
+    expect(screen.queryByTestId('finish-hint')).toBeNull();
+  });
+
   it('shows loading state when feedback is loading', () => {
     render(
       <CardTreino

src/components/dashboard/aluno/card-treino.tsx

@@ -123,7 +123,7 @@ export function CardTreino({
           </div>
         ))}
       </CardContent>
-      <CardFooter className="bg-white/5 p-6 border-t border-white/5">
+      <CardFooter className="bg-white/5 p-6 border-t border-white/5 flex flex-col gap-3">
         <Button
           variant="premium"
           size="lg"
@@ -143,6 +143,15 @@ export function CardTreino({
             </div>
           )}
         </Button>
+        {completedCount === 0 && !isFeedbackLoading && (
+          <p
+            data-testid="finish-hint"
+            aria-live="polite"
+            className="text-sm text-muted-foreground text-center"
+          >
+            Marque os exercícios concluídos acima para finalizar o treino.
+          </p>
+        )}
       </CardFooter>
     </Card>
   );