longueur titre : tests opérationnels

Reviewed-on: #3

src/main/java/com/mycompany/bibliotheque/Contrôle/LivreValide.java

@@ -4,8 +4,6 @@
  */
 package com.mycompany.bibliotheque.Contrôle;
 
-import com.mycompany.bibliotheque.Métier.Livre;
-
 /**
  * Classe de contrôle de la classe Livre
  * @author dthev
@@ -14,35 +12,13 @@ public class LivreValide {
     // 1. ISBN : exactement 13 chiffres
     public static boolean isValidIsbn(String isbn) {
         // TODO Emile: implémenter la validation
-        boolean valide = true;
+        return false;
-        if(isbn.length() == 13){
-            int i = 0;
-            while (i < 13 && valide) {
-                if(!Character.isDigit(isbn.charAt(i))){
-                    valide = false;
-                }else{
-                    i++;
-                } 
-            }
-        }else{
-            valide = false;
-        }
-        return valide;
     }
 
     // 2. Titre : pas de balises HTML/JS (<script>, <img>, etc.)
     public static boolean isValidTitre(String Titre) {
-        // TODO Salomon: implémenter la validation
+        // TODO: implémenter la validation
-        boolean test = true;
+        return false;
-        int i = 0;
-            while (i < Titre.length() && test) {
-                if(Titre.charAt(i) == '<' || Titre.charAt(i) == '>'){
-                    test = false;
-                }else{
-                    i++;
-                } 
-            }
-        return test;
     }
 
     // 3. Auteur : non vide et pas de chiffres ou caractères spéciaux
@@ -53,31 +29,17 @@ public class LivreValide {
 
     // 4. Titre : longueur maximale 200 caractères 
     public static boolean isLongueurTitreValid(String titre) {
-        // TODO: implémenter la validation
+        // TODO Morgann: implémenter la validation
-        return false;
+        boolean valide = false;
+        if(titre.length()<=200 && titre != "" && titre != null){
+            valide = true;
+        }
+        return valide;
     }    
     
     // 5. Titre : non null et au moins 2 caractères 
     public static boolean isContenuTitreValide(String titre) {
-        // TODO: implémenter la validation
+        // TODO Steve: implémenter la validation
         return false;
 }
-    
-    public static boolean isDisponible (Livre leLivre){
-        boolean valide = true;
-        if(leLivre.isEmprunte()){
-            valide = false;
-        }
-        return valide;
-    };
-    
-    
-    
-    
-    
-    
-    
-    
-    
-    
 }

src/main/java/com/mycompany/bibliotheque/GestionBibliotheque.java

@@ -30,7 +30,7 @@ public class GestionBibliotheque {
         System.out.println("2. Ajouter un livre");
         System.out.println("3. Afficher les livres");
         System.out.println("4. Quitter");
-        System.out.println("Votre choix : ");
+        System.out.print("Merci de faire votre choix : ");
         choix = sc.nextInt();
         switch (choix) {
             case 1 :

src/test/java/com/mycompany/bibliotheque/LivreTest.java

@@ -16,7 +16,7 @@ public void testCreationLivre() {
 
     assertNotNull(b);
     assertEquals("1984", b.getTitre());
-    assertEquals("George Orwel", b.getAuteur()); // corrigé
+    assertEquals("George Orwell", b.getAuteur()); // corrigé
     assertEquals("1234567890123", b.getIsbn());
     assertFalse(b.isEmprunte()); // si la méthode existe
 }

src/test/java/com/mycompany/bibliotheque/LivreValideTest.java

@@ -5,7 +5,6 @@
 package com.mycompany.bibliotheque;
 
 import com.mycompany.bibliotheque.Contrôle.LivreValide;
-import com.mycompany.bibliotheque.Métier.Livre;
 import org.junit.Before;
 import org.junit.Test;
 import static org.junit.Assert.*;
@@ -25,93 +24,37 @@ public class LivreValideTest {
 
     /**
      * Test of isValidIsbn method, of class LivreValide.
+     * @author Emile
      */
     @Test
     public void testIsValidIsbn() {
-        System.out.println("|isValidIsbn |");
+        System.out.println("isValidIsbn");
-        
+        String isbn = "";
-        //Test 1 : un isbn avec moins de 13 carractères
+        boolean expResult = false;
-        System.out.println("Test1 : un isbn avec moins de 13 carractères");
+        boolean result = LivreValide.isValidIsbn(isbn);
-        String isbn1 = "1234568912";
+        assertEquals(expResult, result);
-        boolean expResult1 = false;
+        // TODO review the generated test code and remove the default call to fail.
-        boolean result1 = LivreValide.isValidIsbn(isbn1);
+        fail("The test case is a prototype.");
-        assertEquals("Le nombre de carractère est censé est trop petit",expResult1, result1);
-        System.out.println("Le test1 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
-        //Test2 : lettre dans l'isbn
-        System.out.println("Test2 : lettre dans l'isbn");
-        String isbn2 = "123456789ABC2";
-        boolean expResult2 = false;
-        boolean result2 = LivreValide.isValidIsbn(isbn2);
-        assertEquals("Les alphas ne sont pas autorisés", expResult2, result2);
-        System.out.println("Le test2 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
-        //Test3 : mauvais nombre de carratère 15
-        System.out.println("Test3: isbn comportant plus de 13 chiffres");
-        String isbn3 = "1234568912123568";
-        boolean expResult3 = false;
-        boolean result3 = LivreValide.isValidIsbn(isbn3);
-        assertEquals("Le nombre de carractère est censé est trop grand",expResult3, result3);
-        System.out.println("Le test3 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
-        //Test4 : Un isbn valide
-        System.out.println("Test4: Un isbn valide");
-        String isbn4 = "1234568912126";
-        boolean expResult4 = true;
-        boolean result4 = LivreValide.isValidIsbn(isbn4);
-        assertEquals("L'isbn est censé être valide",expResult4, result4);
-        System.out.println("Le test4 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
     }
 
     /**
      * Test of isValidTitre method, of class LivreValide.
+     * @author Salomé/Emile
      */
     @Test
     public void testIsValidTitre() {
         System.out.println("isValidTitre");
-        
+        String Titre = "";
-        System.out.println("Test1 : un titre comportant un '<'");
+        boolean expResult = false;
-        String Titre1 = "La chat<perché";
+        boolean result = LivreValide.isValidTitre(Titre);
-        boolean expResult1 = false;
+        assertEquals(expResult, result);
-        boolean result1 = LivreValide.isValidTitre(Titre1);
+        // TODO review the generated test code and remove the default call to fail.
-        assertEquals("Le chevron < n'est pas censé etre accepté",expResult1, result1);
+        fail("The test case is a prototype.");
-        System.out.println("Le test1 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
-        System.out.println("Test2 : un titre comportant un '>'");
-        String Titre2 = "La chat >perché";
-        boolean expResult2 = false;
-        boolean result2 = LivreValide.isValidTitre(Titre2);
-        assertEquals("Le chevron > n'est pas censé etre accepté",expResult2, result2);
-        System.out.println("Le test2 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
-        
-        System.out.println("Test3 : un titre valide");
-        String Titre3 = "La chat perché";
-        boolean expResult3 = true;
-        boolean result3 = LivreValide.isValidTitre(Titre3);
-        assertEquals("Ce titre est cencé être valide",expResult3, result3);
-        System.out.println("Le test3 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
-        
     }
 
     /**
      * Test of isValidAuteur method, of class LivreValide.
+     * @author Medhi/Steve
      */
     @Test
     public void testIsValidAuteur() {
@@ -126,37 +69,50 @@ public class LivreValideTest {
 
     /**
      * Test of isLongueurTitreValid method, of class LivreValide.
+     * @author Morgann
      */
     @Test
     public void testIsLongueurTitreValid() {
         System.out.println("isLongueurTitreValid");
+        
+        //test 1 : aucune valeur saisie
+        System.out.println("Test avec aucune valeur saisie");
+        String titre1 = "";
+        boolean expResult1 = false;
+        boolean result1 = LivreValide.isLongueurTitreValid(titre1);
+        assertEquals(expResult1, result1);
+        System.out.println("Titre invalide. Aucune valeur n'a été saisie\n");
+        
+        //test 2 : titre inférieur à 200
+        System.out.println("Test avec aucune valeur saisie");
+        String titre2 = "Hadal Blacksite bestiary";
+        boolean expResult2 = true;
+        boolean result2 = LivreValide.isLongueurTitreValid(titre2);
+        assertEquals(expResult2, result2);
+        System.out.println("Titre valide. Le titre a moins de 200 caractères\n");
+        
+        System.out.println("Test avec un titre avec plus de 200 caractères");
+        String titre3 = "YOU !! You could've had everything you ever wanted... Everything I ever wanted... And you still went out of your way to take everything I had left in the process. You entitled brat. You expect me to sit idly by and keep smiling, As if nothing ever happened? Oh, I'm smiling alright... GRINNING ear to ear. Don't even start with that 'following orders' schlock. You knew what you were doing all too well. Sure took your sweet time. Enjoyed every last second of it? Good. *Chuckles* EXCELLENT, even! I'll merely return the favor. And you bet, I'll be enjoying, every last moment, of THIS!!! THE BEST PART!? I get to do this, over, and over, again. You'll come back, I'll know, and I'll be waiting... You have no one to blame but yourself. You're in a hell of your own making... And you're NEVER GETTING OUT!!! *grrgh* WHAT!!! WHAT IS IT THIS TIME!?!?"; 
+        boolean expResult3 = false;
+        boolean result3 = LivreValide.isLongueurTitreValid(titre3);
+        assertEquals(expResult3, result3);
+        System.out.println("Titre invalide. Titre trop long\n");
+        
+    }
+    
+    /**
+     * Test of isTitreValid method, of class LivreValide.
+     * @author Steve
+     */
+    @Test
+    public void testIsContenuTitreValide() {
+        System.out.println("isTitreValid");
         String titre = "";
         boolean expResult = false;
-        boolean result = LivreValide.isLongueurTitreValid(titre);
+        boolean result = LivreValide.isContenuTitreValide(titre);
         assertEquals(expResult, result);
         // TODO review the generated test code and remove the default call to fail.
         fail("The test case is a prototype.");
     }
     
-    @Test
-    public void testIsDisponible(){
-        System.out.println("isDisponible");
-        Livre unLivre = new Livre("Le comte de MonteCristo","Alexandre Dumas", "1234567891245", false);
-        
-        System.out.println("Test1 : un livre disponible");
-        boolean result1 = LivreValide.isDisponible(unLivre);
-        assertTrue("Le chevron < n'est pas censé etre accepté", result1);
-        System.out.println("Le test1 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-        
-        unLivre.setEmprunte(true);
-        System.out.println("Test2 : un livre indiisponible");
-        boolean result2 = LivreValide.isDisponible(unLivre);
-        assertFalse("Le chevron < n'est pas censé etre accepté", result2);
-        System.out.println("Le test2 est validé");
-        System.out.println("--------------------------------------------------");
-        System.out.println(" ");
-    }
-    
 }