JUNit Test IOException

[tanzverfuehrung]

also ich habe einem MEthode die zu testen ist

@Override

Invoked when any exception is thrown by user IoHandler implementation or by MINA. If cause is an instance of IOException, MINA will close the connection automatically.

Overrides: exceptionCaught(...) in IoHandlerAdapter
Parameters:
session
cause
Throws:
IOException

@Override
	public void exceptionCaught(IoSession session, Throwable cause) throws IOException {
		deviceLog.log(ERROR, ""+cause.getLocalizedMessage(), cause);
	}

meine JUnit test dazu

/**
	 * Prüft, ob eine erhaltenen {@link Exception} geworfen wird.
	 * 
	 * @throws Exception
	 */
	@Test(expected = IOException.class)
	public void testExceptionCaught() throws Exception {
		handler.exceptionCaught(session, new IOException("error"));
	}

die fehlermeldung:

java.lang.AssertionError: Expected exception: java.io.IOException
at org.junit.internal.runners.statements.ExpectException.evaluate(ExpectException.java:32)
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:28)
at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:31)
at org.junit.rules.ExpectedException$ExpectedExceptionStatement.evaluate(ExpectedException.java:110)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:76)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:193)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:52)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:191)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:42)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:184)
at org.junit.runners.ParentRunner.run(ParentRunner.java:236)
at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50)
at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:467)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:683)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:390)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:197)


kann mir jemand eventuell weiter helfen, wie ich diese methode am besten teste:bahnhof:

 

[faetzminator]

Natürlich throwt in diesem Fall [c]exceptionCaught()[/c] keine [c]IOException[/c]. Du musst aber überprüfen, ob diese Exception geloggt wird. Entweder kannst du das irgendwie im Nachhinein machen, oder du mockst [c]deviceLog[/c] und erkennst gleich den Aufruf von [c]log()[/c].

 

[turtle]

Das sehe ich wie faetzminator.

Was möchtest Du denn mit Deinem Test "beweisen"? Die Signatur gibt ja schon an, dass exceptionCaught eine IOException werfen kann. Dieses muss nicht getestet werden.

Bleibt deviceLog.log() und dieses kannst Du testen, indem Du eine Mock-Implementierung für deviceLog unterschiebst und verifiziert, dass da der Text drinsteht.

 

[tanzverfuehrung]

also ich habe den TEst neu implementiert

@Before
	public void setUp() throws Exception {
		modelManager = context.mock(IMsecModelManager.class);
		session = context.mock(IoSession.class);
		message = context.mock(IStaffMessage.class);
		deviceLog = context.mock(IDeviceLog.class);
		handler = new MsecIoHandler(modelManager, NULL_DEVICE_LOG);
	}

	@After
	public void tearDown() throws Exception {
		context.assertIsSatisfied();
	}

/**
	 * Prüft, ob die Log Ausgabe richig ausgegeben wird.
	 * 
	 * @throws Exception
	 */
	@Test
	public void testExceptionCaught() throws Exception {
		final Throwable cause = new Throwable();
		//@formatter:off
			context.checking(new Expectations(){{
				oneOf(deviceLog).log(ERROR, ""+cause.getLocalizedMessage(), cause);	
				}});// @formatter:on
		handler.exceptionCaught(session, cause);
	}

so was mich nun wundert...wenn ich es so habe, schlägt der Test fehl!

java.lang.AssertionError: not all expectations were satisfied
expectations:
expected once, never invoked: iDeviceLog.log(<ERROR>, "null", <java.lang.Throwable>); returns a default value
at org.jmock.lib.AssertionErrorTranslator.translate(AssertionErrorTranslator.java:20)
at org.jmock.Mockery.assertIsSatisfied(Mockery.java:196)
at eu.gemtec.eagle.device.gemtec.msec.core.system.TestMsecServerHandler.tearDown(TestMsecServerHandler.java:74)

wenn ich auf diese Zeile klicke ,komme ich zum
@After
public void tearDown() throws Exception {
context.assertIsSatisfied();
}

, also wenn ich das
context.assertIsSatisified rausnehme funktioniert es!DEr test ist GRÜN!
aber wieso ist dAS so???:bahnhof:

 

[tanzverfuehrung]

das gleiche Problem habe ich auch in einer anderen TestKlasse

Testen wollte ich diese MEthode

/**
	 * Überprüft die Positionsangabe der {@link IStaffMessage} auf ihre
	 * Korrektheit. Sind die Angaben der {@link Position#latitude} und/oder
	 * {@link Position#longitude} falsch, wird dies geloggt.
	 * 
	 * @param message
	 *            {@link IStaffMessage}, die die Positionsangabe enthält
	 * @return <code>true</code>, wenn die Postionsange korrekt ist, für
	 *         {@link Position#longitude} zwischen 180 und -180 und für
	 *         {@link Position#latitude} zwischen 90 und -90 <br>
	 *         <code>false</code>, wenn dies nicht der Fall
	 */
	private boolean hasCorrectPosition(IStaffMessage message) {
		Position position = message.getCurrentPos();
		double longitude = position.longitude;
		double latitude = position.latitude;

		if (longitude < -180 || longitude > 180  || longitude == NaN) {
			log(WARNING, "Invalid longitude: >" + longitude + "< in message: "+message);
			return false;
		}

		if (latitude < -90 || latitude > 90 || longitude == NaN) {
			log(WARNING, "Invalid latitude >"+latitude+"< in message: "+message);
			return false;
		}
		return true;
	}

für die Latitude funktioniert es egal ob ich
@After
public void tearDown() throws Exception {
context.assertIsSatisfied();
}
mache oder nicht!

	/**
	 * Prüft, ob ein Long-Eintrag gemacht wird, wenn die
	 * {@link Position#latitude} einen Wert außerhalb des Intervalls (-90-90)
	 * hat.
	 * 
	 * @throws Exception
	 */
	@Test
	public void testUpdateSystemModelWithIncorrectPositionInfoLatitude() throws Exception {
		final IStaffMessage message = context.mock(IStaffMessage.class);
		final IStaff staff = context.mock(IStaff.class);

		//@formatter:off
		context.checking(new Expectations(){{
			allowing(staff).getId();
				will(returnValue("2"));
			allowing(message).getId();
				will(returnValue("2"));
			allowing(message).getCurrentPos();
				will(returnValue(new Position(12.3, 100)));
			oneOf(deviceLog).log(WARNING, "Invalid latitude >100.0< in message: iStaffMessage");
		}});//@formatter:on
		modelManager.setModel(newHashSet(staff));
		modelManager.updateSystemModel(message);
	}

oder bei der Longitude geht mein test bloß wenn ich
@After
public void tearDown() throws Exception {
context.assertIsSatisfied();
} auskommentiere

/**
	 * Prüft, ob ein Log-Eintrag gemacht wird, wenn die
	 * {@link Position#longitude} einen Wert außerhalb des Intervalls (-180,180)
	 * hat.
	 * 
	 * @throws Exception
	 */
	@Test
	public void testUpdateSystemModelWithIncorrectPositionInfoLongitude() throws Exception {
		final IStaffMessage message = context.mock(IStaffMessage.class);
		final IStaff staff = context.mock(IStaff.class);

		//@formatter:off
		context.checking(new Expectations(){{
			allowing(staff).getId();
				will(returnValue("2"));
			allowing(message).getId();
				will(returnValue("2"));
			allowing(message).getCurrentPos();
				will(returnValue(new Position(NaN, 12.3)));
				oneOf(deviceLog).log(WARNING, "Invalid longitude >100.0< in message: iStaffMessage");
			oneOf(staff).setCurrentPosition(new Position(NaN, 12.3));
			oneOf(message).getState();
				will(returnValue(IStaff.State.AVAILABLE));
			oneOf(staff).setState(IStaff.State.AVAILABLE);		
		}});//@formatter:on
		modelManager.setModel(newHashSet(staff));
		modelManager.updateSystemModel(message);
	}

WIESO????????

 

[tanzverfuehrung]

so also dann werde ich wohl mal selber beantworten...

wenn man objekte mockt ist context.assertIsSatisfied(); sehr wichtig und aknn nicht weg gelassen werden!!!!:toll::applaus:

wenn das der grund ist, weshalb der test nicht geht!

haben sie ein fehler!!!im Test oder in der klasse:rtfm: