JUnit Suite frage

[Thaitanium]

Hallo, ich bin neu bei Java und im Forum.

Ich habe eine frage und zwar habe ich zwei Testklassen in einem Package sowie eine Testsuite in dem selben Package. Diese Testsuite führt beide Testklassen nacheinander aus. Das funktioniert auch alles einwandfrei. Wenn nun Testklasse A fehlschlägt wird eine Mail mit einer Log Datei verschickt. Dann läuft Testklasse B durch und läuft durch. Diese information steht nun aber nicht in der Log Datei der Mail weil diese ja schon vorher verschickt wird (nämlich beim fehlschlagen der Testklasse A). Ich möchte das ganze gern so implementieren, dass beide Testklassen erst komplett durchlaufen und dann geschaut wird ob eine davon fehlschlägt und erst wenn das der Fall ist, soll eine Mail versendet werden.
Leider habe ich gerade keine Idee wie ich das umsetzen kann. Hat jemand eine Idee?

Besten Dank im Voraus.

 

[Firephoenix]

Ich kenne den genauen Aufbau leider nicht, aber ist es nicht Möglich dieLogging-logik von A auf die Ebene der Suite heraufzuziehen?
JUnit ist doch beim Zusammenfassen von Tests relativ modular.
Gruß

 

[Thaitanium]

Hallo Firephoenix,

leider weiss ich nicht genau wie du das meinst. In der Suite rufe ich nur die Zwei Testklassen auf.
Jetzt versuche ich gerade irgendwie das Email versenden in der Suite einzubauen, und zwar so dass nur eine Mail gesendet wird wenn ein Test fehlschlägt und dementsprechend auch erst dann wenn beide Testklassen durch sind. Da ich noch Anfänger bin weiß ich leider noch nicht so genau wie ich das implementieren soll. Aber das wäre so mein erster Lösungsansatz. Vllt fällt Dir ja noch was ein.

Gruß

 

[Firephoenix]

Wie genau sieht dein Mail-versenden denn aus?
Prüfst du händisch in jedem Test ob er fehlschlagen wird und gibst das entsprechend in das Log oder arbeitest du mit irgendwelchen Listenern von JUnit?

Ich hab gerade mal bei mir eine Version gebastelt bei der ich eine Suite in Eclipse mit JUnit 4 ausführe und dabei einen eigenen Runner verwende um noch ein Log dazu zu basteln (ergibt die normale Testview in Eclipse und ordentlich Konsolenoutput). Ich weiß aber nicht wie weit du das auf deine Anforderungen adaptieren kannst (bin selber kein JUnit-Experte - vielleicht kennt noch jemand einen eleganteren Weg?).
Aus den Description bzw Result-Klassen von Junit kriegt man aber ne Menge infos raus denke ich.

Hab dir meine Klassen mal angehängt, zum Ausführen in Eclipse einfach die AllTests als Junit-Test starten.

import org.junit.runner.RunWith;
import org.junit.runners.Suite.SuiteClasses;

@RunWith(MySuite.class)
@SuiteClasses({ TestsA.class, TestsB.class })
public class AllTests {

}

import org.junit.runner.notification.RunNotifier;
import org.junit.runners.Suite;
import org.junit.runners.model.InitializationError;
import org.junit.runners.model.RunnerBuilder;

public class MySuite extends Suite {

	// den konstruktor scheint junit zu verwenden, die anderen habe ich daher
	// nicht überschrieben (suite hat noch 4 andere Konstruktoren)
	public MySuite(Class<?> klass, RunnerBuilder builder)
			throws InitializationError {
		super(klass, builder);
	}

	@Override
	public void run(RunNotifier rn) {
		rn.addListener(new LoggingRunListener());
		super.run(rn);
	}

}

import org.junit.runner.Description;
import org.junit.runner.Result;
import org.junit.runner.notification.Failure;
import org.junit.runner.notification.RunListener;

public class LoggingRunListener extends RunListener {

	private List<String> logs = new ArrayList<String>();

	public void testFailure(Failure failure) throws Exception {
		super.testFailure(failure);
		logs.add("Fehler: " + failure.toString());
	}

	public void testFinished(Description description) throws Exception {
		super.testFinished(description);
		logs.add("Finished: " + description.toString());
	}

	public void testIgnored(Description description) throws Exception {
		super.testIgnored(description);
		logs.add("Ignored: " + description.toString());
	}

	public void testRunFinished(Result result) throws Exception {
		super.testRunFinished(result);		
		printLogs();
		printResult(result);
	}

	private void printResult(Result r) {
		System.out.println("Result");
		String result = "Ausführungszeit: " + r.getRunTime() + " Tests: "
				+ r.getRunCount() + " Ignoriert: " + r.getIgnoreCount()
				+ " Fehlgeschlagen: " + r.getFailureCount();
		System.out.println(result);
		for (Failure f : r.getFailures()) {
			System.out.println("Fehler: "+f.getTestHeader());
			System.out.println(f.getException().toString());
		}
	}

	private void printLogs() {
		for (String log : logs) {
			System.out.println(log);
		}

	}

	public void testRunStarted(Description description) throws Exception {
		super.testRunStarted(description);
		logs.add("Run started: " + description.toString());
	}

	public void testStarted(Description description) throws Exception {
		super.testStarted(description);
		logs.add("Test gestartet: " + description.toString());
	}
}

public class TestsA {

	
	@Test
	@Ignore
	public void test0(){
	}
	
	@Test
	public void test1(){
		assertTrue("Fehler im test1",false);
	}
	
	@Test
	public void test2(){
		assertTrue(true);
	}
}

public class TestsB {

	
	@Test
	public void test3(){
		assertTrue("Fehler im test3",false);
	}
	
	@Test
	public void test4(){
		assertTrue(true);
	}
	
	@Test
	public void test(){
		throw new RuntimeException();
	}
}

Meine spartanische Ausgabe:

Ignored: test0(tests.TestsA)
Test gestartet: test1(tests.TestsA)
Fehler: test1(tests.TestsA): Fehler im test1
Finished: test1(tests.TestsA)
Test gestartet: test2(tests.TestsA)
Finished: test2(tests.TestsA)
Test gestartet: test(tests.TestsB)
Fehler: test(tests.TestsB): null
Finished: test(tests.TestsB)
Test gestartet: test3(tests.TestsB)
Fehler: test3(tests.TestsB): Fehler im test3
Finished: test3(tests.TestsB)
Test gestartet: test4(tests.TestsB)
Finished: test4(tests.TestsB)
Result
Ausführungszeit: 18 Tests: 5 Ignoriert: 1 Fehlgeschlagen: 3
Fehler: test1(tests.TestsA)
java.lang.AssertionError: Fehler im test1
Fehler: test(tests.TestsB)
java.lang.RuntimeException
Fehler: test3(tests.TestsB)
java.lang.AssertionError: Fehler im test3

Hilft dir das weiter?
Gruß

 

[Thaitanium]

Hey also erstmal Danke schonmal für Deine Mühe.

Ich habe eine Klasse die sich aufs Mailen + LogDatei im Anhang konzentriert. Die macht nix anderes außer sich beim Server einzuloggen eine Mail mit text + Log im Anhang zu verschicken.
Testklasse A macht dann seinen Kram in einem Try Catch Block und in dem Catch Block wird dann eben die Mail versenden Klasse aufgerufen. Selbes Spiel für Testklasse B. Wenn jetzt TestA fehlschlägt schreibt er das direkt ins Log und ruft die MailSenden Methode aus der Mail Klasse auf. Problem ist nur dass TestB ja auch noch was in die Log schreibt, was dann aber nicht mit in der Mail steht weil diese ja vorher schon verschickt wird. Ich würde gerne das Ergebnis aus TestA und das Ergebnis aus TestB in jeweils eigene Variablen schreiben wenn das geht. Ich weiß aber auch nicht ob das Sinn macht... Vllt gibts da schönere Lösungen. Ich hoffe das Problem der unvollständigen Log-Datei in der Mail wird klar.
Wenn ich die Testklassen einzeln ausführe läuft ja alles wunderbar, da in beiden Klassen jeder für sich das mail senden aufruft.
Hilft es vllt wenn ich ein paar Auszüge aus meinem Code Poste?

Gruß

 

[Firephoenix]

Try-catch sollte in JUnit eigentlich (fast) nicht benötigt werden (Ausnahmen gibt es immer )
Wenn du einen groben Überblick geben kannst wäre das super, muss aber nicht der ganze Code sein, ein grobes Schema vom Aufbau reicht.
Gruß

 

[Thaitanium]

Hey,
Problem ist gelöst.
Meine JUnit klassen wurden von der Basis her mit Selenium erstellt. Da hat er von sich aus schon einen try catch Block erstellt.
Ich hab das so übernommen und entsprechend meinen Anforderungen angepasst.
Das Problem hab ich dann über boolsche Variablen gelöst die im Catchblock dann entsprechend auf True gesetzt wird und in einer anderen Klasse dann abgefragt wird... Musste so um zwei Ecken gelöst werden. Was anderes viel mir aber nicht ein und ich streite auch nicht ab, dass es elegantere Lösungen gibt. Aber es läuft und das reicht mir erstmal.

Und Danke nochmal für Deine Hilfe.
Gruß.