warszawajug/spring-labs — pull request #1 by gwalen (2014-06-24 19:40:00+00:00)
Title: Fixes tests and eclipse

Pull Request Info :

1.lab09 :
#1) W savings.repository.impl.RepositoryConfiguration brakowało jednej linii
49 : entityManagerFactoryBean.setPackagesToScan("savings.model");
przez co podczas testów leciał :
>java.lang.IllegalStateException: Failed to load ApplicationContext	(..)
>Caused by: java.lang.IllegalStateException: No persistence units parsed from {classpath*:METAINF/persistence.xml}

z treści zadania nie wynikało że ta poprawka jest jego częścią.

#2) Usunięcie @Ignore z testu PaybackBookKeeperModuleTest (projekt start oraz solution) oraz poprawki w PaybackBookKeeperImpl żeby przechodził bez błędów.
Co do testu PaybackBookKeeperTransactionTest @Ignore zostawiłem, nie wiem dlaczego bean (mock) definiowany w teście nie podmienia oryginalnego.
Podczas startu mówi :
>[2014-06-22 13:26:38.689] boot - ???? INFO [main] --- ConfigurationClassBeanDefinitionReader: Skipping bean definition for [BeanMethod:name=paybackRepository,declaringClass=savings.service.PaybackBookKeeperTransactionTest$Config]: a definition for bean 'paybackRepository' already exists. This top-level bean definition is considered as an override.

w efekcie test nie przechodzi bo mockito krzyczy, że bean to nie mock
Ten błąd wygląda podobnie ale w 4.0.1 podobno był poprawiony :
https://jira.spring.io/browse/SPR-11229

2.lab10 :
#1) Zmiany jak wyżej dla #2) w lab09

3.lab11 :
#1) ./mvnw clean install wykładał się ma testach PaybackBookKeeperModuleTest.groovy oraz PaybackControllerTest.groovy. Dodałem @Ignore. Komentarz co się dzieje umieściłem w obu plikach.
W skrócie oba rzucają :
>java.lang.RuntimeException: DB error!

Jakby zachowanie nagrane w teście
PaybackBookKeeperTransactionTest -> shouldRegisterPaybackInTransaction było cały czas aktywne (PaybackBookKeeperModuleTest, PaybackControllerTest muszą być odpalone w kolejności po PaybackBookKeeperTransactionTest żeby tak się stało, jak odpalane są oddzielnie to problemu nie ma)
W temat się nie wgłębiałem poprawiłem tylko żeby się budowało.

4.Dodałem w Readme.md informacje o tym jaki plugin zainstalować i co dalej zrobić żeby wszytko hulało pod Eclipsem.