JUnit testy jendostkowe

Question

Witam, piszę test metody obliczającej sumę pierwiastków x1 i x2 lecz nie wiem z czym porównać otrzymany wynik. Czy w klasie testującej mam na nowo liczyć deltę, x1 i x2, potem porównać to co zostało zwrócone z tym co test obliczył ? Pozdrawiam :)

Metoda obliczająca x1 i x2, oraz zwracająca sumę tych pierwiastków.

public class FunkcjaKwadratowa {
   public double oblicz(double a, double b, double c){
        double delta;
        double x1 = 0, x2 = 0;
        double suma = 0;
       
        if(a != 0){
           delta = Math.pow(b, 2) - 4*a*c;
             if(delta == 0){
               x1 = (-b - Math.sqrt(delta)) / 2*a;
               suma = x1;
            }else if(delta > 0){
               x1 = (-b - Math.sqrt(delta)) / 2*a;
               x2 = (-b + Math.sqrt(delta)) / 2*a;
               suma = x1 + x2;
            }
        }
        return suma;
    } 
}

Metoda testująca.

 @Test
    public void testOblicz() {
        System.out.println("oblicz");
        double a = 1.0;
        double b = 4.0;
        double c = 3.0;
        FunkcjaKwadratowa instance = new FunkcjaKwadratowa();
        double wynik = instance.oblicz(a, b, c);
        assertEquals("Coś nie tak. ",wynik, TUTAJ);
    }

 

Answer 1

Jeżeli robisz assertEquals to porównujesz coś co wiesz jakie ma być czymś co chcesz przetestować.

Oblicz sobie ile na kartce wyjdzie suma pierwiastków.

http://junit.org/apidocs/org/junit/Assert.html#assertEquals(java.lang.String, double, double, double)

np. assertEquals( "wiadomosc", 3, ( 1+ 2) , 0);

Comments

Ale jak zmienię dane wejściowe to muszę liczyć sumę na nowo, nie da się automatycznie zmieniać sumy pierwiastków w teście ?

---

to sobie zrób assert na kilka wartości? 

• - assert dla delty większej od 0 
• - assert dla delty równej 0 
• - assett dla delty < 0
• - assert dla a = 0 

Ale jak zmienię dane wejściowe to muszę liczyć sumę na nowo, nie da się automatycznie zmieniać sumy pierwiastków w teście ?

hmm a co właśnie testujesz, jak nie funkcje która sumuje Ci pierwiastki?? 

Najpierw zbuduj sobie test, później podaj oczekiwane sumy dla danych które znasz , buduj funkcje.

Podejście TDD:

---

Mogłem gdzieś pominąć jeden test, bo masz 3 warunki więc będzie na oko 6 testów :d aby mieć code coverage na 100%

---

Czemu ukrywasz pytanie? 

Łap kilka ciekawych screenów:

 

Łap kod do junita https://gist.github.com/efiku/75495266a370f156418f tylko sobie pozmieniaj i rób refactor ;)

Pozdrawiam.

Answer 2

Mogę to tak zrobić? Chodzi mi głównie o optymalność,żeby nie pisać czegoś co można dokonać inaczej - prościej.

Metoda obliczeniowa.

public double oblicz(double a, double b, double c){
        double delta;
        double x1 = 0, x2 = 0;
        double suma = 0;
       
        if(a != 0){
           delta = Math.pow(b, 2) - 4*a*c;
             if(delta == 0){
               x1 = (-b - Math.sqrt(delta)) / 2*a;
               suma = x1;
            }else if(delta > 0){
               x2 = (-b + Math.sqrt(delta)) / 2*a;
               suma = x1 + x2;
            }
        }
        return suma;
    } 

Metoda sprawdzająca.

 ​
public void testOblicz() {
        System.out.println("oblicz");
        double a = 0.0;
        double b = 1.0;
        double c = 3.0;

        double delta = Math.pow(b, 2) - 4 * a * c;
        double x1 = 0, x2 = 0;

        if (delta >= 0) {
            x1 = (-b - Math.sqrt(delta)) / 2 * a;
            if (delta > 0) {
                x1 = (-b - Math.sqrt(delta)) / 2 * a;
                x2 = (-b + Math.sqrt(delta)) / 2 * a;
            }
        }

        FunkcjaKwadratowa instance = new FunkcjaKwadratowa();
        double wynik = instance.oblicz(a, b, c);
        assertEquals("wiadomosc", wynik, x1 + x2, 0);
    }

Comments

Szkoda, że nie zrozumiałeś mojej wypowiedzi. Cóż, nie jestem Ci w stanie pomóc, może znajdzie się tu ktoś inny. Zasze możesz sobie zrobić metodę prywatną która przygotowuje Ci jakieś dane, dzięki którym nie duplikujesz kodu. Ja bym zerknął na jakiś data provider. Pierwszy wynik z google: https://github.com/TNG/junit-dataprovider/wiki

---

Nie kumasz o co chodzi w testowaniu. Duplikujesz kod zamiast sprawdzać jego poprawność.

Przykładowo dla dodawania dwóch liczb powinno być:

a = 1, b = 2;
assertEquals(4, a, b);

(To był pseudokod)

Chodzi o to, że podajesz z palca oczekiwany wynik tutaj dla zmiennych.

Zobacz, co masz u siebie - duplikacja kodu, łamanie SRP.

Test nie sprawdza działania, tylko wynik jakiejś funkcjonalności.