Problem beim erstellen eines Tests

[Wellensittich]

Hallo liebes Forum und zwar will ich zu meinem Programm einen Test erstellen, [jUnit] ..weiß aber nicht ganz wie ich das machen soll...

ich werde euch jetzt nicht gen ganzen Code reinkopieren aber hier mal der Teil:

/*
 * Adding Worker
 */
@Override
public void addWorker(String firstName, String lastName, double salary){
	Worker p = new Worker(firstName,lastName,salary);
	pListe.push(p);
}

das hier soll getestet werden. ein Worker wird mit firstName lastName und salary hinzugefügt in eine pListe

	@Test
	public void testGetWorkerCount(){
		assertTrue(enterprise.getWorkerCount()==0);
		testsPassed++;
	}
	
	@Test
	public void testaddWorker(){
		assertTrue(enterprise.addWorker("James", "Bond", 30000));
		testsPassed++;
	}

das obere geht, war auch vorgegeben,
das untere ist der von mir erstellte neue test addWorker

bei assertTrue meckert er aber... kann mir jemand verraten was ich da ändern muss

 

[faetzminator]

addWorker() gibt void zurück. Was willst du dort auf true überprüfen? Du kannst zum Beispiel diese Methode aufrufen und danach testen, ob size() 1 zurückgibt. Ob get...() wieder diese Werte zurückliefert etc.

 

[maki]

bei assertTrue meckert er aber... kann mir jemand verraten was ich da ändern muss

assertTrue erwartet eine boolean, die Methode liefert aber void

testsPassed++ klingt nicht so, als ob du JUnit richtig nutzt/verstehst.

Wenn pListe (schlechter Name übrigens) durch ein Mock zu ersetzen wäre, könnte das eine Möglichkeit sein, aber Code muss bestimmte Merkmale ausweisen um testbar bzw. einfach testbar zu sein.

 

[Wellensittich]

Stimmt das mit der Boolean Ausgabe kann nicht stimmen... hmm ansonsten weiß ich auch nicht wie ich das mache

 

[Gast2]

die Methode addWorker hat keinen Rückgabewert. Wenn nichts von der Methode zurückgegeben wird kann man ja auch nicht auswerten ob der Ausdruck true oder false ist.

EDIT:// zu spät 😉

 

[Wellensittich]

Wie kann ich dass dann machen, habe echt 0 plan wie das mit dem Testen gehen soll, wäre erfreut, wenn mir das jemand verraten könnte

 

[faetzminator]

So was?

    @Test
    public void testaddWorker() {
        enterprise.addWorker("James", "Bond", 30000)
        assertTrue(enterprise.getWorkerCount() == 0);
        Worker worker = enterprice.getWorker(); // oder wie auch immer...
        assertEquals(worker.getFirstName(), "James");
        assertEquals(worker.getLastName(), "Bond");
        assertEquals(worker.getSalary(), 30000);
    }

 

[maki]

Wie kann ich dass dann machen, habe echt 0 plan wie das mit dem Testen gehen soll, wäre erfreut, wenn mir das jemand verraten könnte

"Verraten" klingt so als ob es ein Geheimnis wäre wie man JUnit nutzt und Tests schreibt.

Das einzige Gemheimnis ist der Code den du testen willst, addWorker bietet da nicht viel, wenn du zB. in er Lage wärest dier diese "pList" geben zu lassen, könntest du ja prüfen, ob es einen Worker gibt mit den Werten die du übergeben hattest.

 

[Wellensittich]

hmm ok Danke ich schau mir das mal an

 

[Wellensittich]

  @Test
    public void testaddWorker() {
        enterprise.addWorker("James", "Bond", 30000);
        assertTrue(enterprise.getWorkerCount() == 0);
        Worker worker = enterprise.getWorker(); 
        assertEquals(worker.getfirstName(), "James");
        assertEquals(worker.getlastName(), "Bond");
        assertEquals(worker.getsalary(), 30000);
    }

bei getWorker();

sagt eclipse mir: The method getWorker() is undefined for the type Enterprise

 

[maki]

bei getWorker();

sagt eclipse mir: The method getWorker() is undefined for the type Enterprise

Sag mal... ohne jetzt böse klingen zu wollen... sollen wir dir den Code erklären den du hast und wir nicht kennen?

 

[Wellensittich]

Das Problem ist, dass ich den ganzen Code nicht ganz veröffentlichen will, da ich nicht möchte, dass mir das jemand 1:1 kopiert... hab dir deshalb ne PN geschickt, wäre nett du dir das anschaust, wenn du willst, vielleicht weißt du dann wo mein Problem besteht...danke

 

[faetzminator]

Wie wärs, wenn du einfach die Methode, welche dir den Worker wieder zurückgeben sollte, aufrufst? das [c]getWorker()[/c] war nur ein Beispiel / eine Idee.

 

[maki]

Wellensittich, keiner hier will deinen ganzen Code (ich eingeschlossen).
Ich mache keine Arbeit umsonst in PMs, das hier ist ein öffentliches Forum, PMs widersprechen diesem Prinzip.

Denke einfach dass dein problem ist dass du mit Unittest nix anfangen kannst und nicht wirklich weisst worum es geht, dir hilft kein Copy&Paste aus einem Forum, du brauchst die Grundlagen zum schreiben von Unittests.

Im Prinzip geht es ja nur darum, dass du irgendwie verifizieren kannst, dass der Code das macht was er soll, d.h. du brauchst eine Möglichekit zu überprüfen, ob addWorker einen neuen worker erstellt, mit den übergebenen Werten.

Ich hab ja schon geschrieben/gefragt, ob du die pList gebe lassen kannst, dann könnte der Test prüfen (und damit testen) ob der Worker hinzugefügt worden ist.
Wenn nicht, gibt es wohl keine Möglichkeit zu testen, ob addWorker wirklich macht was es soll, könntest höchstens versuchen die Größe der Liste zu überprüfen

 

[maki]

Bevor ich noch mehr an dem letzten Post editiere, hier mal ein bisschen ungetesteter (Pseudo) Testcode:

  @Test
    public void testaddWorker() {
        final String firstName = "James";
        final String lastName = "Bond";
        final int salary = 30000;

        enterprise.addWorker(firstName, lastName, salary);

        assertEquals("worker count", 1, enterprise.getWorkerCount());

        Worker worker = enterprise.getWorkerList().get(0); 

        assertEquals("firstName", firstName, worker.getfirstName());
        assertEquals("lastName", lastName, worker.getlastName());
        assertEquals("salary", salary, worker.getsalary());
    }

 

[Crian]

Man könnte jetzt einwenden, dass der Test zu viel testet, aber das geht wohl an der Problematik der TE vorbei.

Außerdem ist er so nicht unabhängig von der Ausführungsreihenfolge, da du das Objekt

enterprise

nicht selbst anlegst. Ist die Liste wirklich vorher leer?

Falls sich das nicht vermeiden lässt, ermittle die Größe vor dem Aufruf und danach und verwende auch einen passenden Index.

 

[maki]

Man könnte jetzt einwenden, dass der Test zu viel testet, aber das geht wohl an der Problematik der TE vorbei.

Das ist ein guter & richtiger Einwand, denn das Ergebnis des Tests hängt auch von der richtigen Funktionsweise der Worker Klasse ab.

Aber ohne die Möglichkeit Mocks zu verwenden wird jeder test früher oder später zu viel testen (und mutiert ganz schnell zum Integrationstests).

Außerdem ist er so nicht unabhängig von der Ausführungsreihenfolge, da du das Objekt enterprise nicht selbst anlegst. Ist die Liste wirklich vorher leer?

Falls sich das nicht vermeiden lässt, ermittle die Größe vor dem Aufruf und danach und verwende auch einen passenden Index.

Bevor man anfängt den index zu berechnen weil das Enterprise Objekt von mehreren Tests verwendet wird, sollte man lieber sicherstellen dass das Enterprise Objekt für jeden Test neu erstellt wird, "shared fixtures" sind ein Problem bei isolierten Unittests (und nciht nur da), setUp bzw. @Before bieten sich dafür an, oder im test ein neues Enterprise Objekt anlegen.
Sonst testen man nämlich auch zuviel 😉