test de la classe commande cas 4

nbproject/project.properties

@@ -1,99 +1,98 @@
-annotation.processing.enabled=true
-annotation.processing.enabled.in.editor=false
-annotation.processing.processor.options=
-annotation.processing.processors.list=
-annotation.processing.run.all.processors=true
-annotation.processing.source.output=${build.generated.sources.dir}/ap-source-output
-build.classes.dir=${build.dir}/classes
-build.classes.excludes=**/*.java,**/*.form
-# This directory is removed when the project is cleaned:
-build.dir=build
-build.generated.dir=${build.dir}/generated
-build.generated.sources.dir=${build.dir}/generated-sources
-# Only compile against the classpath explicitly listed here:
-build.sysclasspath=ignore
-build.test.classes.dir=${build.dir}/test/classes
-build.test.results.dir=${build.dir}/test/results
-# Uncomment to specify the preferred debugger connection transport:
-#debug.transport=dt_socket
-debug.classpath=\
-    ${run.classpath}
-debug.modulepath=\
-    ${run.modulepath}
-debug.test.classpath=\
-    ${run.test.classpath}
-debug.test.modulepath=\
-    ${run.test.modulepath}
-# Files in build.classes.dir which should be excluded from distribution jar
-dist.archive.excludes=
-# This directory is removed when the project is cleaned:
-dist.dir=dist
-dist.jar=${dist.dir}/MusicAndCo.jar
-dist.javadoc.dir=${dist.dir}/javadoc
-dist.jlink.dir=${dist.dir}/jlink
-dist.jlink.output=${dist.jlink.dir}/MusicAndCo
-excludes=
-includes=**
-jar.compress=false
-javac.classpath=
-# Space-separated list of extra javac options
-javac.compilerargs=\ --enable-preview --enable-preview
-javac.deprecation=false
-javac.external.vm=true
-javac.modulepath=
-javac.processormodulepath=
-javac.processorpath=\
-    ${javac.classpath}
-javac.source=21
-javac.target=21
-javac.test.classpath=\
-    ${javac.classpath}:\
-    ${build.classes.dir}:\
-    ${libs.testng.classpath}:\
-    ${libs.junit_5.classpath}:\
-    ${libs.junit_4.classpath}:\
-    ${libs.hamcrest.classpath}
-javac.test.modulepath=\
-    ${javac.modulepath}
-javac.test.processorpath=\
-    ${javac.test.classpath}
-javadoc.additionalparam=
-javadoc.author=false
-javadoc.encoding=${source.encoding}
-javadoc.html5=false
-javadoc.noindex=false
-javadoc.nonavbar=false
-javadoc.notree=false
-javadoc.private=false
-javadoc.splitindex=true
-javadoc.use=true
-javadoc.version=false
-javadoc.windowtitle=
-# The jlink additional root modules to resolve
-jlink.additionalmodules=
-# The jlink additional command line parameters
-jlink.additionalparam=
-jlink.launcher=true
-jlink.launcher.name=MusicAndCo
-main.class=Test
-manifest.file=manifest.mf
-meta.inf.dir=${src.dir}/META-INF
-mkdist.disabled=false
-platform.active=default_platform
-run.classpath=\
-    ${javac.classpath}:\
-    ${build.classes.dir}
-# Space-separated list of JVM arguments used when running the project.
-# You may also define separate properties like run-sys-prop.name=value instead of -Dname=value.
-# To set system properties for unit tests define test-sys-prop.name=value:
-run.jvmargs=\ --enable-preview
-run.modulepath=\
-    ${javac.modulepath}
-run.test.classpath=\
-    ${javac.test.classpath}:\
-    ${build.test.classes.dir}
-run.test.modulepath=\
-    ${javac.test.modulepath}
-source.encoding=UTF-8
-src.dir=src
-test.src.dir=test
+annotation.processing.enabled=true
+annotation.processing.enabled.in.editor=false
+annotation.processing.processor.options=
+annotation.processing.processors.list=
+annotation.processing.run.all.processors=true
+annotation.processing.source.output=${build.generated.sources.dir}/ap-source-output
+build.classes.dir=${build.dir}/classes
+build.classes.excludes=**/*.java,**/*.form
+# This directory is removed when the project is cleaned:
+build.dir=build
+build.generated.dir=${build.dir}/generated
+build.generated.sources.dir=${build.dir}/generated-sources
+# Only compile against the classpath explicitly listed here:
+build.sysclasspath=ignore
+build.test.classes.dir=${build.dir}/test/classes
+build.test.results.dir=${build.dir}/test/results
+# Uncomment to specify the preferred debugger connection transport:
+#debug.transport=dt_socket
+debug.classpath=\
+    ${run.classpath}
+debug.modulepath=\
+    ${run.modulepath}
+debug.test.classpath=\
+    ${run.test.classpath}
+debug.test.modulepath=\
+    ${run.test.modulepath}
+# Files in build.classes.dir which should be excluded from distribution jar
+dist.archive.excludes=
+# This directory is removed when the project is cleaned:
+dist.dir=dist
+dist.jar=${dist.dir}/MusicAndCo.jar
+dist.javadoc.dir=${dist.dir}/javadoc
+dist.jlink.dir=${dist.dir}/jlink
+dist.jlink.output=${dist.jlink.dir}/MusicAndCo
+excludes=
+includes=**
+jar.compress=false
+javac.classpath=
+# Space-separated list of extra javac options
+javac.compilerargs=\ --enable-preview --enable-preview
+javac.deprecation=false
+javac.external.vm=true
+javac.modulepath=
+javac.processormodulepath=
+javac.processorpath=\
+    ${javac.classpath}
+javac.source=21
+javac.target=21
+javac.test.classpath=\
+    ${javac.classpath}:\
+    ${build.classes.dir}:\
+    ${libs.junit_5.classpath}:\
+    ${libs.junit_4.classpath}:\
+    ${libs.hamcrest.classpath}
+javac.test.modulepath=\
+    ${javac.modulepath}
+javac.test.processorpath=\
+    ${javac.test.classpath}
+javadoc.additionalparam=
+javadoc.author=false
+javadoc.encoding=${source.encoding}
+javadoc.html5=false
+javadoc.noindex=false
+javadoc.nonavbar=false
+javadoc.notree=false
+javadoc.private=false
+javadoc.splitindex=true
+javadoc.use=true
+javadoc.version=false
+javadoc.windowtitle=
+# The jlink additional root modules to resolve
+jlink.additionalmodules=
+# The jlink additional command line parameters
+jlink.additionalparam=
+jlink.launcher=true
+jlink.launcher.name=MusicAndCo
+main.class=Test
+manifest.file=manifest.mf
+meta.inf.dir=${src.dir}/META-INF
+mkdist.disabled=false
+platform.active=default_platform
+run.classpath=\
+    ${javac.classpath}:\
+    ${build.classes.dir}
+# Space-separated list of JVM arguments used when running the project.
+# You may also define separate properties like run-sys-prop.name=value instead of -Dname=value.
+# To set system properties for unit tests define test-sys-prop.name=value:
+run.jvmargs=\ --enable-preview
+run.modulepath=\
+    ${javac.modulepath}
+run.test.classpath=\
+    ${javac.test.classpath}:\
+    ${build.test.classes.dir}
+run.test.modulepath=\
+    ${javac.test.modulepath}
+source.encoding=UTF-8
+src.dir=src
+test.src.dir=test

src/Test.java

@@ -19,10 +19,16 @@ public class Test {
      */
     public static void main(String[] args) {
         Entreprise lEntreprise=new Entreprise("MusicAndCo");
+        Instrument instr1=new Instrument(1,"Piano",10, 8300f);
+        Instrument instr2=new Instrument(2,"Violon",9, 105f);
+        Instrument instr3=new Instrument(3,"Guitare",8, 575f);
         Commande laCommande = new Commande(1,1,"12/09/2024");
         lEntreprise.ajouterCommande(laCommande);
         
-        //ajouter 3 piano droit et 2 violon alto dans la commande
+        //Test Cas0-Ajouter à la commande une ligne de 2 instr1
+        System.out.println("Résultat de l'ajout de 2 instr1 à la commande :" + laCommande.ajouter(instr1, 2) );
+        System.out.println("Nouveau stock de instr1 : "+instr1.getQteStock());
+        System.out.println("Quantité de la ligne de commande de instr1 :"+laCommande.getLesLignes().get(instr1));
     }
     
 }

src/métiers/Commande.java

@@ -1,123 +1,145 @@
-/*
- * Click nbfs://nbhost/SystemFileSystem/Templates/Licenses/license-default.txt to change this license
- * Click nbfs://nbhost/SystemFileSystem/Templates/Classes/Class.java to edit this template
- */
-
-package métiers;
-
-import java.util.HashMap;
-import java.util.Map;
-import java.util.Objects;
-
-/**
- *
- * @author ilona
- */
-public class Commande {
-    
-private int noCom, noVendeur;
-private String dateCom;
-
-
-private final HashMap<Instrument, Integer> lesLignes = new HashMap<>(); 
-
-
-    public Commande(int noCom, int noVendeur, String dateCom) {
-        this.noCom = noCom;
-        this.noVendeur = noVendeur;
-        this.dateCom = dateCom;
-    }
-
-
-    /**
-    * Ajoute une quantité donnée d'un instrument à la commande
-    * @param unInstrument instrument à ajouter
-    * @param qte quantité à ajouter 
-    * @return true si l'ajout est bon
-    */
-    public boolean ajouter(Instrument unInstrument, int qte){
-        //il faut gérer l'ajout d'un instrument existant dans la commande, ce sera une mise à jour de la ligne
-        //il faut empêcher l'ajout d'une ligne avec une quantité à 0
-        //bien s'assurer que le stock est suffisant
-        boolean ajoutOK;
-        int qteDisponible = unInstrument.getQteStock();
-        if (qteDisponible<qte){
-            ajoutOK = false;
-        }else {
-            ajoutOK = true;
-            lesLignes.put(unInstrument,qte);
-            unInstrument.setQteStock(qteDisponible-qte);
-        }
-        return ajoutOK;
-    }
-    /**
-    * Supprimer un instrument de la commande
-    * @param unInstrument instrument à supprimer
-    * @return true si la suppression est effectuée
-    */
-    public boolean supprimer(Instrument unInstrument){
-        //lesLignes.remove(unInstrument);
-        boolean suppOK;
-        if (lesLignes.containsKey(unInstrument) == true){
-            suppOK = true;
-            lesLignes.remove(unInstrument);
-        } else {
-            suppOK = false;
-        }
-        return suppOK;
-    }
-   
-/////////////////////////////////////////////////////////////////////////////////
-    
-    public int getNoCom() {
-        return noCom;
-    }
-
-    public int getNoVendeur() {
-        return noVendeur;
-    }
-
-    public String getDateCom() {
-        return dateCom;
-    }
-
-    public HashMap<Instrument, Integer> getLesLignes() {
-        return lesLignes;
-    }
-    
-
-    public void setNoCom(int noCom) {
-        this.noCom = noCom;
-    }
-
-    public void setNoVendeur(int noVendeur) {
-        this.noVendeur = noVendeur;
-    }
-
-    public void setDateCom(String dateCom) {
-        this.dateCom = dateCom;
-    }
-
-    @Override
-    public boolean equals(Object obj) {
-        if (this == obj) {
-            return true;
-        }
-        if (obj == null) {
-            return false;
-        }
-        if (getClass() != obj.getClass()) {
-            return false;
-        }
-        final Commande other = (Commande) obj;
-        if (this.noCom != other.noCom) {
-            return false;
-        }
-        if (this.noVendeur != other.noVendeur) {
-            return false;
-        }
-        return Objects.equals(this.dateCom, other.dateCom);
-    }
-    
-    
-}
+/*
+ * Click nbfs://nbhost/SystemFileSystem/Templates/Licenses/license-default.txt to change this license
+ * Click nbfs://nbhost/SystemFileSystem/Templates/Classes/Class.java to edit this template
+ */
+
+package métiers;
+
+import java.util.HashMap;
+import java.util.Map;
+import java.util.Objects;
+
+/**
+ *
+ * @author ilona
+ */
+public class Commande {
+    
+private int noCom, noVendeur;
+private String dateCom;
+
+
+private final HashMap<Instrument, Integer> lesLignes = new HashMap<>(); 
+
+
+    public Commande(int noCom, int noVendeur, String dateCom) {
+        this.noCom = noCom;
+        this.noVendeur = noVendeur;
+        this.dateCom = dateCom;
+    }
+
+
+    /**
+    * Ajoute une quantité donnée d'un instrument à la commande
+    * @param unInstrument instrument à ajouter
+    * @param qte quantité à ajouter 
+    * @return true si l'ajout est bon
+    */
+    public boolean ajouter(Instrument unInstrument, int qte){
+       
+        //il faut empêcher l'ajout d'une ligne avec une quantité à 0
+        //bien s'assurer que le stock est suffisant
+         boolean ajoutOK=false;
+         //ajout d'un instrument existant dans la commande, ce sera une mise à jour de la ligne
+        if (lesLignes.containsKey(unInstrument))//l'instrument existe
+        {
+            //récupération de la quantité en stock
+            int oldQte=unInstrument.getQteStock();
+            //réaffectation dans le stock de la quantité commandée
+            int qteStock=oldQte+ this.getLesLignes().get(unInstrument);
+            unInstrument.setQteStock(qteStock);
+            //suppression ligne commande
+            lesLignes.remove(unInstrument);
+            //nouvel ajout
+            this.ajouter(unInstrument,qte);
+        
+        }
+        else //nouvel instrument
+        {
+        //on s'assure qu'il y a assez de stock pour la commande
+       
+        int qteDisponible = unInstrument.getQteStock();
+        if (qteDisponible<qte){
+            ajoutOK = false;
+        }else {
+            ajoutOK = true;
+            lesLignes.put(unInstrument,qte);
+            //màj qté stock pour déduire la qté commandée
+            unInstrument.setQteStock(qteDisponible-qte);
+        }
+        }
+        
+        
+        return ajoutOK;
+    }
+    /**
+    * Supprimer un instrument de la commande
+    * @param unInstrument instrument à supprimer
+    * @return true si la suppression est effectuée
+    */
+    public boolean supprimer(Instrument unInstrument){
+        //lesLignes.remove(unInstrument);
+        boolean suppOK;
+        if (lesLignes.containsKey(unInstrument) == true){
+            suppOK = true;
+            lesLignes.remove(unInstrument);
+        } else {
+            suppOK = false;
+        }
+        return suppOK;
+    }
+   
+/////////////////////////////////////////////////////////////////////////////////
+    
+    public int getNoCom() {
+        return noCom;
+    }
+
+    public int getNoVendeur() {
+        return noVendeur;
+    }
+
+    public String getDateCom() {
+        return dateCom;
+    }
+
+    public HashMap<Instrument, Integer> getLesLignes() {
+        return lesLignes;
+    }
+    
+
+    public void setNoCom(int noCom) {
+        this.noCom = noCom;
+    }
+
+    public void setNoVendeur(int noVendeur) {
+        this.noVendeur = noVendeur;
+    }
+
+    public void setDateCom(String dateCom) {
+        this.dateCom = dateCom;
+    }
+
+    @Override
+    public boolean equals(Object obj) {
+        if (this == obj) {
+            return true;
+        }
+        if (obj == null) {
+            return false;
+        }
+        if (getClass() != obj.getClass()) {
+            return false;
+        }
+        final Commande other = (Commande) obj;
+        if (this.noCom != other.noCom) {
+            return false;
+        }
+        if (this.noVendeur != other.noVendeur) {
+            return false;
+        }
+        return Objects.equals(this.dateCom, other.dateCom);
+    }
+    
+    
+}

src/métiers/Entreprise.java

@@ -5,8 +5,6 @@
 package métiers;
 
 import java.util.ArrayList;
-import métiers.Commande;
-import métiers.Instrument;
 
 /**
  *
@@ -15,17 +13,10 @@ import métiers.Instrument;
 public class Entreprise {
     private String raisonSociale;
     private ArrayList<Commande> lesCommandes = new ArrayList<>();
+    private ArrayList<Instrument> lesInstruments = new ArrayList<>();
 
     public Entreprise(String raisonSociale) {
         this.raisonSociale = raisonSociale;
-        
-        ArrayList<Instrument> lesInstruments = new ArrayList<>();
-        Instrument instr1=new Instrument(23,"Piano droit",3, 8300f);
-         Instrument instr2=new Instrument(54,"Violon Alto",5, 105f);
-         Instrument instr3=new Instrument(67,"Guitare Classique",8, 575f);
-         lesInstruments.add(instr3);
-         lesInstruments.add(instr2);
-         lesInstruments.add(instr1);
     }
 
     public String getRaisonSociale() {
@@ -47,9 +38,6 @@ public class Entreprise {
     }
     
     public void supprimerCommande(Commande uneCommande){
-        //parcours pour supprimer les instruments de la commande
-        
-        //suppression de la commande
         lesCommandes.remove(uneCommande);
     }
     

test/métiers/CommandeTest.java

@@ -29,8 +29,10 @@ public class CommandeTest {
     public void testAjouter() {
          Instrument instr1=new Instrument(1,"Piano",10, 8300f);
           Instrument instr2=new Instrument(2,"Violon",9, 105f);
+          Instrument instr3=new Instrument(3,"piano",8, 105f);
           Commande laCommande = new Commande(1,1,"12/09/2024");
         System.out.println("ajouter");
+       //cas 0
        boolean ajoutRes= laCommande.ajouter(instr1, 2);
        boolean ajoutAttendu = true;
         System.out.println("cas0 : cas normal, ajout une ligne de commande");
@@ -42,45 +44,69 @@ public class CommandeTest {
        int qteAttendu=2;
        assertEquals("Test0 quantite",qteRes, qteAttendu);
        
+        //cas 1
        System.out.println("Cas1 : ajout autre ligne de commande");
        laCommande.ajouter(instr2, 6);
        int tailleR=laCommande.getLesLignes().size();
        int tailleAttendue=2;
        assertEquals("Test1 taille",tailleAttendue,tailleR);
        
+       //Cas2
        System.out.println("Cas2 : màj quantité commandée d'une ligne existante");
        laCommande.ajouter(instr2, 5);
        stockRes=instr2.getQteStock();
        stockAttendu=4;
        assertEquals("test2 stock",stockAttendu,stockRes );
        
-       /*qteRes=laCommande.getLesLignes().get(instr2);
+       qteRes=laCommande.getLesLignes().get(instr2);
        qteAttendu=5;
-       assertEquals("Test2 quantite", qteAttendu,qteRes);*/
+       assertEquals("Test2 quantite", qteAttendu,qteRes);
        
         /*INES*/
         System.out.println("Cas3 : ajout nouvelle ligne avec quantité > stock");
+        
+        boolean test= laCommande.ajouter(instr3, 12);
+        stockRes=instr3.getQteStock();
+        stockAttendu=8;
+        boolean testResultExpected=false;
+
+        assertEquals("test3 stock",stockAttendu,stockRes );
+        assertEquals("La méthode ajouter retourne false",testResultExpected,test );
+        
+         System.out.println("Nombre de ligne dans le dictionnaire: "+laCommande.getLesLignes().size());
+        System.out.println("Résultat de la méthode ajouter: " + test);
+
         /*MAISSANE*/
         System.out.println("Cas4 : ajout nouvelle ligne avec quantité 0");
         /*MORGANN*/ 
         System.out.println("Cas5 : mise à 0 quantité commandée d'une ligne existante");
-        /*INES*/
+        /*ILONA*/
         System.out.println("Cas6 : màj d'une ligne existante avec qté>stock");
+        
     }
 
-    /**
-     * Test of supprimer method, of class Commande.
-     
+
     @Test
     public void testSupprimer() {
-        System.out.println("supprimer");
-        Instrument unInstrument = null;
-        Commande instance = null;
-        boolean expResult = false;
-        boolean result = instance.supprimer(unInstrument);
-        assertEquals("test supprimer",expResult, result);
-        // TODO review the generated test code and remove the default call to fail.
-        fail("The test case is a prototype.");
-    }*/
+    //instanciation
+        
+    Instrument instr1=new Instrument(1,"Piano",10, 8300f);
+    Instrument instr2=new Instrument(2,"Violon",9, 105f);
+    Commande laCommande = new Commande(1,1,"12/09/2024");
+    laCommande.ajouter(instr1, 2);
+    boolean expectedResult=false;
+    boolean expectedResult2=true;
+    
+    // test
+    
+    boolean test= laCommande.supprimer(instr2);
+    boolean test2= laCommande.supprimer(instr1);
+    
+    assertEquals("suppression impossible",expectedResult,test);
+    assertEquals("suppression possible ",test2,expectedResult2 );
+     
+    System.out.println("Nombre de ligne dans le dictionnaire: "+laCommande.getLesLignes().size());
+     
+    }
 
 }

test/métiers/MétiersSuite.xml

@@ -1,27 +0,0 @@
-<?xml version='1.0' encoding='UTF-8' ?>
-<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd" >
-<suite name="MusicAndCo">
-
-    <!--
-    see examples at http://testng.org/doc/documentation-main.html#testng-xml
-
-    <suite-files>
-        <suite-file path="./junit-suite.xml" />
-    </suite-files>
-
-    <test name="TimeOut">
-        <classes>
-            <class name="test.timeout.TimeOutTest" />
-            <class name="test.timeout.TimeOutFromXmlTest"/>
-            <class name="test.timeout.TimeOutThreadLocalSampleTest"/>
-        </classes>
-    </test>
-    -->
-
-    <test name="métiers suite">
-        <packages>
-            <package name="métiers"/>
-        </packages>
-    </test>
-
-</suite>