search

openpkw / openpkw-weryfikator

Backend dla aplikacji weryfikującej wyniki wyborów
2 stars 8 forks source link

Ow u 16 #21

Closed pogos closed 8 years ago

pogos commented 8 years ago

Zmiany:

Odpalanie testów e2e:

mvn verify -DskipIntegrationTests=false -Dopenpkw-env=test
SebastianCelejewski commented 8 years ago

Proponuję następujące drobne modyfikacje a propos parametryzacji testów:

  1. Profile uruchamiać parametrem o nazwie sugerującej co profil zrobi, np. -Dintegration_tests=test
  2. W pom.xml zamienić na lub
  3. Czy można umieścić profile albo w nadrzędnym pom.xml? Bo tam też są profile.
SebastianCelejewski commented 8 years ago

Drobnostki:

  1. Nieużywane importy (np. w org.openpkw.weryfikator.rest.usersAndSessions.SessionsCreationTests)
  2. Nieużywane metody (np. org.openpkw.weryfikator.rest.JAXRSTestBase.getMessageBody())
  3. MessageHelper mógłby tworzyć od razu ResponseDTO (obecnie MessageHelper tworzy Map, który jest brany przez konstruktor ResponseDTO).
  4. serialVersionUID dla klas serializowalnych?