[Roballo]

Mireczki mam problem, uczę się dopiero testować springowe controllery i tutaj wyrzuca mi taki błąd:

[{"id":1,"login":"login1","password":"password","email":"email","avatar":null}]

błąd http://wklej.to/xMWah

klasa user: http://wklej.to/fNwJz
klasa userRepository http://wklej.to/viGWs
klasa userController http://wklej.to/a9YIL
klasa testujaca http://wklej.to/JDU6s

Spring boot 1.3.6

Gdzie robię bład?
#spring #java #naukaprogramowania

[L.....l]

@Roballo: Pokaz klase konfiguracyjna. Jak rozumiem aplikacja dziala jak ja normalnie uruchamiasz?

[Roballo]

@Legol: mam application.properties http://wklej.to/EDXl7
normalnie działa po uruchomieniu

[ppawel]

@Roballo: oo, NPE ( ͡° ͜ʖ ͡°)

Coś nie halo w 2 miejscach:
- numeracja linii chyba jest nieaktualna
- upewnij się czy @Autowired to na pewno odpowiednia adnotacja dla pola klasy testowej

[Roballo]

@ppawel: we wszystkich przykładach które oglądałem to tak było i nie wiem gdzie u mnie ten durny błąd jest.

at com.serwis.controller.UserControllerTests.createUser(UserControllerTests.java:56)
dotyczy w userControllerTests http://wklej.to/L7UDR

at com.serwis.controller.UserController.findAll(UserController.java:37)
dotyczy w userController
http://wklej.to/x6O18

[NorthPL93]

Za zagłębiałem się za bardzo w kod, ale to new mi wygląda podejrzanie:

this.mockMvc = standaloneSetup(new UserController()).build();

Jak zrobisz new X() to wtedy Spring chyba nie da rady zrobić Autowire itp.
Zrób sobie zmienną w tej klasie od testów
@Autowired
private UserController userController;
i wtedy przekazuj ją do tego standaloneSetup

[Roballo]

@NorthPL93: to już jakiś trop, wyskoczyło mi teraz

java.lang.AssertionError: No value at JSON path "$.login", exception: Expected to find an object with property ['login'] in path $ but found 'net.minidev.json.JSONArray'. This is not a json object according to the JsonProvider: 'com.jayway.jsonpath.spi.json.JsonSmartJsonProvider'.

dla linii pokazanej w komentarzu
http://wklej.to/qr9eK

już googluje

dałem normalne
jsonPath("$" zamiast $.login

i wypluło

java.lang.AssertionError: JSON path "$"

Expected: is "login1"

but: was <[{"id":1,"login":"login1","password":"password","email":"email","avatar":null}]>

[NorthPL93]

@Roballo: Jestem trochę zielonka w tych sprawach, ale o ile rozumiem tuturial z neta to:
jak Twoje API zwraca ci tablicę to w tym

.andExpect(jsonPath("$.login", is("login1")))

chyba musisz wskazać o który element Ci chodzi
Przykład z netu:

.andExpect(jsonPath("$[0].description", is("Lorem ipsum")))

[Roballo]

@NorthPL93: no dobra rzeczywiście to głupie już było z mojej strony, wczoraj próbowałem w ten sposób, ale że cały czas mi wyskakiwał błąd z NPE to go odrzuciłem. Dzięki Ci bardzo!

Swoją drogą masz może pomysł czemu spring nie łapie nowego obiektu w tym?

Jak zrobisz new X() to wtedy Spring chyba nie da rady zrobić Autowire itp.

[NorthPL93]

@Roballo Bo Spring musi zrobić magię z refleksjami żeby móc powypełniać te pola danymi, jak robisz new X() to po prostu Spring nie wie, że tworzysz obiekt więc nic w nim nie robi.

[Roballo]

@NorthPL93: czyli wszystko jak najbardziej jawnie i łopatologicznie, w sumie ma to sens. Dzięki raz jeszcze męczyłem się z tym od wczoraj i zapewne jeszcze by mi zeszło zanim bym znalazł ten fragment ( ͡° ʖ̯ ͡°)

[afe1]

@Roballo:

@Transactional
public interface UserRepository extends CrudRepository{}

hmm, nie powinienes czasem dodac @Repository do tego?

[Leihto]

@Roballo: Jak piszesz testy, to nie wolno Ci korzystać z adnotacji @Autowired. Testy nie tworzą beanów, a tylko nie można wstrzykiwać. Zamiast tego powinieneś użyć spy z mockito, a przy tym powinieneś mockować konkretne metody (oczywiście publiczne metody, bo tylko takie pozwala mockować mockito)

[Roballo]

@Leihto tutaj chciałem controller testować to jak, co rozumiesz przez spy w mocku?

[Leihto]

@Roballo: Generalnie dobrą praktyką jest aby rozdzielać serwisy od controllerów. Kod powinno się pisać w taki sposób, aby było go jak najłatwiej testować. Jeśli chodzi o używanie spy, to chodzi o to, że w momencie gdy zaciągniesz serwis przez @Autowired w testach, to będziesz operował na danych z bazy użytkowej. Zastosowanie spy (bez adnotacji, w konstruktorze UsersRepository usersRepository = Mockito.spy(UsersRepository.class);) zaciągnie tylko i wyłącznie interface, więc nie będzie działało na żadnych

[Roballo]

@Leihto: aaa tak to wygląda. Wrzucę później cały ten projekt do review to zobaczymy gdzie na grzmociłem, a gdzie nie. Dzięki za wyjaśnienie, jestem w trakcie pisania to sobie wezmę do serca ;)

[Leihto]

@Roballo: jakby co, to pisz na priv ( ͡° ͜ʖ ͡°)