Add integration tests

Author: intmainale

backend/test/integration/conftest.py

@@ -0,0 +1,57 @@
+from unittest.mock import AsyncMock
+
+import pytest
+from fastapi.testclient import TestClient
+
+from adapters.input import create_fastapi_app
+from domain.models import LLMResult, ResultCode, ResultStatus
+
+
+@pytest.fixture
+def mock_text_processor():
+    class MockTextProcessor:
+        summarize = AsyncMock(
+            return_value=LLMResult(
+                status=ResultStatus.SUCCESS,
+                code=ResultCode.OK,
+                rewritten_text="summary output",
+                detected_language="it",
+            )
+        )
+        improve = AsyncMock(
+            return_value=LLMResult(
+                status=ResultStatus.SUCCESS,
+                code=ResultCode.OK,
+                rewritten_text="improved output",
+            )
+        )
+        translate = AsyncMock(
+            return_value=LLMResult(
+                status=ResultStatus.SUCCESS,
+                code=ResultCode.OK,
+                rewritten_text="translated output",
+                detected_language="en",
+            )
+        )
+        analyze_six_hats = AsyncMock(
+            return_value=LLMResult(
+                status=ResultStatus.SUCCESS,
+                code=ResultCode.OK,
+                rewritten_text="six hats output",
+            )
+        )
+        generate = AsyncMock(
+            return_value=LLMResult(
+                status=ResultStatus.SUCCESS,
+                code=ResultCode.OK,
+                rewritten_text="generated output",
+            )
+        )
+
+    return MockTextProcessor()
+
+
+@pytest.fixture
+def client(mock_text_processor):
+    app = create_fastapi_app(mock_text_processor)
+    return TestClient(app)

backend/test/integration/test_generate_flow.py

@@ -0,0 +1,27 @@
+def test_generate_flow_returns_llm_result_and_calls_processor(client, mock_text_processor):
+    response = client.post(
+        "/llm/generate",
+        json={
+            "prompt": "Scrivi una introduzione",
+            "context_text": "Contesto della nota",
+            "word_count": 180,
+        },
+    )
+
+    assert response.status_code == 200
+    assert response.json()["outcome"]["status"] == "success"
+    assert response.json()["data"]["rewritten_text"] == "generated output"
+    mock_text_processor.generate.assert_awaited_once_with(
+        "Scrivi una introduzione",
+        "Contesto della nota",
+        180,
+    )
+
+
+def test_generate_flow_validation_error_when_prompt_is_missing(client):
+    response = client.post(
+        "/llm/generate",
+        json={"context_text": "Contesto della nota", "word_count": 180},
+    )
+
+    assert response.status_code == 422

backend/test/integration/test_health_flow.py

@@ -0,0 +1,9 @@
+def test_health_flow_returns_server_status(client):
+    response = client.get("/health")
+
+    assert response.status_code == 200
+    assert response.json() == {
+        "status": "ok",
+        "message": "Server is awake!",
+        "architecture": "hexagonal",
+    }

backend/test/integration/test_improve_flow.py

@@ -0,0 +1,30 @@
+from domain.models import TextDocument
+
+
+def test_improve_flow_returns_llm_result_and_calls_processor(client, mock_text_processor):
+    response = client.post(
+        "/llm/improve",
+        json={"text": "Testo base", "criterion": "piu formale"},
+    )
+
+    assert response.status_code == 200
+    assert response.json()["outcome"]["status"] == "success"
+    assert response.json()["data"]["rewritten_text"] == "improved output"
+    mock_text_processor.improve.assert_awaited_once()
+
+    document_arg, criterion_arg = mock_text_processor.improve.await_args.args
+    assert isinstance(document_arg, TextDocument)
+    assert document_arg.content == "Testo base"
+    assert criterion_arg == "piu formale"
+
+
+def test_improve_flow_maps_value_error_to_400(client, mock_text_processor):
+    mock_text_processor.improve.side_effect = ValueError("bad criterion")
+
+    response = client.post(
+        "/llm/improve",
+        json={"text": "Testo base", "criterion": "bad"},
+    )
+
+    assert response.status_code == 400
+    assert response.json()["detail"] == "bad criterion"

backend/test/integration/test_six_hats_flow.py

@@ -0,0 +1,30 @@
+from domain.models import TextDocument
+
+
+def test_six_hats_flow_returns_llm_result_and_calls_processor(client, mock_text_processor):
+    response = client.post(
+        "/llm/six-hats",
+        json={"text": "Analizza questo testo", "hat": "Nero"},
+    )
+
+    assert response.status_code == 200
+    assert response.json()["outcome"]["status"] == "success"
+    assert response.json()["data"]["rewritten_text"] == "six hats output"
+    mock_text_processor.analyze_six_hats.assert_awaited_once()
+
+    document_arg, hat_arg = mock_text_processor.analyze_six_hats.await_args.args
+    assert isinstance(document_arg, TextDocument)
+    assert document_arg.content == "Analizza questo testo"
+    assert hat_arg == "Nero"
+
+
+def test_six_hats_flow_maps_runtime_error_to_500(client, mock_text_processor):
+    mock_text_processor.analyze_six_hats.side_effect = RuntimeError("provider down")
+
+    response = client.post(
+        "/llm/six-hats",
+        json={"text": "Analizza questo testo", "hat": "Nero"},
+    )
+
+    assert response.status_code == 500
+    assert "Errore del servizio AI: provider down" in response.json()["detail"]

backend/test/integration/test_summarize_flow.py

@@ -0,0 +1,24 @@
+from domain.models import TextDocument
+
+
+def test_summarize_flow_returns_llm_result_and_calls_processor(client, mock_text_processor):
+    response = client.post(
+        "/llm/summarize",
+        json={"text": "Contenuto da riassumere", "percentage": 50},
+    )
+
+    assert response.status_code == 200
+    assert response.json()["outcome"]["status"] == "success"
+    assert response.json()["data"]["rewritten_text"] == "summary output"
+    mock_text_processor.summarize.assert_awaited_once()
+
+    document_arg, percentage_arg = mock_text_processor.summarize.await_args.args
+    assert isinstance(document_arg, TextDocument)
+    assert document_arg.content == "Contenuto da riassumere"
+    assert percentage_arg == 50
+
+
+def test_summarize_flow_validation_error_when_text_is_missing(client):
+    response = client.post("/llm/summarize", json={"percentage": 40})
+
+    assert response.status_code == 422

backend/test/integration/test_translate_flow.py

@@ -0,0 +1,24 @@
+from domain.models import TextDocument
+
+
+def test_translate_flow_returns_llm_result_and_calls_processor(client, mock_text_processor):
+    response = client.post(
+        "/llm/translate",
+        json={"text": "Ciao mondo", "targetLanguage": "english"},
+    )
+
+    assert response.status_code == 200
+    assert response.json()["outcome"]["status"] == "success"
+    assert response.json()["data"]["rewritten_text"] == "translated output"
+    mock_text_processor.translate.assert_awaited_once()
+
+    document_arg, language_arg = mock_text_processor.translate.await_args.args
+    assert isinstance(document_arg, TextDocument)
+    assert document_arg.content == "Ciao mondo"
+    assert language_arg == "english"
+
+
+def test_translate_flow_validation_error_when_target_language_missing(client):
+    response = client.post("/llm/translate", json={"text": "Ciao mondo"})
+
+    assert response.status_code == 422

frontend/src/components/FileSidebar.integration.test.tsx

@@ -0,0 +1,117 @@
+import { render, screen, waitFor } from '@testing-library/react';
+import userEvent from '@testing-library/user-event';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { useState } from 'react';
+
+import FileSidebar from './FileSidebar';
+import { useNotesManager } from '../hooks/useNotesManager';
+
+const { dbGetMock, dbSetMock, fileImportMock, fileExportMock } = vi.hoisted(() => ({
+  dbGetMock: vi.fn(),
+  dbSetMock: vi.fn(),
+  fileImportMock: vi.fn(),
+  fileExportMock: vi.fn(),
+}));
+
+vi.mock('idb-keyval', () => ({
+  get: dbGetMock,
+  set: dbSetMock,
+}));
+
+vi.mock('../services/fileService', () => ({
+  fileService: {
+    importFile: fileImportMock,
+    exportFile: fileExportMock,
+  },
+}));
+
+function FileSidebarExportHarness() {
+  const [snackbar, setSnackbar] = useState<{
+    open: boolean;
+    message: string;
+    severity: 'success' | 'error';
+  }>({
+    open: false,
+    message: '',
+    severity: 'success',
+  });
+
+  const {
+    notes,
+    activeNoteId,
+    setActiveNoteId,
+    isLoaded,
+    handleCreateNote,
+    handleDeleteNote,
+    handleRenameNote,
+    handleImportNote,
+    handleExportNote,
+  } = useNotesManager(setSnackbar);
+
+  if (!isLoaded) {
+    return <div>Caricamento note...</div>;
+  }
+
+  const activeNote = notes.find((note) => note.id === activeNoteId);
+
+  return (
+    <>
+      <FileSidebar
+        notes={notes}
+        activeId={activeNoteId}
+        onSelect={setActiveNoteId}
+        onCreate={handleCreateNote}
+        onDelete={handleDeleteNote}
+        onRename={handleRenameNote}
+        onImport={handleImportNote}
+        onExport={handleExportNote}
+      />
+      <h2>{activeNote?.title}</h2>
+      <p>{snackbar.open ? snackbar.message : ''}</p>
+    </>
+  );
+}
+
+describe('FileSidebar integration (export flow)', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    dbSetMock.mockResolvedValue(undefined);
+    fileImportMock.mockResolvedValue(null);
+    fileExportMock.mockResolvedValue(undefined);
+  });
+
+  it('esporta la nota attiva quando clicco il bottone salva su disco', async () => {
+    const user = userEvent.setup();
+    dbGetMock.mockResolvedValue([
+      { id: 'note-1', title: 'Nota Uno', content: 'Contenuto uno', createdAt: 1 },
+    ]);
+
+    render(<FileSidebarExportHarness />);
+
+    await screen.findByRole('heading', { name: 'Nota Uno' });
+    await user.click(screen.getByTitle(/salva nota su disco/i));
+
+    await waitFor(() =>
+      expect(fileExportMock).toHaveBeenCalledWith('Nota Uno', 'Contenuto uno'),
+    );
+  });
+
+  it('se cambio nota attiva dalla lista, esporta la nuova nota selezionata', async () => {
+    const user = userEvent.setup();
+    dbGetMock.mockResolvedValue([
+      { id: 'note-1', title: 'Nota Uno', content: 'Contenuto uno', createdAt: 1 },
+      { id: 'note-2', title: 'Nota Due', content: 'Contenuto due', createdAt: 2 },
+    ]);
+
+    render(<FileSidebarExportHarness />);
+
+    await screen.findByRole('heading', { name: 'Nota Uno' });
+    await user.click(screen.getByText('Nota Due'));
+    await screen.findByRole('heading', { name: 'Nota Due' });
+
+    await user.click(screen.getByTitle(/salva nota su disco/i));
+    await waitFor(() =>
+      expect(fileExportMock).toHaveBeenCalledWith('Nota Due', 'Contenuto due'),
+    );
+  });
+});