porównanie dwóch list w teście

Question

Cześć.

Zabrałem się za naukę testów jednostkowych i napotkałem pewien problem. Otóż mam liste, gdzie powrzucałem obiekty:

ofertaPracyList.add(new OfertaPracy("Warszawa", 3000, Set.of("Java", "SQL"), 2));

utworzyłem też metodę, która tworzy listę ze wszystkimi nazwami miast jakie wystepują w liście oferty pracy (przyjmuje liste jako argument):

public Set<String> wszystkieRozneMiasta(List<OfertaPracy> ofertyPracy) {
        
        Set<String> rozneMiasta = new HashSet<>();
        for (OfertaPracy ofertaPracy : ofertyPracy) {
            rozneMiasta.add(ofertaPracy.getMiasto());
        }
        return rozneMiasta;

i chcę to teraz przetestować:

@org.junit.Test
public void wszystkieRozneMiastaJestPrawdziwa2() {
    //given
    Set<String> rozneMiastaOczekiwanyWynik = Set.of("Warszawa", "Wrocław", "Kraków", "Białystok", "Gdańsk", "Lublin");

    //then
    Assert.assertEquals(rozneMiastaOczekiwanyWynik, ofertaPracySerwis.wszystkieRozneMiasta(ofertaPracyList));
}

Test przechodzi prawidłowo, jednak wystarczy, że któreś miasto napiszę z małej litery i test jest failed, a warszawa i Warszawa to "to samo" w sumie :) . Więc mam pomysł aby wstawić toLowerCase(). tylko nie mam pojęcia gdzie to w tym teście lub tej metodzie wstawić. Czy może powinienem to w ogóle jakoś inaczej rozwiązać?

Answer 1

Cześć, głównym celem testów jest sprawdzenie czy Twoja funkcjonalność działa zgodnie z założeniami. Twoja metoda działa w ten sposób, że nie ignoruje wielkości znaków co właśnie weryfikuje Twój test jednostkowy. Jeśli chcesz, żeby przechodziła Ci "warszawa" to Twoja testowana metoda 'wszystkieRozneMiasta' powinna ignorować wielkość liter poprzez np. wywołanie .equalsIgnoreCase() bądź też ustawienie pierwszego znaku na wielką literę a resztę na małe.

Nie możesz wymagać, aby testowana metoda nagle przepuszczała przypadki, które nie zostały zaimplementowane. Wynikiem działania Twoje metody dla danych wejściowych "Warszawa", "warszawa" będzie "Warszawa", "warszawa". Musisz "nauczyć" swoją metodę co ma dokładnie dopuszczać do zbioru. "warszawa".equals("Warszawa") da zawsze false.

Answer 2

Jeśli zadaniem metody wszystkieRozneMiasta jest niemodyfikowanie wyniku to test jest ok. Możesz to tylko jakoś bardziej zaznaczyć, że te dane który wyszły pochodzą z listy wejściowej, jednym ze sposobów może być przygotowanie danych bliżej wywołania testowanej metody:

// given
List<OfertyPracy> offers = createWithCities("Warszawa", ...);
Set<String> expected = Set.of("Warszawa", ...);

// when then
Assert.assertEquals(expected, ofertaPracySerwis.wszystkieRozneMiasta(offers));