[Dominias]

#programowanie #java #junit
Koledzy, uczę się pisać testy jednostkowe ale mam z tym problemy. Po pierwsze nie wiem czy rozumuję w sposób prawidłowy dlatego prosiłbym na zerknięcie na przypadek który chcę przetestować (domyślam się, że dla testowanej metody, która dodaje trening do dnia, będę musiał zrobić trzy przypadki). Druga sprawa nie mam zielonego pojęcia dlaczego w linii 56 (w tym linku) wywala mi NullPointerException. Prosiłbym o rady bo na razie mimo zapoznania się z jednym kursem, testy to dla mnie nadal czarna magia.

[krasnoludkolo]

@Dominias: prawdopodobnie training.getExerciseList(); zwraca nulla. Debbuger w dłoń i jedziesz

[Dominias]

@krasnoludkolo: dokładnie, zamieniłem to na new ArrayList<>(). Tylko teraz wypieprza mi Nulla w zupełnie innym miejscu. Jak widzisz metoda addTraining po LocalDate pobiera Day do którego będę dodawał trening. Niestety teraz w tym miejscu wypieprza mi NullPointera (3 linia w tym paste code co wstawiłem) a chciałbym po prostu, żeby dniem "wejściowym" był getDayWithNulls

[Dominias]

@krasnoludkolo: dobra nie wywowałem dnia w metodzie i NullPointer już nie wyskakuje. Teraz inny problem ehh: java.lang.StackOverflowError, wygooglowałem, że to przez rekurencję, IDE mi podpowiada, że występuje to zjawisko w przypadku getDay i getExpectedDay, jakaś rada jak sobie z tym poradzić?

[losowyNickk]

@Dominias: Wystarczy spojrzeć na metody getDayWithNulls() oraz expectedDayWithAddedTrainingToNulls() i to co zwracaja. Z każdym wywołaniem jednej z metod w return masz getDayWithNulls() i jak wołasz metodę drugi raz to co ona zwróci? No pewnie, że getDayWithNulls() i tak do usranej śmierci.

[Dominias]

@losowyNickk: tak już to zauważyłem i poprawiłem. Teraz wyskakują mi NullPointery z metodami związanymi z DB, chociaż DB zamokowałem ehhh myślałem, że testy jednostkowe będą nieco łatwiejsze ( ͡° ͜ʖ ͡°)

[losowyNickk]

@Dominias: to pewnie problem z implementacją a nie testami :P

[Dominias]

@krasnoludkolo: @losowyNickk: odnośnie testów i sytuacji z jaką próbuję sobie poradzić to jeszcze taka sytuacja: https://paste.ofcode.org/FduL8AnzkUULx8YRfEKCjj i w tym przypadku błąd mam zapewne w when...

Błąd:
You cannot use argument matchers outside of verification or stubbing.
Examples of correct usage

[Endr3w]

@Dominias: when(eq(item).getJSON(anyString())).thenReturn(anyString());
żeby na itemie dało sie wywołać "when" to musi to być mock a u Ciebie z tego co widzę jest to prawdziwy obiekt. W dodatku w thenReturn musisz mu zapodać jakąś wartość, anyString() jest argument matcherem, czyli możesz go użyć w taki sposób:
when(itemMock.getJSON(anyString())).thenReturn("blablabla");

[Dominias]

@Endr3w: item to jest mock właśnie, próbowałem też w thenReturn dać stringa jakiegoś i to samo niestety

[Endr3w]

@Dominias: coś kręcisz. Daj cały kod

[Dominias]

@Endr3w: https://paste.ofcode.org/ksXpu7sGyn5bSB6ZRJnABs

[Darjo]

@Dominias: mocki mockami ale po co wywołujesz getDayWithNulls w 103 linii skoro nie używasz wyniku tego?

[Dominias]

@Darjo: to jest pierdoła bez wpływu na wynik testu i nie tego dotyczy mój obecny problem, dodałem to jak kombinowałem na różne sposoby kiedy wywalało mi NullPointerException i zapomniałem usunąć po prostu

[Endr3w]

@Darjo: to: when(eq(item).getJSON(anyString())).thenReturn(anyString());
powinno wyglądać tak: when(item.getJSON(anyString())).thenReturn(tu wstaw wartość jaką chcesz zwrócić, nie może tu być anyString());

[Darjo]

@Endr3w: a to akurat nie do mnie a do @Dominias:

[Dominias]

@Endr3w: teoretycznie to przechodzi, z tym że teraz wywala mi, że item jest nullem przez co dostaję nullpointerexception. Także teraz nie wiem jak sobie poradzić z testowaniem czegoś co używa obiektu z bazy danych skoro zamokowanie item nie pomogło

[diabel_z_piekla]

@Dominias: Wrzuć jeszcze raz aktualny kod, ale kompletny (cała klasa testowa + klasa serwisu który testujesz).
Pisz w której linijce leci exception i wklej output z konsoli wraz ze stacktrace.

[Dominias]

@diabel_z_piekla: już sobie poradziłem i chyba ogarnąłem w miarę o co chodzi w pisaniu testow jednostkowych