Wydaje się, że testy automatyczne zagościły w świecie wytwarzania oprogramowania na dobre. Jednak jeśli są one niedostatecznej jakości, szybko mogą stać się obciążeniem przynoszącym więcej szkody niż pożytku. Podczas prezentacji omówię najczęstsze błędy popełniane w testach jednostkowych i integracyjnych przez średnio zaawansowanych praktyków Test-Driven Development. Pokażę problemy wynikające z nieznajomości lub niewłaściwego wykorzystania narzędzi przede wszystkim z rodziny JUnit. Wytłumaczę również pojęcie betonowania oraz kwestię, jak poprawić swoje testy bazy danych i API REST-owych.

1. Kręta droga do dobrych testów
Mateusz Sulima

2. Dlaczego testujemy?

3. Rola testów
• Pozwalają bezpiecznie wprowadzać zmiany
• Najlepsza dokumentacja
• Upraszczają refaktoring
• Wymuszają refaktoring
• Przyspieszają rozwój oprogramowania

4. Złe zrozumienie roli testów
• Zielony słupek w Sonarze
• Wszyscy tak robią
• Odbicie kodu produkcyjnego

5. Niewłaściwe wykorzystanie
narzędzi

6. Przykład początkowy
@Test
public void testShouldWork() {
assertTrue(validator.isValid(
dateTime("2015-02-19 15:00"),
dateTime("2015-02-20 15:00")));
}

7. Zmiana nazwy
@Test
public void shouldBeValidWhenStartDateIsBeforeEndDate() {
assertTrue(validator.isValid(
dateTime("2015-02-19 15:00"),
dateTime("2015-02-20 15:00")));
}

8. Struktura testu
@Test
public void shouldBeValidWhenStartDateIsBeforeEndDate() {
DateTime startDate = dateTime("2015-02-19 15:00");
DateTime endDate = dateTime("2015-02-20 15:00");
boolean valid = validator.isValid(startDate, endDate);
assertTrue(valid);
}

9. Given / When / Then
@Test
public void shouldBeValidWhenStartDateIsBeforeEndDate() {
// given
DateTime startDate = dateTime("2015-02-19 15:00");
DateTime endDate = dateTime("2015-02-20 15:00");
// when
boolean result = validator.isValid(startDate, endDate);
// then
assertTrue(result);
}
✓

10. Inicjalizacja zmiennych
User user = new User();
user.setId(1);
user.setEnabled(true);
user.setFirstName("Jan");
user.setLastName("Kowalski");
Address address = new Address();
address.setCity("Gliwice");
address.setStreet("Akademicka");
address.setNumber(16);
user.setAddress(address);
// albo
User user = new User(1, true, "Jan", "Kowalski", new
Address("Gliwice" , "Akademicka", 16));

11. Buildery
User user = UserBuilder.anUser()
.withId(1)
.enabled()
.withFirstName("Jan")
.withLastName("Kowalski")
.livesIn(AddressBuilder.anAddress()
.inCity("Gliwice")
.onStreet("Akademicka")
.atNumber(16).build()
).build();
✓

12. Asercje
assertThat(user.getId()).isEqualTo(1);
assertThat(user.isEnabled()).isTrue();
assertThat(user.getFirstName()).isEqualTo("Jan");
assertThat(user.getLastName()).isEqualTo("Kowalski");
assertThat(user.getAddress().getCity()).isEqualTo("Gliwice");
assertThat(user.getAddress().getStreet()).isEqualTo("Akademicka");
assertThat(user.getAddress().getNumber()).isEqualTo(16);

13. Asercje domenowe
assertUser(user)
.hasId(1)
.isEnabled()
.hasFullName("Jan", "Kowalski");
assertAddress(user.getAddress())
.hasCity("Gliwice")
.hasStreet("Akademicka")
.hasNumber(16);
✓

14. Płynne asercje
assertThat("Quality Excites").startsWith("Quality");
assertThat("Quality Excites").containsIgnoringCase("excites");
assertThat(Math.PI).isCloseTo(3.14, within(0.002));
LocalDateTime qeDate =
LocalDateTime.parse("2015-05-30T00:00:00");
assertThat(qeDate).isAfter("1999-12-31T23:59:59");
assertThat(qeDate).matches(date ->
date.getDayOfWeek() == DayOfWeek.SATURDAY);
✓

15. Pętle
@Test
public void shouldBeValidWhenStartDateIsBeforeEndDate() {
// given
LocalDateTime endDate = dateTime("2015-02-20 15:00");
for (int i = 16; i <= 19; i++) {
LocalDateTime startDate = dateTime("2015-02-" + i + " 15:00");
// when
boolean result = validator.isValid(startDate, endDate);
// then
assertTrue(result);
}
}

16. Data-Driven Tests
@Test
@Parameters(method="nullDates")
public void shouldBeValidWhenAnyDateIsNull(
DateTime startDate, DateTime endDate) {
// when
boolean result = validator.isValid(startDate, endDate);
// then
assertTrue(result);
}
private Object nullDates() {
return $($(null, dateTime("2015-02-19 15:00")),
$(dateTime("2015-02-19 15:00"), null),
$(null, null));
}

17. Spock
def "be valid when any of dates is null"() {
expect:
validator.isValid(startDate, endDate)
where:
startDate | endDate
null | null
dateTime("2015-02-20 15:00") | null
null | dateTime("2015-02-20 15:00")
}
✓

18. Mocki
public void shouldFilterOutDeletedItems() {
// given
Item deleted = ItemBuilder.withId(1).deleted().build();
Item active = ItemBuilder.withId(2).build();
when(itemsRepository.getAllItems()).thenReturn(
newArrayList(deleted, active));
// when
List<Item> activeItems =
itemsService.getActiveItems();
// then
assertThat(activeItems).containsExactly(active);
}
✓

19. Nowoczesna skrzynka narzędziowa
• Given/when/then
• Data-driven tests
• Buildery
• Asercje domenowe
• Fluent assertions
• Mocki
• Testowanie wyjątków
• Testowanie asynchroniczności
• Mocki dla REST API

20. Testowanie interakcji,
a nie zachowania

21. Then
…
verify(discountValidator).isValid(discount);
verify(userValidator).isValid(userId, discount);
verify(userRepository).getById(userId);
verify(itemsRepository).findItems(user);
verify(itemsRepository, times(2)).save(any(Item.class));
verify(discountApplier).apply(discount, item1);
verify(discountApplier).apply(discount, item2);
verifyNoMoreInteractions(userRepository);
verifyNoMoreInteractions(discountApplier);

22. Given / When
…
when(discountValidator.isValid(discount))
.thenReturn(true);
when(userValidator.isValid(userId, discount))
.thenReturn(true);
when(userRepository.getById(userId))
.thenReturn(user);
when(itemsRepository.findItems(user))
.thenReturn(Lists.newArrayList(item1, item2));
// then
discountService.applyDiscount(userId, discount);
…

23. Given
Discount discount = null;
int userId = 0;
User user = null;
Item item1 = new Item(user, 0);
Item item2 = mock(Item.class);
…

24. Przewaga testów integracyjnych
• Są krótsze
• Odporne na zmiany w implementacji
• Sprawdzają rzeczywistość
• Pozwalają na eksperymenty

25. Test integracyjny
// given
Discount discount = new Discount(20);
User user = UserBuilder.withId(123).build();
Item item1 = ItemBuilder.withId(1).soldByUser(user).withPrice(100).build();
Item item2 = ItemBuilder.withId(2).soldByUser(user).withPrice(200).build();
userRepository.save(user);
itemsRepository.saveAll(item1, item2);
// when
discountService.applyDiscount(user.getId(), discount);
// then
assertThat(itemsRepository.findItems(user))
.extracting("price").containsExactly(80, 160);

26. Don’t mock what
you don’t own

27. Serializacja
verify(mockWriter, times(1)).writeStartElement("doc");
verify(mockWriter, times(5)).writeStartElement("field");
verify(mockWriter, times(1)).writeAttribute("name", "id");
verify(mockWriter, times(1)).writeAttribute("name", "product_id");
verify(mockWriter, times(1)).writeAttribute("name", "name");
verify(mockWriter, times(5)).writeCharacters(anyString());
verify(mockWriter, times(6)).writeEndElement();
verify(mockWriter, times(1)).flush();

28. Mockowanie bazy danych
// given
ResultSet resultSet = mock(ResultSet.class);
when(session.execute(any(Select.Where.class)))
.thenReturn(resultSet);
Row row = mock(Row.class);
when(resultSet.all()).thenReturn(newArrayList(row));
when(row.getInt(anyString())).thenReturn(0, 123, 0);
// when
Collection<Item> items =
itemsRepository.findItems(UserBuilder.withId(1).build());
// then
assertThat(items).extracting("id").containsExactly(123);

29. Definicja tabeli
CREATE TABLE test.items (
user_id INT,
item_id INT,
price DECIMAL,
PRIMARY KEY(user_id)
)

30. Test integracyjny
// given
session.execute(
"insert into items(user_id, item_id, price) values (1, 123, 0)");
session.execute(
"insert into items(user_id, item_id, price) values (1, 124, 0)");
session.execute(
"insert into items(user_id, item_id, price) values (2, 777, 0)");
// when
Collection<Item> items =
itemsRepository.findItems(UserBuilder.withId(1).build());
// then
assertThat(items).extracting("id").containsExactly(123, 124);
✓

31. Testy REST API
// given
WebTarget webTargetMock = mock(WebTarget.class);
Invocation.Builder builderMock = mock(Invocation.Builder.class);
Invocation invocationMock = mock(Invocation.class);
when(clientMock.target("http://localhost:8080/items/1/price"))
.thenReturn(webTargetMock);
when(webTargetMock.request()).thenReturn(builderMock);
when(builderMock.accept(MediaType.TEXT_PLAIN)).thenReturn(builderMock);
when(builderMock.buildPost(any(Entity.class))).thenReturn(invocationMock);
// when
notifyService.updatePrice(1, 50);
// then
verify(invocationMock).submit();

32. WireMock
// when
notifyService.updatePrice(1, 50);
// then
verify(postRequestedFor(urlEqualTo("/items/1/price"))
.withRequestBody(equalToJson("{"price": 50}"))
.withHeader(HttpHeaders.ACCEPT, equalTo("text/plain")));
✓

33. Podsumowanie
• Czysty kod się liczy
• Im mniej kodu tym lepiej
• No silver bullets

34. Q/A?

35. Thank you!

36. Check us: allegrotech.io
Join us: kariera.allegro.pl
Twitter: allegrotechblog
e-commerce full of technology
github.com/msulima/tests-showcase