Methoden JUnit-Test einer void-Methode

[shiroX]

Hallo, ich brauche nochmal eure Hilfe,
in der Uni programmieren wir momentan als Klausurersatz ein 4-Gewinnt-Spiel auf der Konsole, das Interface und die Rückgabewerte der Methoden sind vorgegeben.

Nun sollen wir wenn möglich für alle Methoden des Interfaces einen jUnit-Test schreiben, was bei den meisten auch kein Problem ist.
Zu meiner Frage: Meine Methode um einen Spielzug auszuführen hat void als Rückgabewert, kann ich diese Methode trotzdem irgendwie testen? Ich hatte überlegt vielleicht das veränderte Array des Spiels irgendwie zu testen mit AssertArrayEquals, aber das testet ja nur 2 Arrays auf Gleichheit.

Hier meine Methode für einen Spielzug:

/**
     * Mit dieser Methode führt der Spieler einen Zug aus
     * @param column Gewählte Spalte zum "Einwerfen" des Spielsteins
     */

    public void makeMove(int column, int player) {

        int row = 5;

        for (; row >= 0 && gameBoard[column][row] != 0; row--) {
        }

        if (isMoveValid(column) != -1) {

            gameBoard[column][row] = player;
            history.add(column);

            if (player == 1) {
                player = 2;
            } else {
                player = 1;
            }
        }
    }

Also meine Überlegung wäre jetzt mit ein paar Aufrufen von makeMove für mein Board ein paar Züge zu machen und dann irgendwie das entstehende Array als erwartetes Ergebnis zurückzugeben, habe es aber wie gesagt noch nicht hinbekommen.
Vielleicht habt ihr ja da eine Idee

 

[Tobse]

Dein Ansatz geht in die Richtige Richtung: Du erstellst ein Spielfeld (ich vermute mal, dass das die Variable gameBoard ist?), rufst die Methode auf und prüfst dann, ob das Spielfeld den richtigen Zustand hat und ob danach der Richtige Spieler dran ist.
Du erstellst am besten auch einen Testfall für jeden Spezialfall, den dein Code behandeln muss (bswp. Spalte voll).

P.S.: Best-Practice: Schreibe in den Doc-Tag genau rein, was die Methode tun soll:
* Den Stein platzieren
* Den Zug zur Spielhistorie hinzufügen
* Den nächsten Spieler bestimmen und festlegen

Im Unit-Test prüfst du dann jede dieser Aufgaben. Fachsprache: Den Contract ausdefinieren und im Unit-Test immer auf die kompletten Contract prüfen. Siehe Contracts

 

[shiroX]

Hallo,

ja habe es jetzt mal so gemacht und es scheint auch alles zu stimmen, aber irgendwie sagt er mir beim Testen folgendes:
java.lang.AssertionError: array lengths differed, expected.length=6 actual.length=7

Hier mein Code:

public void testMakeMove(){
       
        int[][] gameBoard = new int[7][6];
       
        Board play = new Playboard(gameBoard);
       
        play.makeMove(2, 1);
       
        int[][] expectedArray = {{0, 0, 0, 0, 0, 0, 0},
                                 {0, 0, 0, 0, 0, 0, 0},
                                 {0, 0, 0, 0, 0, 0, 0},
                                 {0, 0, 0, 0, 0, 0, 0},
                                 {0, 0, 0, 0, 0, 0, 0},
                                 {0, 0, 1, 0, 0, 0, 0}};
       
        assertArrayEquals(expectedArray, gameBoard);
        }

Die Arrays sind doch von der Größe genau gleich, irgendwie sehe ich den Fehler nicht - 7 Spalten und 6 Reihen.

 

[Tobse]

Nein, das sind sie nicht. Dein gameBoard hat 7 Reihen und 6 Spalten, dein expectedArray ist andersrum: 6 Reihen und 7 Spalten.

Ein paar Ratschläge:

• erstelle das gameBoard im Playboard. Dann passiert auch so ein Fehler wie dieser auf garnicht. Wenn ich 4 Gewinn spiele setze ich mich auch nicht immer erst mit der Spielfeldgröße auseinander
• play ist ein grottenschlechter Variablenname für ein Spielbrett
• expectedArray ist ein grottenschlechter Variablenname für ein Spielbrett

 

[shiroX]

Ja bei den Namen gebe ich dir Recht war jetzt nur schnell zum Testen, wäre vermutlich schlauer geweden [6][7] zu machen, ist mir aber jetzt erst aufgefallen - Musste es halt im Kopf drehen für den Test.

Trotzdem danke werde mich jetzt mal in deine Vorschläge reinlesen!