diff --git a/README.md b/README.md index 71a3e93c3f3b0201dd6f2a9ff817d7101eace2ae..cdc02276aa442eb186f8ca3b2a5d6530422c7de1 100755 --- a/README.md +++ b/README.md @@ -202,6 +202,9 @@ Pour vérifier que votre programme respecte les conventions de codage vous pouve Pour la partie **test** du programme c'est moins réjouissant, en effet Hacker a créé une classe de test de son cru, plutôt que d'utiliser le *framework* `Junit`, cela peut être une bonne idée dans certains cas notamment pour les tests d'*intégration* (quand on regroupe plusieurs classes ensemble), mais les tests de Hacker laissent faussement penser que le programme fonctionne. Pour apercevoir les dysfonctionnements il suffit pour d'exécuter les méthodes `testPlus` et `testMinus` deux fois de suite. +Vérifiez la couverture de code : `mvn jacoco:prepare-agent install jacoco:report` + +Puis visualisez le résultat : `open target/site/jacoco/index.html` ## Débogage diff --git a/src/main/java/fr/nancy/iut/CalcEngineTester.java b/src/main/java/fr/nancy/iut/CalcEngineTester.java deleted file mode 100755 index dc31dd95a93608f8a4607d855560754034274ac0..0000000000000000000000000000000000000000 --- a/src/main/java/fr/nancy/iut/CalcEngineTester.java +++ /dev/null @@ -1,74 +0,0 @@ -package fr.nancy.iut; - -/** - * Test the CalcEngine class. - * - * @author Hacker T. Largebrain - * @version 1.0 - */ -// package calculator; - -public class CalcEngineTester -{ - // The engine to be tested. - private CalcEngine engine; - - /** - * Constructor for objects of class CalcEngineTester - */ - public CalcEngineTester() - { - engine = new CalcEngine(); - } - - /** - * Test everything. - */ - public void testAll() - { - System.out.println("Testing the addition operation."); - System.out.println("The result is: " + testPlus()); - System.out.println("Testing the subtraction operation."); - System.out.println("The result is: " + testMinus()); - System.out.println("All tests passed."); - } - - /** - * Test the plus operation of the engine. - * @return the result of calculating 3+4. - */ - public int testPlus() - { - // Make sure the engine is in a valid starting state. - engine.clear(); - // Simulate the key presses: 3 + 4 = - engine.numberPressed(3); - engine.plus(); - engine.numberPressed(4); - engine.equals(); - // Return the result, which should be 7. - return engine.getDisplayValue(); - } - - /** - * Test the minus operation of the engine. - * @return the result of calculating 9 - 4. - */ - public int testMinus() - { - // Make sure the engine is in a valid starting state. - engine.clear(); - // Simulate the presses: 9 - 4 = - engine.numberPressed(9); - engine.minus(); - engine.numberPressed(4); - engine.equals(); - // Return the result, which should be 5. - return engine.getDisplayValue(); - } - - public static void main(String[] args) { - CalcEngineTester tester = new CalcEngineTester(); - tester.testAll(); - } -} diff --git a/src/main/java/fr/nancy/iut/Calculator.java b/src/main/java/fr/nancy/iut/Calculator.java index 31d3c3f925d5bd17097501e61f462900ddeaa304..4209ddf9339b5b1004462389a17974df493c0d73 100755 --- a/src/main/java/fr/nancy/iut/Calculator.java +++ b/src/main/java/fr/nancy/iut/Calculator.java @@ -37,4 +37,15 @@ public class Calculator public static void main(String[] args) { (new Calculator()).show(); } + + /** + * @return the engine + */ + public CalcEngine getEngine() {return engine;} + + /** + * @return the gui + */ + public UserInterface getGui() {return gui;} + } diff --git a/src/main/java/fr/nancy/iut/UserInterface.java b/src/main/java/fr/nancy/iut/UserInterface.java index 45c3ae32593b651dff61bd992177bd3dd5a6f7f2..453f59ed904a5befcea10b004eed8945741cd25c 100755 --- a/src/main/java/fr/nancy/iut/UserInterface.java +++ b/src/main/java/fr/nancy/iut/UserInterface.java @@ -163,4 +163,26 @@ public class UserInterface showingAuthor = !showingAuthor; } + /** + * Getter visible + */ + public boolean getVisible() + { + return frame.isVisible(); + } + + /** + * getter Display + */ + public JTextField getDisplay() + { + return display; + } + + + public JLabel getStatus() + { + return status; + } + } diff --git a/src/test/java/fr/nancy/iut/CalcEngineTest.java b/src/test/java/fr/nancy/iut/CalcEngineTest.java index 0b38cf4ecdf55e3380366358a687e918ee1644ce..c66b6d4603d0609c4d7a72a7fe73bfa2fc68f3a0 100644 --- a/src/test/java/fr/nancy/iut/CalcEngineTest.java +++ b/src/test/java/fr/nancy/iut/CalcEngineTest.java @@ -16,7 +16,7 @@ public class CalcEngineTest { * @return the number pressed. */ @Test - void testClear() { + public void testClear() { // Je crée une nouvelle calculatrice CalcEngine calculette = new CalcEngine(); // Je lui donne une valeur et je vérifie que la valeur en sortie est bien celle que je lui ai donnée @@ -33,7 +33,7 @@ public class CalcEngineTest { * @return the result of calculating 9-2. */ @Test - void testEquals() { + public void testEquals() { CalcEngine calculette = new CalcEngine(); ////Je fais le teste en faisant la soustraction des 2 chiffres @@ -56,7 +56,7 @@ public class CalcEngineTest { * Test of getAuthor method, of class CalcEngine. */ @Test - void testGetAuthor() { + public void testGetAuthor() { CalcEngine calculette = new CalcEngine(); assertEquals("Hacker T. Largebrain", calculette.getAuthor()); @@ -66,7 +66,7 @@ public class CalcEngineTest { * @return the tow number pressed. */ @Test - void testGetDisplayValue() { + public void testGetDisplayValue() { CalcEngine calculette = new CalcEngine(); //Je fais le teste en vérifiant que ce que je rentre est bien égale à ce que m'affiche la calculatrice @@ -80,7 +80,7 @@ public class CalcEngineTest { * Test of getTitle method, of class CalcEngine. */ @Test - void testGetTitle() { + public void testGetTitle() { // //Je fais le teste pour la valeur voulut String res = "Super Calculator"; CalcEngine calculette = new CalcEngine(); @@ -93,7 +93,7 @@ public class CalcEngineTest { * Test of getVersion method, of class CalcEngine. */ @Test - void testGetVersion() { + public void testGetVersion() { // //Je fais le teste pour la valeur voulut String res = "version 0.2"; CalcEngine calculette = new CalcEngine(); @@ -107,7 +107,7 @@ public class CalcEngineTest { * @return the result of calculating 8-2. */ @Test - void testMinus() { + public void testMinus() { CalcEngine calculette = new CalcEngine(); //Je fais le teste en faisant la soustraction des 2 chiffres calculette.numberPressed(8); @@ -123,7 +123,7 @@ public class CalcEngineTest { * @return the tow number pressed. */ @Test - void testNumberPressed() { + public void testNumberPressed() { CalcEngine calculette = new CalcEngine(); //Je fais le teste en vérifiant que ce que je rentre est bien égale à ce que m'affiche la calculatrice calculette.numberPressed(9); @@ -138,7 +138,7 @@ public class CalcEngineTest { * @return the result of calculating 7+1. */ @Test - void testPlus() { + public void testPlus() { CalcEngine calculette = new CalcEngine(); //Je fais le teste en faisant l'addition des 2 chiffres