Add unit tests for QuizSpreadsheetService

7 tests covering generateTemplate(), quizToXlsx(), and xlsxToQuiz():
- valid XLSX output and MIME type
- template without example reimports as empty
- template example data survives a reimport
- round-trip (export → reimport) preserves questions, answers, and correct flags
- empty quiz exports and reimports cleanly
- invalid MIME type throws InvalidArgumentException
- question with no answers throws SpreadsheetDataException with error list

.gitignore

@@ -72,10 +72,8 @@ Icon
 /.idea/
 /.vscode/
 
-# Junie
+# Claude Code
-!/.junie/
+/.claude/settings.local.json
-/.junie/memory/
-/.junie/plans/
 
 # Windows
 Thumbs.db

.junie/AGENTS.md

@@ -1,82 +0,0 @@
-# Agent Guide: Tijd Voor De Test (Tvdt)
-
-This document provides essential context and instructions for AI agents working on the **Tijd Voor De Test** project.
-
-## Project Overview
-A web application for managing "Wie is de Mol?" style tests, including seasons, quizzes, candidates, and eliminations.
-
-- **Namespace**: `Tvdt`
-- **PHP Version**: 8.5+
-- **Framework**: Symfony 8.0
-
-## Tech Stack
-- **Server**: FrankenPHP (Caddy-based PHP server)
-- **Database**: PostgreSQL
-- **Frontend**: Symfony Asset Mapper (no Node.js/Webpack), Stimulus, Turbo
-- **Styling**: Sass (via `symfonycasts/sass-bundle`)
-- **Persistence**: Doctrine ORM 3.x
-
-## Core Domain Entities
-- **Season**: Groups quizzes and candidates for a specific period.
-- **SeasonSettings**: Configuration for a season.
-- **Quiz**: A test within a season containing multiple questions.
-- **Question**: Questions belonging to a quiz.
-- **Answer**: Possible answers for a question.
-- **Candidate**: A participant in the season.
-- **QuizCandidate**: Represents a candidate's attempt at a specific quiz (tracking start/end time).
-- **GivenAnswer**: The specific answer a candidate selected during a quiz.
-- **Elimination**: Records of red/green screens and forced results.
-- **User**: Administrative accounts for managing the system.
-
-## Development Workflow
-The project uses `just` as the primary task runner. Always prefer `just` commands over manual docker calls.
-
-### Common Commands
-- `just up`: Start the environment.
-- `just down`: Stop the environment.
-- `just shell`: Enter the PHP container.
-- `just migrate`: Run database migrations.
-- `just fixtures`: Load development fixtures.
-- `just fix-cs`: Run `php-cs-fixer` and `twig-cs-fixer`.
-- `just phpstan`: Run static analysis.
-- `just rector`: Run Rector for automated refactorings.
-- `just reload-tests`: Reset the test database and load test fixtures.
-
-## Coding Standards
-- **PSR-12**: Follow standard PHP coding styles.
-- **Strict Typing**: Use strict types in all PHP files.
-- **Doctrine ORM 3**: Be aware of ORM 3 changes (e.g., lazy loading behavior, attribute-based mapping).
-- **Symfony 8**: Use modern Symfony features (Attributes, Type-hinting).
-- **Safe Functions**: Use `thecodingmachine/safe` for standard PHP functions that throw exceptions instead of returning false.
-
-## Testing
-- **Framework**: PHPUnit
-- **Bundle**: `dama/doctrine-test-bundle` is used to wrap tests in transactions.
-- **Location**: `tests/` directory mirroring `src/`.
-- **Execution**: Run via `bin/phpunit` inside the container or `just reload-tests` to prepare the environment.
-
-## Frontend Development
-- JavaScript is managed via **Import Maps**.
-- Stimulus controllers are located in `assets/controllers/`.
-- CSS/Sass is in `assets/styles/`.
-- Assets are compiled on-the-fly or mapped; do not look for a `node_modules` folder.
-
-## Key Components
-### Controllers
-- **Backoffice**: Located in `src/Controller/Backoffice`, handles season and quiz management.
-- **Quiz**: `src/Controller/QuizController` handles the candidate-facing side of quizzes.
-- **Elimination**: `src/Controller/EliminationController` handles elimination screens.
-
-### Services
-- **QuizSpreadsheetService**: Handles importing quizzes from XLSX files.
-
-### Base Classes & Enums
-- **AbstractController**: Base class for all controllers, containing common regexes and flash helpers.
-- **FlashType Enum**: Used for consistent flash messaging (`FlashType::Success`, `FlashType::Danger`, etc.).
-
-## Key Files
-- `composer.json`: Dependency management.
-- `importmap.php`: JavaScript module mapping.
-- `Justfile`: Automation shortcuts.
-- `config/`: Application configuration.
-- `templates/`: Twig templates.

CLAUDE.md

@@ -0,0 +1,232 @@
+# CLAUDE.md
+
+This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
+
+## Project Overview
+
+**Tijd voor de test** is a PHP/Symfony 8.1 application for managing quizzes in the style of **Wie is de Mol?** (WIDM) — a Dutch TV show where contestants try to identify a saboteur ("de Mol") among them. At the end of each episode, participants take a quiz about the Mol's identity and actions; the candidate with the least correct answers is eliminated. This app replicates that quiz format with:
+- Test creation with variable question counts
+- Season management with active test controls
+- Candidate answer tracking with automatic timing
+- Elimination tracking with joker adjustments
+- Backoffice management for quiz administration and statistics
+
+Tech Stack:
+- **Framework**: Symfony 8.1
+- **PHP**: 8.5+
+- **Database**: PostgreSQL 16
+- **ORM**: Doctrine
+- **Server**: FrankenPHP with Caddy
+- **Container**: Docker Compose
+- **Frontend**: Twig templates with SASS (via asset mapper)
+- **Testing**: PHPUnit 13 with DAMA Doctrine test bundle
+
+## Build & Development Commands
+
+All commands assume Docker is running. The project uses a [Justfile](https://just.systems) as the primary interface.
+
+### Essential Commands
+
+```bash
+just up           # Start Docker services (PHP, PostgreSQL)
+just stop         # Stop services
+just down         # Stop and remove containers/orphans
+just shell        # Interactive shell inside the PHP container
+just shell-run    # Shell in a fresh one-off container
+```
+
+### Database
+
+```bash
+just migrate                  # Run Doctrine migrations (starts services first)
+just fixtures                 # Load dev fixtures (truncates first)
+just reload-tests             # Drop/recreate test DB, migrate, load test fixtures
+```
+
+### Testing
+
+```bash
+just test                                  # Run full PHPUnit suite
+just test tests/Path/To/TestFile.php       # Run a specific test file
+just test --coverage-html var/coverage     # Generate HTML coverage report
+```
+
+### Code Quality & Linting
+
+```bash
+just fix-cs                        # Auto-fix PHP-CS-Fixer + Twig-CS-Fixer
+just phpstan                       # PHPStan static analysis (level 9)
+just phpstan --no-progress         # Without progress output
+just rector                        # Apply Rector modernizations
+just rector --dry-run              # Preview Rector changes
+```
+
+### Other
+
+```bash
+just translations   # Extract/update nl translation strings
+just clean          # Nuke containers (volumes) + all generated files (prompts for confirmation)
+just trust-cert     # Trust the local Caddy TLS certificate (macOS)
+just exec <cmd>     # Run any command inside the PHP container
+```
+
+All code quality checks run in CI/CD (.github/workflows/ci.yml) and should pass before merging.
+
+## Project Structure
+
+```
+src/
+  Controller/              # HTTP request handlers (attribute-routed)
+    Backoffice/           # Admin panel controllers
+  Entity/                 # Doctrine ORM entities
+  Repository/             # Database queries
+  Service/                # Business logic
+  Command/                # CLI commands
+  Form/                   # Symfony form types
+  Dto/                    # Data transfer objects
+  Enum/                   # Enumerations (FlashType, etc.)
+  Exception/              # Custom exceptions
+  Factory/                # Object factories
+  Helpers/                # Utility functions
+  Security/               # Auth and voter classes
+    Voter/                # Authorization voters
+  DataFixtures/           # Test data loaders
+
+config/
+  packages/               # Symfony bundle configurations
+  routes/                 # Route definitions
+  services.yaml           # Service container configuration
+  routes.yaml             # Main route entry point
+
+templates/
+  backoffice/             # Admin UI templates
+  quiz/                   # Public quiz UI templates
+  base.html.twig          # Main layout
+
+tests/
+  Command/                # Command tests
+  Controller/             # Controller/integration tests
+  Repository/             # Repository tests
+  Security/               # Auth tests
+  Helpers/                # Utility tests
+  bootstrap.php           # PHPUnit bootstrap with test container setup
+```
+
+## Core Domain Entities
+
+- **Season**: Groups quizzes and candidates for a specific period, with a linked `SeasonSettings`.
+- **Quiz**: A test within a season containing multiple `Question`s, each with multiple `Answer`s.
+- **Candidate**: A participant in the season.
+- **QuizCandidate**: Represents a candidate's attempt at a specific quiz (tracks start/end time).
+- **GivenAnswer**: The specific answer a candidate selected during a quiz.
+- **Elimination**: Records red/green screens and forced results with joker adjustments.
+- **User**: Administrative accounts for managing the system.
+
+## Architecture Notes
+
+### Routing
+- Routes are **attribute-based** (PHP 8 attributes in controller methods)
+- Configured in `config/routes/attributes.yaml` for automatic discovery
+- Main entry point: `config/routes.yaml`
+
+### Service Container & Dependency Injection
+- Services in `src/` are automatically registered via PSR-4 namespace `Tvdt\`
+- Exclusions: Entity, DependencyInjection, Kernel classes
+- Autowiring and autoconfiguration enabled by default
+- Service definitions in `config/services.yaml`
+
+### Database & Migrations
+- PostgreSQL-based with Doctrine ORM
+- Migrations in `migrations/` at project root, namespace `DoctrineMigrations` (intentionally not autoloaded); generate with `bin/console make:migration`
+- Test fixtures in `src/DataFixtures/` (loaded with `--group=test`)
+- Test database configured separately via `.env.test`
+
+### Testing Infrastructure
+- **PHPUnit 13** with DAMA Doctrine Test Bundle for transaction rollback
+- Bootstrap: `tests/bootstrap.php` loads env vars and autoloader; `tests/symfony-container.php` boots the test kernel/container (used by Rector)
+- Symfony test utilities (BrowserKit, CSS selectors) available
+- Coverage excluded from: `src/DataFixtures/`
+- Test environment: `APP_ENV=test` (set in phpunit.dist.xml)
+
+### Code Style & Standards
+- **PHP-CS-Fixer**: Symfony ruleset + risky rules enabled
+  - Strict types declaration required
+  - Trailing commas in multiline structures
+  - No else-only blocks
+- **Rector**: Aggressive modernization with all attribute sets + prepared sets (dead code, code quality, Doctrine, Symfony, PHPUnit)
+- **PHPStan**: Level 8 with extensions for Doctrine and Symfony
+- **Twig-CS-Fixer**: Template style enforcement
+- **Safe functions**: Use `thecodingmachine/safe` wrappers for standard PHP functions that return `false` on failure — they throw exceptions instead
+
+### Environment Configuration
+- `.env` - Local development defaults (uncommitted in .env.local)
+- `.env.dev` - Development overrides
+- `.env.test` - Test environment configuration
+- Production uses `composer dump-env prod` for compiled configuration
+- Key variables:
+  - `APP_ENV` - Environment (dev/test/prod)
+  - `DATABASE_URL` - PostgreSQL connection string
+  - `MAILER_SENDER` - From address for emails
+
+### Frontend Build
+- Asset mapper (no Node.js/Webpack) for JS/CSS bundling; JS modules declared in `importmap.php`
+- **Stimulus** controllers in `assets/controllers/`, **Turbo** for SPA-like navigation
+- Sass sources in `assets/styles/`, compiled via `bin/console sass:build`
+- Production: Assets precompiled during Docker build
+- Development: Watch mode enabled in FrankenPHP container
+
+## CI/CD Pipeline
+
+GitHub Actions workflow (`.github/workflows/ci.yml`):
+
+1. **Linting**: Dockerfile (hadolint), Twig templates
+2. **Code Quality**:
+   - PHP-CS-Fixer style check
+   - Twig-CS-Fixer style check
+   - PHPStan static analysis
+   - Rector dry-run
+3. **Integration Tests**:
+   - Docker image build and start services
+   - Database creation and migration
+   - Fixture loading
+   - Full PHPUnit test suite with JUnit XML output
+   - Doctrine schema validation
+4. **Build & Deploy** (on tags or main, disabled currently):
+   - Docker image push to GitHub Container Registry
+   - Sentry release creation
+   - Portainer webhook trigger for production deployment
+
+Runs on all pushes to main and pull requests. Concurrency cancels old runs on new commits.
+
+## Important Files & Conventions
+
+- **Kernel**: `src/Kernel.php` - Symfony kernel class
+- **AbstractController**: Base class for all controllers — defines route parameter regexes (`SEASON_CODE_REGEX`, `CANDIDATE_HASH_REGEX`) and flash helpers
+- **Flash Messages**: Use `FlashType` enum instead of string literals
+- **QuizSpreadsheetService**: Handles importing quizzes from XLSX files
+- **Rector container**: `tests/symfony-container.php` — boots a test kernel so Rector can resolve Symfony service types
+- **.gitignore**: Excludes var/, vendor/, .env.local, .phpunit.cache
+- **Dockerfile**: Multi-stage build with dev/prod separation, FrankenPHP-based
+- **Docker Compose**: PHP service with Caddy, PostgreSQL database, persistent volumes
+
+## Security & Authorization
+
+- Doctrine extensions enabled (timestamps, slugs, etc.)
+- Voter-based authorization in `src/Security/Voter/`
+- User entity with security encoding configured
+- CSRF protection enabled
+- Email verification available via SymfonyCasts bundle
+
+## Composer Scripts
+
+Auto-executed scripts on install/update:
+- `cache:clear` - Symfony cache clear
+- `assets:install` - Copy public assets
+- `importmap:install` - JS import map setup
+
+## Notes for Future Work
+
+- The backoffice elimination logic is in `Controller/Backoffice/PrepareEliminationController.php`
+- Quiz timing logic starts on candidate start click and stops on final answer selection
+- Background music feature noted but not yet implemented (requirements only)
+- Statistics functionality is marked TBD in README

Makefile

@@ -1,16 +0,0 @@
-.DEFAULT_GOAL := help
-
-.PHONY: up
-up: ## Start application
-	@docker compose up -d
-
-stop: ## Stop application
-	@docker compose stop
-
-.PHONY: shell
-shell: ## Start a shell inside the container
-	@docker compose exec php bash
-
-.PHONY: help
-help:
-	@grep -E '^[a-zA-Z_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-10s\033[0m %s\n", $$1, $$2}'

assets/styles/backoffice.scss

@@ -0,0 +1,3 @@
+.col-result-xs { width: 10%; }
+.col-result-sm { width: 15%; }
+.col-result-md { width: 20%; }

assets/styles/quiz.scss

@@ -1,14 +1,17 @@
-html, body {
+html {
     height: 100%;
+}
+
+body {
+    min-height: 100%;
     background-image: url("../img/background.png");
-    background-position: center center;
+    background-position: center;
     background-repeat: no-repeat;
     background-color: black;
     color: white;
-
+    display: flex;
-    display: grid;
+    flex-direction: column;
-    align-items: center;
+    justify-content: center;
-    justify-self: center;
 }
 
 .elimination-screen {

config/reference.php

@@ -127,7 +127,7 @@ use Symfony\Component\Config\Loader\ParamConfigurator as Param;
  * }
  * @psalm-type ServicesConfig = array{
  *     _defaults?: DefaultsType,
- *     _instanceof?: InstanceofType,
+ *     _instanceof?: array<class-string, InstanceofType>,
  *     ...<string, DefinitionType|AliasType|PrototypeType|StackType|ArgumentsType|null>
  * }
  * @psalm-type ExtensionType = array<string, mixed>
@@ -727,7 +727,7 @@ use Symfony\Component\Config\Loader\ParamConfigurator as Param;
  *             servicename?: scalar|Param|null, // Overrules dbname parameter if given and used as SERVICE_NAME or SID connection parameter for Oracle depending on the service parameter.
  *             sessionMode?: scalar|Param|null, // The session mode to use for the oci8 driver
  *             server?: scalar|Param|null, // The name of a running database server to connect to for SQL Anywhere.
- *             default_dbname?: scalar|Param|null, // Override the default database (postgres) to connect to for PostgreSQL connexion.
+ *             default_dbname?: scalar|Param|null, // Override the default database (postgres) to connect to for PostgreSQL connection.
  *             sslmode?: scalar|Param|null, // Determines whether or with what priority a SSL TCP/IP connection will be negotiated with the server for PostgreSQL.
  *             sslrootcert?: scalar|Param|null, // The name of a file containing SSL certificate authority (CA) certificate(s). If the file exists, the server's certificate will be verified to be signed by one of these authorities.
  *             sslcert?: scalar|Param|null, // The path to the SSL client certificate file for PostgreSQL.
@@ -773,7 +773,7 @@ use Symfony\Component\Config\Loader\ParamConfigurator as Param;
  *                 servicename?: scalar|Param|null, // Overrules dbname parameter if given and used as SERVICE_NAME or SID connection parameter for Oracle depending on the service parameter.
  *                 sessionMode?: scalar|Param|null, // The session mode to use for the oci8 driver
  *                 server?: scalar|Param|null, // The name of a running database server to connect to for SQL Anywhere.
- *                 default_dbname?: scalar|Param|null, // Override the default database (postgres) to connect to for PostgreSQL connexion.
+ *                 default_dbname?: scalar|Param|null, // Override the default database (postgres) to connect to for PostgreSQL connection.
  *                 sslmode?: scalar|Param|null, // Determines whether or with what priority a SSL TCP/IP connection will be negotiated with the server for PostgreSQL.
  *                 sslrootcert?: scalar|Param|null, // The name of a file containing SSL certificate authority (CA) certificate(s). If the file exists, the server's certificate will be verified to be signed by one of these authorities.
  *                 sslcert?: scalar|Param|null, // The path to the SSL client certificate file for PostgreSQL.
@@ -852,7 +852,7 @@ use Symfony\Component\Config\Loader\ParamConfigurator as Param;
  *                     lock_path?: scalar|Param|null, // Default: "%kernel.cache_dir%/doctrine/orm/slc/filelock"
  *                     lock_lifetime?: scalar|Param|null, // Default: 60
  *                     type?: scalar|Param|null, // Default: "default"
- *                     lifetime?: scalar|Param|null, // Default: 0
+ *                     lifetime?: scalar|Param|null, // Default: null
  *                     service?: scalar|Param|null,
  *                     name?: scalar|Param|null,
  *                 }>,

src/Controller/Backoffice/BackofficeController.php

@@ -11,12 +11,15 @@ use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpFoundation\StreamedResponse;
 use Symfony\Component\HttpKernel\Attribute\AsController;
 use Symfony\Component\Routing\Attribute\Route;
+use Symfony\Component\Routing\Requirement\Requirement;
 use Symfony\Component\Security\Http\Attribute\IsGranted;
 use Tvdt\Controller\AbstractController;
+use Tvdt\Entity\Quiz;
 use Tvdt\Entity\Season;
 use Tvdt\Entity\User;
 use Tvdt\Form\CreateSeasonFormType;
 use Tvdt\Repository\SeasonRepository;
+use Tvdt\Security\Voter\SeasonVoter;
 use Tvdt\Service\QuizSpreadsheetService;
 
 #[AsController]
@@ -78,4 +81,20 @@ final class BackofficeController extends AbstractController
 
         return $response;
     }
+
+    #[IsGranted(SeasonVoter::EDIT, subject: 'quiz')]
+    #[Route(
+        '/backoffice/quiz/{quiz}/export',
+        name: 'tvdt_backoffice_quiz_export',
+        requirements: ['quiz' => Requirement::UUID],
+        methods: ['GET'],
+    )]
+    public function exportQuiz(Quiz $quiz): StreamedResponse
+    {
+        $response = new StreamedResponse($this->excel->quizToXlsx($quiz));
+        $response->headers->set('Content-Type', 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet');
+        $response->headers->set('Content-Disposition', 'attachment; filename="'.$quiz->name.'.xlsx"');
+
+        return $response;
+    }
 }

src/Entity/GivenAnswer.php

@@ -31,14 +31,14 @@ class GivenAnswer
     public function __construct(
         #[ORM\JoinColumn(nullable: false)]
         #[ORM\ManyToOne(inversedBy: 'givenAnswers')]
-        private(set) Candidate $candidate,
+        public private(set) Candidate $candidate,
 
         #[ORM\JoinColumn(nullable: false, onDelete: 'CASCADE')]
         #[ORM\ManyToOne]
-        private(set) Quiz $quiz,
+        public private(set) Quiz $quiz,
 
         #[ORM\JoinColumn(nullable: false)]
         #[ORM\ManyToOne(inversedBy: 'givenAnswers')]
-        private(set) Answer $answer,
+        public private(set) Answer $answer,
     ) {}
 }

src/Service/QuizSpreadsheetService.php

@@ -120,9 +120,47 @@ class QuizSpreadsheetService
         }
     }
 
-    public function quizToXlsx(Quiz $quiz): void
+    public function quizToXlsx(Quiz $quiz): \Closure
     {
-        throw new \Exception('Not implemented');
+        $spreadsheet = new Spreadsheet();
+        $sheet = $spreadsheet->getActiveSheet();
+
+        $sheet->getStyle('1:1')->getFont()->setBold(true);
+
+        $sheet->setCellValue('A1', 'Question');
+        $sheet->getColumnDimension('A')->setWidth(30);
+        $sheet->getStyle('A:A')->getAlignment()->setWrapText(true);
+
+        $counter = 1;
+        foreach (range('B', 'L', 2) as $column) {
+            $sheet->setCellValue($column.'1', 'Answer '.$counter++);
+            $sheet->getColumnDimension($column)->setWidth(30);
+            $sheet->getStyle($column.':'.$column)->getAlignment()->setWrapText(true);
+        }
+
+        foreach (range('C', 'M', 2) as $column) {
+            $sheet->setCellValue($column.'1', 'Correct');
+            $sheet->getColumnDimension($column)->setAutoSize(true);
+        }
+
+        $answerColumns = range('B', 'L', 2);
+        $correctColumns = range('C', 'M', 2);
+
+        $row = 2;
+        foreach ($quiz->questions as $question) {
+            $sheet->setCellValue('A'.$row, $question->question);
+
+            $col = 0;
+            foreach ($question->answers as $answer) {
+                $sheet->setCellValue($answerColumns[$col].$row, $answer->text);
+                $sheet->setCellValue($correctColumns[$col].$row, $answer->isRightAnswer);
+                ++$col;
+            }
+
+            ++$row;
+        }
+
+        return $this->toXlsx($spreadsheet);
     }
 
     private function toXlsx(Spreadsheet $spreadsheet): \Closure

templates/backoffice/quiz/tab_overview.html.twig

@@ -1,3 +1,27 @@
+{% macro confirm_modal(id, target, body, formAction, csrfToken) %}
+    <div class="modal fade" id="{{ id }}" data-bs-backdrop="static"
+         tabindex="-1"
+         data-bo--quiz-target="{{ target }}"
+         aria-labelledby="{{ id }}Label" aria-hidden="true">
+        <div class="modal-dialog">
+            <div class="modal-content">
+                <div class="modal-header">
+                    <h1 class="modal-title fs-5" id="{{ id }}Label">{{ 'Please Confirm'|trans }}</h1>
+                    <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
+                </div>
+                <div class="modal-body">{{ body }}</div>
+                <div class="modal-footer">
+                    <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">{{ 'No'|trans }}</button>
+                    <form action="{{ formAction }}" method="POST">
+                        <input type="hidden" name="_token" value="{{ csrfToken }}">
+                        <button type="submit" class="btn btn-danger">{{ 'Yes'|trans }}</button>
+                    </form>
+                </div>
+            </div>
+        </div>
+    </div>
+{% endmacro %}
+
 <div data-controller="bo--quiz">
     <h4 class="mb-3">{{ 'Quick actions'|trans }}</h4>
     <div class="mb-3 btn-group">
@@ -25,6 +49,11 @@
         </button>
     </div>
 
+    <a class="btn btn-outline-secondary mb-3"
+       href="{{ path('tvdt_backoffice_quiz_export', {quiz: quiz.id}) }}">
+        {{ 'Export to XLSX'|trans }}
+    </a>
+
     <h4 class="mb-3">{{ 'Questions'|trans }}</h4>
     <div class="accordion">
         {%~ for question in quiz.questions ~%}
@@ -66,53 +95,19 @@
         {% endfor %}
     </div>
 
-    {# Modal Clear #}
+    {{ _self.confirm_modal(
-    <div class="modal fade" id="clearQuizModal" data-bs-backdrop="static"
+        'clearQuizModal',
-         tabindex="-1"
+        'clearModal',
-         data-bo--quiz-target="clearModal"
+        'Are you sure you want to clear all the results? This will also delete all the eliminations.'|trans,
-         aria-labelledby="staticBackdropLabel" aria-hidden="true">
+        path('tvdt_backoffice_quiz_clear', {quiz: quiz.id}),
-        <div class="modal-dialog">
+        csrf_token('clear_quiz'),
-            <div class="modal-content">
+    ) }}
-                <div class="modal-header">
-                    <h1 class="modal-title fs-5" id="staticBackdropLabel">{{ 'Please Confirm'|trans }}</h1>
-                    <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
-                </div>
-                <div class="modal-body">
-                    {{ 'Are you sure you want to clear all the results? This will also delete all the eliminations.'|trans }}
-                </div>
-                <div class="modal-footer">
-                    <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">{{ 'No'|trans }}</button>
-                    <form action="{{ path('tvdt_backoffice_quiz_clear', {quiz: quiz.id}) }}" method="POST">
-                        <input type="hidden" name="_token" value="{{ csrf_token('clear_quiz') }}">
-                        <button type="submit" class="btn btn-danger">{{ 'Yes'|trans }}</button>
-                    </form>
-                </div>
-            </div>
-        </div>
-    </div>
 
-    {# Modal Delete #}
+    {{ _self.confirm_modal(
-    <div class="modal fade" id="deleteQuizModal" data-bs-backdrop="static"
+        'deleteQuizModal',
-         tabindex="-1"
+        'deleteModal',
-         data-bo--quiz-target="deleteModal"
+        'Are you sure you want to delete this quiz?'|trans,
-         aria-labelledby="staticBackdropLabel" aria-hidden="true">
+        path('tvdt_backoffice_quiz_delete', {quiz: quiz.id}),
-        <div class="modal-dialog">
+        csrf_token('delete_quiz'),
-            <div class="modal-content">
+    ) }}
-                <div class="modal-header">
-                    <h1 class="modal-title fs-5" id="staticBackdropLabel">{{ 'Please Confirm'|trans }}</h1>
-                    <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
-                </div>
-                <div class="modal-body">
-                    {{ 'Are you sure you want to delete this quiz?'|trans }}
-                </div>
-                <div class="modal-footer">
-                    <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">{{ 'No'|trans }}</button>
-                    <form action="{{ path('tvdt_backoffice_quiz_delete', {quiz: quiz.id}) }}" method="POST">
-                        <input type="hidden" name="_token" value="{{ csrf_token('delete_quiz') }}">
-                        <button type="submit" class="btn btn-danger">{{ 'Yes'|trans }}</button>
-                    </form>
-                </div>
-            </div>
-        </div>
-    </div>
 </div>

templates/backoffice/quiz/tab_result.html.twig

@@ -1,7 +1,6 @@
 <h4 class="mb-3">{{ 'Score'|trans }}</h4>
 <div class="btn-toolbar mb-3" role="toolbar">
     <div class="btn-group me-2">
-        {# <a class="btn btn-primary">{{ 'Start Elimination'|trans }}</a> #}
         <form action="{{ path('tvdt_prepare_elimination', {seasonCode: season.seasonCode, quiz: quiz.id}) }}" method="POST">
             <input type="hidden" name="_token" value="{{ csrf_token('prepare_elimination') }}">
             <button type="submit" class="btn btn-secondary rounded-0 rounded-start">{{ 'Prepare Custom Elimination'|trans }}</button>
@@ -20,15 +19,15 @@
     </div>
 </div>
 <p class="mb-3">{{ 'Number of dropouts:'|trans }}  {{ quiz.dropouts }} </p>
-<table class="table table-hover mb-3">
+<table class="table table-hover table-result mb-3">
     <thead>
     <tr>
         <th scope="col">{{ 'Candidate'|trans }}</th>
-        <th style="width: 15%" scope="col">{{ 'Correct Answers'|trans }}</th>
+        <th class="col-result-sm" scope="col">{{ 'Correct Answers'|trans }}</th>
-        <th style="width: 20%" scope="col">{{ 'Corrections'|trans }}</th>
+        <th class="col-result-md" scope="col">{{ 'Corrections'|trans }}</th>
-        <th style="width: 20%" scope="col">{{ 'Penalty'|trans }}</th>
+        <th class="col-result-md" scope="col">{{ 'Penalty'|trans }}</th>
-        <th style="width: 10%" scope="col">{{ 'Score'|trans }}</th>
+        <th class="col-result-xs" scope="col">{{ 'Score'|trans }}</th>
-        <th style="width: 20%" scope="col">{{ 'Time'|trans }}</th>
+        <th class="col-result-md" scope="col">{{ 'Time'|trans }}</th>
     </tr>
     </thead>
     <tbody>
@@ -40,15 +39,11 @@
                 <form method="post"
                       action="{{ path('tvdt_backoffice_modify_correction', {quiz: quiz.id, candidate: candidate.id}) }}">
                     <input type="hidden" name="_token" value="{{ csrf_token('candidate_correction') }}">
-                    <div class="row">
+                    <div class="d-flex gap-1">
-                        <div class="col-8">
+                        <input class="form-control form-control-sm" type="number"
-                            <input class="form-control form-control-sm" type="number"
+                               value="{{ candidate.corrections }}" step="0.5"
-                                   value="{{ candidate.corrections }}" step="0.5"
+                               name="corrections">
-                                   name="corrections">
+                        <button class="btn btn-sm btn-primary" type="submit">{{ 'Save'|trans }}</button>
-                        </div>
-                        <div class="col-2">
-                            <button class="btn btn-sm btn-primary" type="submit">{{ 'Save'|trans }}</button>
-                        </div>
                     </div>
                 </form>
             </td>
@@ -56,15 +51,11 @@
                 <form method="post"
                       action="{{ path('tvdt_backoffice_modify_penalty', {quiz: quiz.id, candidate: candidate.id}) }}">
                     <input type="hidden" name="_token" value="{{ csrf_token('candidate_penalty') }}">
-                    <div class="row">
+                    <div class="d-flex gap-1">
-                        <div class="col-8">
+                        <input class="form-control form-control-sm" type="number"
-                            <input class="form-control form-control-sm" type="number"
+                               value="{{ candidate.penaltySeconds }}" step="1"
-                                   value="{{ candidate.penaltySeconds }}" step="1"
+                               name="penalty">
-                                   name="penalty">
+                        <button class="btn btn-sm btn-primary" type="submit">{{ 'Save'|trans }}</button>
-                        </div>
-                        <div class="col-2">
-                            <button class="btn btn-sm btn-primary" type="submit">{{ 'Save'|trans }}</button>
-                        </div>
                     </div>
                 </form>
             </td>

templates/backoffice/season.html.twig

@@ -14,7 +14,7 @@
     <h2 class="mb-3">{{ 'Season'|trans }}: {{ season.name }}</h2>
     <div class="row">
         <div class="col-md-6 col-12">
-            <div class="d-flex flex-row align-items-center mb-3">
+            <div class="d-flex align-items-center mb-3">
                 <h4 class="mb-0 pe-2">{{ 'Quizzes'|trans }}</h4>
                 <a class="link"
                    href="{{ path('tvdt_backoffice_quiz_add', {seasonCode: season.seasonCode}) }}">{{ 'Add'|trans }}</a>
@@ -29,7 +29,7 @@
             </div>
         </div>
         <div class="col-md-3 col-12">
-            <div class="d-flex flex-row align-items-center mb-3">
+            <div class="d-flex align-items-center mb-3">
                 <h4 class="mb-0 pe-2">{{ 'Candidates'|trans }}</h4>
                 <a class="link"
                    href="{{ path('tvdt_backoffice_add_candidates', {seasonCode: season.seasonCode}) }}">{{ 'Add Candidate'|trans }}
@@ -40,11 +40,10 @@
                     <li>{{ candidate.name }}</li>{% endfor %}
             </ul>
 
-            <div class="d-flex flex-row align-items-center mb-3">
+            <div class="d-flex align-items-center mb-3">
                 <h4 class="mb-0 pe-2">{{ 'Settings'|trans }}</h4>
             </div>
             {{ form(form) }}
         </div>
-        <div class="col-12 col-md-3"></div>
     </div>
 {% endblock body %}

tests/Service/QuizSpreadsheetServiceTest.php

@@ -0,0 +1,182 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tvdt\Tests\Service;
+
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\TestCase;
+use Symfony\Component\HttpFoundation\File\File;
+use Tvdt\Entity\Answer;
+use Tvdt\Entity\Question;
+use Tvdt\Entity\Quiz;
+use Tvdt\Exception\SpreadsheetDataException;
+use Tvdt\Service\QuizSpreadsheetService;
+
+use function Safe\file_put_contents;
+use function Safe\ob_get_clean;
+use function Safe\ob_start;
+use function Safe\unlink;
+
+#[CoversClass(QuizSpreadsheetService::class)]
+final class QuizSpreadsheetServiceTest extends TestCase
+{
+    private QuizSpreadsheetService $subject;
+
+    /** @var list<string> */
+    private array $tempFiles = [];
+
+    protected function setUp(): void
+    {
+        $this->subject = new QuizSpreadsheetService();
+    }
+
+    protected function tearDown(): void
+    {
+        foreach ($this->tempFiles as $path) {
+            if (file_exists($path)) {
+                unlink($path);
+            }
+        }
+    }
+
+    public function testGenerateTemplateProducesValidXlsx(): void
+    {
+        $path = $this->captureXlsx($this->subject->generateTemplate());
+
+        $this->assertSame(
+            'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
+            new File($path)->getMimeType(),
+        );
+    }
+
+    public function testGenerateTemplateWithoutExampleHasNoQuestions(): void
+    {
+        $path = $this->captureXlsx($this->subject->generateTemplate(fillExample: false));
+
+        $quiz = new Quiz();
+        $this->subject->xlsxToQuiz($quiz, new File($path));
+
+        $this->assertCount(0, $quiz->questions);
+    }
+
+    public function testGenerateTemplateExampleCanBeReimported(): void
+    {
+        $path = $this->captureXlsx($this->subject->generateTemplate(fillExample: true));
+
+        $quiz = new Quiz();
+        $this->subject->xlsxToQuiz($quiz, new File($path));
+
+        $this->assertCount(1, $quiz->questions);
+
+        /** @var Question $question */
+        $question = $quiz->questions->first();
+        $this->assertSame('Is de mol een man of een vrouw?', $question->question);
+        $this->assertCount(2, $question->answers);
+    }
+
+    public function testQuizToXlsxEmptyQuizImportsWithNoQuestions(): void
+    {
+        $path = $this->captureXlsx($this->subject->quizToXlsx(new Quiz()));
+
+        $imported = new Quiz();
+        $this->subject->xlsxToQuiz($imported, new File($path));
+
+        $this->assertCount(0, $imported->questions);
+    }
+
+    public function testQuizToXlsxRoundTrip(): void
+    {
+        $original = $this->makeQuiz();
+        $path = $this->captureXlsx($this->subject->quizToXlsx($original));
+
+        $imported = new Quiz();
+        $this->subject->xlsxToQuiz($imported, new File($path));
+
+        $this->assertCount(2, $imported->questions);
+
+        /** @var Question $first */
+        $first = $imported->questions->first();
+        $this->assertSame('Who is de Mol?', $first->question);
+        $this->assertCount(2, $first->answers);
+
+        $answers = $first->answers->toArray();
+        $this->assertSame('Alice', $answers[0]->text);
+        $this->assertFalse($answers[0]->isRightAnswer);
+        $this->assertSame('Bob', $answers[1]->text);
+        $this->assertTrue($answers[1]->isRightAnswer);
+
+        /** @var Question $second */
+        $second = $imported->questions->last();
+        $this->assertSame('What did de Mol sabotage?', $second->question);
+        $this->assertCount(3, $second->answers);
+    }
+
+    public function testXlsxToQuizThrowsOnInvalidMimeType(): void
+    {
+        $path = $this->createTempPath('.txt');
+        file_put_contents($path, 'not a spreadsheet');
+
+        $this->expectException(\InvalidArgumentException::class);
+        $this->subject->xlsxToQuiz(new Quiz(), new File($path));
+    }
+
+    public function testXlsxToQuizThrowsOnQuestionWithNoAnswers(): void
+    {
+        $quiz = new Quiz();
+        $question = new Question();
+        $question->question = 'Unanswered question';
+        $question->ordering = 1;
+        $quiz->addQuestion($question);
+
+        $path = $this->captureXlsx($this->subject->quizToXlsx($quiz));
+
+        try {
+            $this->subject->xlsxToQuiz(new Quiz(), new File($path));
+            $this->fail('Expected SpreadsheetDataException to be thrown');
+        } catch (SpreadsheetDataException $e) {
+            $this->assertNotEmpty($e->errors);
+        }
+    }
+
+    private function makeQuiz(): Quiz
+    {
+        $quiz = new Quiz();
+
+        $q1 = new Question();
+        $q1->question = 'Who is de Mol?';
+        $q1->ordering = 1;
+        $q1->addAnswer(new Answer('Alice', isRightAnswer: false));
+        $q1->addAnswer(new Answer('Bob', isRightAnswer: true));
+
+        $q2 = new Question();
+        $q2->question = 'What did de Mol sabotage?';
+        $q2->ordering = 2;
+        $q2->addAnswer(new Answer('The boat', isRightAnswer: true));
+        $q2->addAnswer(new Answer('The car', isRightAnswer: false));
+        $q2->addAnswer(new Answer('Nothing', isRightAnswer: false));
+
+        $quiz->addQuestion($q1);
+        $quiz->addQuestion($q2);
+
+        return $quiz;
+    }
+
+    private function captureXlsx(\Closure $closure): string
+    {
+        $path = $this->createTempPath('.xlsx');
+        ob_start();
+        $closure();
+        file_put_contents($path, ob_get_clean());
+
+        return $path;
+    }
+
+    private function createTempPath(string $suffix): string
+    {
+        $path = sys_get_temp_dir().\DIRECTORY_SEPARATOR.uniqid('tvdt_test_', more_entropy: true).$suffix;
+        $this->tempFiles[] = $path;
+
+        return $path;
+    }
+}