Java Input/Output Tests Junit

[harrytut]

Hallo,

Ich habe folgendes Problem: Ich habe eine Klasse, die die Benutzerinteraktion (Input/Output) handhabt und ich muss Junit4 Tests dafür schreiben.

Die Klasse, die ich testen möchte, sieht in etwa so aus:

package program;
// imports
public class UserInteraction {
    private Program program;

    private UserInteraction(){
        program = new Program();
    }

    // Menu display
    private void start() {
        while (!program.getProgramStarted) {
            String input = readInput();
            switch (input) {
            case "1":
                addUser();
                break;
            case "2":
                deleteUser();
                break;
            case "3":
                // ...
            default:
                System.err.println("Unknown option");
                break;
            }
        }
    }

    private void addUser() {
        // Output
        String username = readInput();
        program.addUser(username);
        System.out.println("User " + username + "added");
    }

    private void deleteUser() {
        // Output
        String username = readInput();
        program.deleteUser(username);
        System.out.println("User " + username + "deleted");
    }

    @SuppressWarnings("resource")
    private String readInput() {
        return (new Scanner(System.in)).nextLine();
    }

    public void main(String[] args) {
        UserInteraction ui = new UserInteraction();
        ui.start();
    }

}

Und die Programmklasse sieht in etwa so aus:

package program;
// imports
public class Program {
    // fields
    private User currentUser;
    private List<User> user = new ArrayList<User>();
    // constructor

    public void addUser(String name) throws Exception {
        if (user.size() >= 6) {
            throw new Exception();
        }
        int newID = user.size() + 1;
        user.add(new User(name, newID));
    }

    public String[] getALlUser() {
        int counter = 0;
        String[] userArray = new String[user.size()];

        for (User s : user) {
            userArray[counter++] = s.toString();
        }
        return userArray;
    }
}

Nun möchte ich zum Beispiel testen, ob die folgende Eingabe korrekt verarbeitet wird:

1
John
=> User John added

Zu diesem Zweck habe ich einen Junit-Test geschrieben, der wie folgt aussieht:

package tests;

import java.io.ByteArrayInputStream;
import java.io.ByteArrayOutputStream;
import java.io.InputStream;
import java.io.PrintStream;

import org.junit.After;
import org.junit.Test;

import program.*;

public class UserInteractionTest {

    private final InputStream originalIn = System.in;
    private final PrintStream originalOut = System.out;
    private Program program;

    @After
    public void restoreStreams() {
        System.setIn(originalIn);
        System.setOut(originalOut);
    }

    @Test
    public void testCreateUser() throws Exception {
        String input = "1\nJohn\n";
        ByteArrayInputStream in = new ByteArrayInputStream(input.getBytes());
        System.setIn(in);
        ByteArrayOutputStream outContent = new ByteArrayOutputStream();
        System.setOut(new PrintStream(outContent)); // Leitet System.out um

        program = new Program();
        try {
            program.main(null);
        } finally {
            System.setIn(originalIn);
        }
        String expectedOutput = "User John added";
        assertTrue("Output not found", outContent.toString().contains(expectedOutput));
    }

}

Die Eingabe ("1\n") wird gemacht, aber danach wird keine Eingabe eingefügt, so dass der Test hängen bleibt.
Weiß jemand, wie man dieses Problem lösen kann? Oder gibt es vielleicht eine bessere Möglichkeit, die Eingabe/Ausgabe meines Java-Programms zu testen/zu überprüfen?

Vielen Dank im Voraus!

Ich habe verschiedene Änderungen am Eingabestrom ausprobiert, z. B. die Verwendung von System.lineSeparator() anstelle von n, aber das Problem bleibt bestehen.

 

[KonradN]

Dein Problem ist, wie Du die Klasse Scanner nutzt. Scanner puffert auch, d.h. wenn Du immer einen neuen Scanner erstellst, dann verlierst Du alle Eingaben, die der Scanner bereits gelesen hast, auch wenn diese noch nicht verarbeitet wurden.

Dein erster Scanner liest also bereits komplett alles ein. Wenn dann der nächste Scanner erstellt wird, dann ist der Stream natürlich bereits leer und es gibt keine Eingabe mehr.

Ansonsten sei mir noch der Hinweis erlaubt, dass es Sinn machen würde, den Code besser zu unterteilen und dann bei den Unit Tests entsprechend wirklich nur die jeweiligen Units zu testen.

 

[harrytut]

Dein Problem ist, wie Du die Klasse Scanner nutzt. Scanner puffert auch, d.h. wenn Du immer einen neuen Scanner erstellst, dann verlierst Du alle Eingaben, die der Scanner bereits gelesen hast, auch wenn diese noch nicht verarbeitet wurden.

Dein erster Scanner liest also bereits komplett alles ein. Wenn dann der nächste Scanner erstellt wird, dann ist der Stream natürlich bereits leer und es gibt keine Eingabe mehr.

Ansonsten sei mir noch der Hinweis erlaubt, dass es Sinn machen würde, den Code besser zu unterteilen und dann bei den Unit Tests entsprechend wirklich nur die jeweiligen Units zu testen.

Vielen Dank für die schnelle Antwort.
Ich habe jetzt ein neues Feld für den Scanner angelegt und im Konstruktor
mit scanner = new Scanner(System.in); initialisiert
die readInput Methode hab ich geändert zu

@SuppressWarnings("resource")
    private String readInput() {
        return scanner.nextLine();
    }

Und ich sehe jetzt, dass die Eingabe auch passt, aber dadurch (denke ich zumindest), dass ich das Menü in einer Schleife aufrufe schlägt der Test mit

java.util.NoSuchElementException: No line found

fehl. Gibt es eine Anweisung, dass das Programm nach dem Test beendet wird?
Zur Unterteilung, leider darf ich keine public Methoden anlegen und dann fällt mir keine Möglichkeit ein, die einzelnen Methoden spezifisch aufzurufen, außer Reflections zu verwenden

 

[KonradN]

Wenn der Code genau so bleiben soll, dann wäre dies das zu erwartende Ergebnis. Sprich: Nach der Ausführung erwartest du genau diese Exception.

Da das Programm, das Du testest, sich beenden muss, wäre ansonsten die Alternative, dass Du noch den Menupunkt zum Beenden aufrufst. Also wenn "q" zum beenden aufzurufen ist, dann kommt halt an die Eingabe ein "q\n" dran.

Generell ist das halt kein Unit Test. Du testest eine einzelnen Methoden und der Code ist an sich auch schlicht nicht wirklich testbar.

Selbst wenn Du z.B. Scanner mockst und dann die Eingaben vorgibst: Du bist in der Schleife und musst da irgendwie raus kommen. Das geht dann nur über eine Exception. Denn wirklich komplett beenden willst Du nicht, denn damit beendest Du ja auch den Test-Runner und so.