How to unit-test-eine Methode, die liest eine angegebene Datei

Ich weiß das ist ein bisschen naiv. Wie unit-Tests, dieses Stück code ohne physische Datei als input.
Ich bin neu mockito und unit-Tests. Also bin ich nicht sicher. Bitte helfen Sie.

public static String fileToString(File file) throws IOException
{
    BufferedReader br = new BufferedReader(new FileReader(file));
    try {
        StringBuilder sb = new StringBuilder();
        String line = br.readLine();

        while (line != null) {
            sb.append(line);
            sb.append("\n");
            line = br.readLine();
        }
        return sb.toString();
    } finally {
        br.close();
    }
}

Look at stackoverflow.com/questions/17681708/...
In den verlinkten Beispielen akzeptierte Antwort, die ich glaube, er ist unter einer physischen Datei, um den code zu testen. Ich möchte, um zu testen, dass ohne eine Datei.

InformationsquelleAutor user2990315 | 2013-12-20

9

Sollte man wohl umgestalten Methode. Als man erkannte, eine Methode, wobei eine Datei als Eingabe ist nicht leicht überprüfbar. Auch, es scheint statisch zu sein, die Hilfe nicht die Testbarkeit. Wenn Sie schreiben Sie Ihre Methode so :
```
public String fileToString(BufferedReader input) throws IOException
```
wird es viel einfacher zu testen. Trennen Sie Ihre business-Logik bilden die technischen Einzelheiten einer Datei zu Lesen. Wie ich es verstehe, Ihre business-Logik-Lesen von Strom und Sicherstellung der Zeilenenden auf unix-Stil.

Wenn Sie das tun, Ihre Methode getestet werden. Sie machen es auch mehr generisch : es können jetzt Lesen aus einer Datei, von einer URL oder von jeder Art von stream. Besseren code, einfacher zu testen ...
- Die Methode wird nur einmal verwendet, das auch innerhalb der Klasse nur. Es macht es ein guter Kandidat für die macht es statisch.
- Dann, das macht es ein guter Kandidat, um es privat und es nicht testen, nicht, um es statisch. Ihre tests sollten Agnostiker der Umsetzung, aber testen Sie das Verhalten des Prüflings (die meisten der Zeit, eine Klasse).
- Ich habe nicht verstanden, dein Vorschlag funktioniert mit bufferedreader kann ich verspotte den bufferedreader Eingabe-Oder ist es wie kann ich Inhalte hinzufügen, um den buffered reader leicht und damit ich es testen kann anstelle einer Datei-Eingang.
- Eine 100% code coverage ist es notwendig zu testen privater Methoden zu.
- Sie können ganz einfach erstellen Sie ein BufferedReader komplett im Speicher mit Hilfe einer StringReader, ohne die Abhängigkeit von einer Datei.
- Eine 100% code coverage ist kein vernünftiges Ziel. Es hilft nicht, die Qualität Ihres Codes. Wenn es einen Grund gibt, zu testen, diese spezifische Methode, es bedeutet wahrscheinlich, dass es in der Klasse brechen würde das Prinzip der Einzigen Verantwortung.
- Ich denke, es muss statisch sein, da bin ich den Aufruf von Konstruktor.
InformationsquelleAutor Guillaume
19

Können Sie eine Datei erstellen, die als Teil der Prüfung, keine Notwendigkeit, verspotten Sie es aus.

JUnit hat eine nette Funktionalität für die Erstellung von Dateien für die Tests verwendet und automatisch die Reinigung mit der TemporaryFolder Regel.
```
public class MyTestClass {

    @Rule
    public TemporaryFolder folder = new TemporaryFolder();

    @Test
    public void myTest() {
        //this folder gets cleaned up automatically by JUnit
        File file = folder.newFile("someTestFile.txt");

        //populate the file
        //run your test
    }
}
```
- Aber Ist das nicht gegen die unit-Tests am besten Praxis, um nicht Lesen physische Datei?
- Nein, würde ich nicht so sagen. Ich glaube nicht, dass es problematisch ist der Einsatz von real-Objekten, vor allem, wenn es macht das testen deutlich einfacher. Für ein Objekt wie eine Datei, die ausgiebig getestet wurde und verwendet in fast jeder Java-app (so ist es wahrscheinlich auch ziemlich gut funktioniert) UND hat alle Arten von OS-spezifischen Sachen, die mit ihm verbunden, nicht Schaden, mit einer realen Datei.
InformationsquelleAutor Jeff Storey
3

Warum willst du simulieren Sie eine Datei? Mocking java.io.File ist eine schlechte Idee, weil es hat viele einheimische Sachen. Ich würde empfehlen, dass Sie sicherstellen, dass ein minimalistisches text-Datei steht im classpath, wenn die unit-tests ausgeführt werden. Sie können konvertieren Sie diese Datei, um text und bestätigen Sie die Ausgabe.
- Danke. Aber Ist das nicht gegen die unit-Tests am besten Praxis, um nicht Lesen physische Datei?
- NÖ. Ich denke nicht so. Der ganze Sinn von unit-Tests ist, um Ihren code schreiben, in "Funktionseinheiten" und dann zu testen, wird jede Einheit mit einer kleinen Probe-Eingang und stellen Sie sicher, dass Sie die richtige Ausgabe für dieses kleine Beispiel für die Eingabe.
InformationsquelleAutor Amit Sharma

könnten Sie die Kombination von ByteArrayInputStream und BufferedReader-Klasse, um Ihre gewünschte Datei in Ihrem code. So würde es keine Notwendigkeit zu schaffen, eine echte Datei auf Ihrem system. Was würde passieren, wenn Sie haben nicht genug Berechtigung --auf der Grundlage von einigen bestimmten Umständen-um eine Datei zu erstellen. Auf den folgenden code, erstellen Sie Ihre eigenen wünschenswert Inhalt der Datei:

public static void main(String a[]){

    String str = "converting to input stream"+
                    "\n and this is second line";
    byte[] content = str.getBytes();
    InputStream is = null;
    BufferedReader bfReader = null;
    try {
        is = new ByteArrayInputStream(content);
        bfReader = new BufferedReader(new InputStreamReader(is));
        String temp = null;
        while((temp = bfReader.readLine()) != null){
            System.out.println(temp);
        }
    } catch (IOException e) {
        e.printStackTrace();
    } finally {
        try{
            if(is != null) is.close();
        } catch (Exception ex){

        }
    }

}

InformationsquelleAutor MHPalang

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.