Feature/role 1 UI

.github/pull_request_template.md

@@ -0,0 +1,25 @@
+## Ce que fait ce MR
+
+<!-- Une phrase : "Ajoute X", "Corrige Y", "Implémente Z" -->
+
+## Couche concernée
+
+- [ ] `model` — entités et POJOs
+- [ ] `hypervisor` — IHypervisor / VBoxOrchestrator
+- [ ] `data` — DatabaseManager / schema SQL
+- [ ] `pdf` — PdfExporter
+- [ ] `controller` — LabController
+- [ ] `ui` — vues JavaFX
+- [ ] Config / build (pom.xml, module-info, CI)
+
+## Checklist avant review
+
+- [ ] `./mvnw test` passe en local (0 failure, 0 error)
+- [ ] Javadoc présent sur toutes les classes et méthodes publiques ajoutées
+- [ ] Pas de `System.out.println` de debug oublié
+- [ ] Pas de TODO non intentionnel
+- [ ] Les imports inutilisés ont été supprimés (IntelliJ : Ctrl+Alt+O)
+
+## Pour le reviewer
+
+<!-- Ce qui mérite une attention particulière, un choix d'implémentation à valider, une question -->

.github/workflows/ci.yml

@@ -0,0 +1,34 @@
+name: CI
+
+on:
+  push:
+    branches: [ "main", "develop", "feature/**", "fix/**" ]
+  pull_request:
+    branches: [ "develop", "main" ]
+
+jobs:
+  build-and-test:
+    name: Build & Test (Java 17)
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Set up Java 17
+        uses: actions/setup-java@v4
+        with:
+          java-version: '17'
+          distribution: 'temurin'
+          cache: maven          # met le dépôt Maven en cache → builds 2x plus rapides
+
+      - name: Build & run tests
+        run: ./mvnw test --no-transfer-progress
+
+      - name: Upload Surefire reports on failure
+        if: failure()           # s'exécute UNIQUEMENT si les tests échouent
+        uses: actions/upload-artifact@v4
+        with:
+          name: surefire-reports
+          path: target/surefire-reports/
+          retention-days: 7

.gitignore

@@ -45,3 +45,4 @@ build/
 
 ### Mac OS ###
 .DS_Store
+target/

CONTRIBUTING.md

@@ -0,0 +1,74 @@
+# Guide de contribution — Cyber Lab Manager
+
+## Branches
+
+| Préfixe | Usage | Exemple |
+|---|---|---|
+| `feature/` | Nouvelle fonctionnalité | `feature/vbox-orchestrator` |
+| `fix/` | Correction de bug | `fix/snapshot-uuid-null` |
+| `chore/` | Config, CI, docs | `chore/github-config` |
+| `test/` | Ajout ou correction de tests seuls | `test/hypervisor-coverage` |
+
+**Règle absolue :** on ne pousse jamais directement sur `develop` ou `main`.  
+Toujours passer par un MR depuis une branche `feature/` ou `fix/`.
+
+## Format des commits
+
+On suit [Conventional Commits](https://www.conventionalcommits.org/fr) :
+
+```
+<type>(<scope optionnel>): <description courte en français>
+```
+
+| Type | Quand l'utiliser |
+|---|---|
+| `feat` | Ajout d'une fonctionnalité |
+| `fix` | Correction d'un bug |
+| `test` | Ajout ou modification de tests |
+| `docs` | Documentation (Javadoc, Markdown) |
+| `refactor` | Refactoring sans changement de comportement |
+| `chore` | Config build, CI, dépendances |
+
+**Exemples :**
+```
+feat(model): add Snapshot uuid and online fields
+fix(hypervisor): handle VBoxManage non-zero exit code
+test(model): add LabTest unmodifiable list assertion
+docs(hypervisor): complete Javadoc on IHypervisor
+chore: add GitHub Actions CI workflow
+```
+
+## Ouvrir un MR
+
+1. Ta branche est à jour avec `develop` (`git rebase develop` ou `git merge develop`)
+2. `./mvnw test` passe en local — **0 failure, 0 error**
+3. Javadoc présent sur toutes les classes et méthodes publiques ajoutées
+4. Ouvre le MR vers `develop` sur GitHub
+5. Remplis le template de MR (pré-rempli automatiquement)
+6. Assigne au moins un reviewer
+
+## Revue de code
+
+- Le reviewer a **48h** pour donner son retour
+- Un commentaire `[bloquant]` doit être résolu avant le merge
+- Un commentaire `[suggestion]` est optionnel
+- Approuver = confirmer que les tests passent ET que le code est lisible
+
+## Standards de code
+
+- **Java 17**, style standard IntelliJ (Ctrl+Alt+L pour formatter)
+- Javadoc obligatoire sur toutes les classes et méthodes `public`
+- Pas de `System.out.println` → utiliser `java.util.logging.Logger`
+- Imports inutilisés supprimés (Ctrl+Alt+O dans IntelliJ)
+- Pas de TODO non documenté dans un MR
+
+## Responsabilités par package
+
+| Package | Responsable |
+|---|---|
+| `tg.cyberlabmanager.model` | Rôle 3 |
+| `tg.cyberlabmanager.hypervisor` | Rôle 3 |
+| `tg.cyberlabmanager.data` | Rôle 2 |
+| `tg.cyberlabmanager.pdf` | Rôle 2 |
+| `tg.cyberlabmanager.ui` | Rôle 1 |
+| `tg.cyberlabmanager.controller` | Rôle 1 |

docs/Markdowns/SETUP.md

@@ -0,0 +1,89 @@
+# SETUP — Environnement de développement
+
+## Prérequis
+
+| Outil | Version minimale | Vérification |
+|---|---|---|
+| JDK | 17 (LTS) | `java -version` |
+| Maven | 3.8+ | `./mvnw -version` |
+| VirtualBox | 7.0+ | Pour les tests d'intégration |
+| Git | 2.x | `git --version` |
+
+> Le projet utilise le Maven Wrapper (`./mvnw`) — pas besoin d'installer Maven globalement.
+
+---
+
+## Configuration IDE — point critique à lire
+
+Le projet compile en **Java 17** même si tu as un JDK plus récent (21, 22…) sur ta machine.
+C'est contrôlé par cette ligne dans `pom.xml` :
+
+```xml
+<maven.compiler.release>17</maven.compiler.release>
+```
+
+Cette propriété fixe simultanément la syntaxe acceptée, le bytecode cible (class file version 61)
+et les APIs accessibles — le compilateur de ton JDK s'adapte automatiquement.
+
+### IntelliJ IDEA
+
+**File → Project Structure → Project** :
+- SDK : ton JDK installé (17, 21, peu importe)
+- Language level : **17**
+
+**File → Settings → Build → Compiler → Java Compiler** :
+- Target bytecode version : **17**
+
+### Pourquoi c'est important
+
+Si quelqu'un compile avec bytecode Java 21 et pousse du code, les autres obtiendront :
+```
+UnsupportedClassVersionError: Unsupported major.minor version 65.0
+```
+La version 65 = Java 21. La version 61 = Java 17 (notre cible).
+Garder tout le monde sur la même cible évite ce type de conflit au merge.
+
+---
+
+## Lancer le projet
+
+```bash
+# Compiler et lancer tous les tests
+./mvnw test
+
+# Lancer l'application JavaFX
+./mvnw javafx:run
+
+# Générer la Javadoc dans target/reports/apidocs/
+./mvnw javadoc:javadoc
+```
+
+> Si tu utilises un JDK non standard, préfixe avec JAVA_HOME :
+> `JAVA_HOME=/usr/lib/jvm/jdk-21.0.11-oracle-x64 ./mvnw test`
+
+---
+
+## Structure des branches
+
+```
+main         ← releases stables uniquement
+develop      ← intégration continue (branche de référence)
+feature/x    ← développement d'une fonctionnalité
+fix/x        ← correction de bug
+chore/x      ← config, CI, documentation
+```
+
+Voir [CONTRIBUTING.md](../CONTRIBUTING.md) pour les conventions complètes.
+
+---
+
+## Responsabilités par package
+
+| Package | Rôle | Contenu |
+|---|---|---|
+| `tg.cyberlabmanager.model` | Rôle 3 | Entités métier (Lab, VM, Snapshot…) |
+| `tg.cyberlabmanager.hypervisor` | Rôle 3 | IHypervisor, VBoxOrchestrator |
+| `tg.cyberlabmanager.data` | Rôle 2 | DatabaseManager, SQLite |
+| `tg.cyberlabmanager.pdf` | Rôle 2 | PdfExporter |
+| `tg.cyberlabmanager.ui` | Rôle 1 | Vues JavaFX |
+| `tg.cyberlabmanager.controller` | Rôle 1 | LabController |

src/main/java/module-info.java

@@ -1,3 +1,6 @@
+/**
+ * Module principal de l'application Cyber Lab Manager.
+ */
 module tg.cyberlabmanager {
 
     // ── JavaFX ──────────────────────────────────────────────────────────
@@ -23,12 +26,12 @@
     opens tg.cyberlabmanager.controller to javafx.fxml;
     opens tg.cyberlabmanager.ui         to javafx.fxml;
 
-    // ── Exports publics ──────────────────────────────────────────────────
+    // ── Exports publics 
     exports tg.cyberlabmanager.app;
     exports tg.cyberlabmanager.controller;
     exports tg.cyberlabmanager.model;
     exports tg.cyberlabmanager.data;
     exports tg.cyberlabmanager.hypervisor;
     exports tg.cyberlabmanager.pdf;
     exports tg.cyberlabmanager.ui;
-}
+}

src/main/java/tg/cyberlabmanager/app/CyberLabApp.java

@@ -4,16 +4,49 @@
 import javafx.fxml.FXMLLoader;
 import javafx.scene.Scene;
 import javafx.stage.Stage;
+import tg.cyberlabmanager.controller.LabController;
 
-import java.io.IOException;
-
+/**
+ * Application JavaFX principale de Cyber Lab Manager.
+ */
 public class CyberLabApp extends Application {
+
+    private LabController controller;
+
+    /**
+     * Cree l'application JavaFX.
+     */
+    public CyberLabApp() {
+    }
+
+    /**
+     * Initialise et affiche la fenetre principale JavaFX.
+     *
+     * @param stage scene principale fournie par JavaFX
+     * @throws IOException si le chargement du fichier FXML echoue
+     */
     @Override
-    public void start(Stage stage) throws IOException {
-        FXMLLoader loader = new FXMLLoader(CyberLabApp.class.getResource("/tg/cyberlabmanager/ui/main-view.fxml"));
-        Scene scene = new Scene(loader.load(), 900, 600);
+    public void start(Stage stage) throws Exception {
+        FXMLLoader loader = new FXMLLoader(
+            CyberLabApp.class.getResource("/tg/cyberlabmanager/ui/main-view.fxml")
+        );
+        Scene scene = new Scene(loader.load(), 1400, 820);
+        controller = loader.getController();
         stage.setTitle("Cyber Lab Manager");
+        stage.setMinWidth(1100);
+        stage.setMinHeight(650);
         stage.setScene(scene);
         stage.show();
     }
+
+    @Override
+    public void stop() {
+        if (controller != null) {
+            controller.shutdown();
+        }
+    }
+
+    public static void main(String[] args) {
+        launch(args);
+    }
 }

src/main/java/tg/cyberlabmanager/app/Launcher.java

@@ -2,7 +2,21 @@
 
 import javafx.application.Application;
 
+/**
+ * Point d'entree deleguant le lancement a l'application JavaFX.
+ */
 public class Launcher {
+    /**
+     * Cree le lanceur de l'application.
+     */
+    public Launcher() {
+    }
+
+    /**
+     * Lance l'application JavaFX.
+     *
+     * @param args arguments de ligne de commande
+     */
     public static void main(String[] args) {
         Application.launch(CyberLabApp.class, args);
     }