jUnit 5 Test für eine addMethode

[uppss]

Hallo Jungs,

ich muss eine Testmethode schreiben und prüfen, ob nur eindeutige Personen hinzugefügt werden können.
@Test
public void test_addPerson_liefert_true() {
    Person p1 = new Person("name1, 1);
    Assertions.assertTrue(test.addPerson(p1));
}

Nun möchte ich die andere Seite der Medaille testen und zwei identische Personen hinzufügen wie gehe ich damit am besten um?
@Test
public void test_addPerson_liefert_false() {
    Person p1 = new Person("name1, 1);
    Assertions.assertTrue(test.addPerson(p1));
    Assertions.assertFalse(test.addPerson(p1));
}

 

[KonradN]

Vom Prinzip her ist es doch schon das, was du brauchst. Musst Du nur in eine Form bringen, die übersetzbar ist. Und Du musst die Spezifikation genau prüfen - was am Status kannst Du alles prüfen? Ggf. prüft man auch Dinge wie: Ist die Anzahl der Elemente unverändert nach der zu testenden Aktion.

Und was auch auffällt: Du testest in dem Test nicht nur das Hinzufügen einer bereits vorhandenen Person sondern auch das erste Hinzufügen. Das kann - wenn man strenge Regeln hat - auch schon als unerwünscht angesehen werden.

 

[uppss]

Hallo Konrad, könntest du ein paar Beispiele zeigen, vielen Dank dafür

 

[KonradN]

Was für Beispiele brauchst Du hier noch? Du hast ja in Deinem Code die Tests eigentlich schon - wenn man mal davon absieht, dass die Variable test nicht deklariert & initialisiert wurde.

Daher sehe ich gerade nicht, an welchem Punkt es gerade bei Dir "hängt" und was ich Dir daher zeigen könnte.

Evtl. paar Sourcen für Bücher / Dokumentation:

• https://riptutorial.com/ebook/junit ist ein kleines, freies Werk
• JavaCodeGeeks bietet eine kostenlose Registrierung und danach hat man Zugriff auf viele (teilweise ältere) EBooks. Darunter auch welche zu JUnit und Mocking (nur JUnit 4 meine ich). Aber man bekommt da dann auch regelmäßig Werbung und Infos per Email. Die Seite bietet darüber auch noch etwas mehr und die Emails von Zeit zu Zeit sind da ein kleiner Preis denke ich (Ich habe damit keine Probleme).
• JUnit 5 User Guide: https://junit.org/junit5/docs/5.6.2/user-guide/index.pdf

Das wären paar freie Quellen - es gibt aber auch ein paar gute Bücher zu JUnit 5, die interessant sein könnten so man Bücher bevorzugt. Ansonsten findet man sehr viel Einführungen und Erklärungen per Google.

 

[uppss]

Hallo Konrad,
Vielen Dank für hilfsreiche tipps.

Das Problem bei meinem Test besteht darin, dass p1 immer neu hinzugefügt wird und ich nur einmal testen kann. Ich habe noch kein Plan, wie ich die liste aus +1, +2 ect. testen kann. Das die Liste größer geworden ist, bzw. unverändert geblieben ist…

@test
public void test_addPerson_liefert_false() {
Person p1 = new Person("name1, 1);
//test ist zu testende klasse
Assertions.assertTrue(test.addPerson(p1));
Assertions.assertFalse(test.addPerson(p1));
}

 

[KonradN]

Hallo Konrad,
Vielen Dank für hilfsreiche tipps.

Das Problem bei meinem Test besteht darin, dass p1 immer neu hinzugefügt wird und ich nur einmal testen kann. Ich habe noch kein Plan, wie ich die liste aus +1, +2 ect. testen kann. Das die Liste größer geworden ist, bzw. unverändert geblieben ist…

Wie sieht denn der ganze Test aus? Wo ist denn die Variable test Deklariert und Initialisiert worden?

Das sollte alles Bestandteil des Tests selbst sein!

 

[Jw456]

Zeige bitte auch die Klasse in der die Methode addPerson ist. Und wie schon gesagt, wie du zu der Instanz test kommst.

 

[uppss]

import com.java.testpolygon.domain.Person;

public interface PersonService {

    public boolean addPerson(Person argPerson);

}

package com.java.testpolygon.personservice;

import com.java.testpolygon.domain.Person;

import java.util.ArrayList;
import java.util.List;

public class PersonServiceImpl implements PersonService{

    @Override
    public boolean addPerson(Person argPerson) {
        List<Person> list = new ArrayList<>();
        if (list.contains(argPerson)) {
            return list.add(argPerson);
        }
        return false;
    }
}

package com.java.testpolygon.domain;

public class Person {

    private String name;
    private int age;

    public Person(String name, int age) {
        this.name = name;
        this.age = age;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public int getAge() {
        return age;
    }

    public void setAge(int age) {
        this.age = age;
    }

    @Override
    public String toString() {
        return "Person{" +
                "name='" + name + '\'' +
                ", age=" + age +
                '}';
    }

    @Override
    public boolean equals(Object obj) {
        boolean result = false;
        if (obj == null || obj.getClass() != getClass()) {
            return false;
        } else {
            Person person = (Person) obj;
            if (this.name.equals(person.getName())) {
                result = true;
            }
        }
        return result;
    }
}

package com.java.testpolygon.personservice;

import com.java.testpolygon.domain.Person;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.Test;

public class PersonServiceImplTest {

    @Test
    public class testAddPerson() {
        Person person = new Person("a", 1);
        PersonServiceImpl test = new PersonServiceImpl();
        Assertions.assertTrue(test.addPerson(person)); // OK
        Assertions.assertFalse(test.addPerson(person)); // NOK

    }
}

 

[KonradN]

Du hast kein Problem mit dem Unit-Test sondern mit der Implementation. Deine Implementation von PersonServiceImpl ist einfach falsch.

 

[uppss]

Konrad danke für die Antwort. Was würdest du da anders machen?
Die Vorgabe ist, dass Person genau nur einmal da sein darf und die Methode true zurückliefen soll, wenn er zur Collection erfolgreich hinzugefügt wurde

 

[KonradN]

Dann schau Dir Deine Methode addPerson einmal im Detail an. Was passiert denn da? Welche Lebensdauer hat denn die Liste?

 

[uppss]

local, ich habe die Liste rasgenommen und global deklariert:
trotzdem:

public class PersonServiceImpl implements PersonService{

    List<Person> list = new ArrayList<>();
    
    @Override
    public boolean addPerson(Person person) {
        if (list.contains(argPerson)) {
            return list.add(person);
        }
        return false;
    }
}

Expected :false
Actual :true

 

[KonradN]

Erst einmal übersetzt der gezeigte Code so nicht.

Und dann geh einmal die gezeigte Logik durch. Was soll wann passieren?

 

[uppss]

Es soll eine eindeutige Person zu einer Collection hinzugefügt werden und die Methode soll true zurückliefen, falls diese Person hinzugefügt wurde und false, falls die Person schon in der Collection vorhanden war.

 

[KonradN]

Das ist die Anforderung - aber was macht Dein Code? Schau was Dein Code macht.
Füge ansonsten zu Allem ein Kommentar hinzu, das den Code Deiner Meinung nach erläutert.

 

[uppss]

List<Person> list = new ArrayList<>();
public boolean addPerson(Person person) {
        if (list.contains(person)) { // wenn vorhanden, dann false
            return false;
        } else {
            return list.add(argPerson); // sonst einfügen und true zurückliefern
        }
    }

 

[KonradN]

Das sieht jetzt deutlich besser aus. Und die Unit Tests laufen jetzt durch nehme ich an.

 

[uppss]

Ja, vielen Dank. Könntest du mir vlt. noch zeigen, wie man adden auf moderne Art und weite mit Streams realisieren könnte und was man noch testen kann. Vielen Dank

 

[KonradN]

Wenn Du Deine Klasse auch mit Streams nutzen können willst, dann schau Dir einfach einmal an, was da so an Möglichkeiten vorhanden sind. Wenn Du Elemente eines Streams einsammeln willst, dann wird ja collect benutzt. Da kannst Du also schauen, was da so benötigt wird.

Ansonsten musst Du Dir überlegen, was für Funktionalität du haben willst. Und jede Funktionalität, die Du hinzu fügst, musst Du dann auch testen.