From 4f0e2d6ff7c98759d8ea72afdb85a240f62ba40a Mon Sep 17 00:00:00 2001 From: Danny van Heumen Date: Mon, 11 Aug 2014 19:49:49 +0200 Subject: [PATCH] Fixed remaining failing cases to work with JUnit 3. --- .../protocol/irc/ChatRoomIrcImplTest.java | 54 ++++++++++++++++--- 1 file changed, 48 insertions(+), 6 deletions(-) diff --git a/test/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImplTest.java b/test/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImplTest.java index 077e5a4c5..c745a9845 100644 --- a/test/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImplTest.java +++ b/test/net/java/sip/communicator/impl/protocol/irc/ChatRoomIrcImplTest.java @@ -37,42 +37,84 @@ public void testConstruction() public void testConstructionNullIdentifier() { EasyMock.replay(this.providerMock, this.stackMock); - new ChatRoomIrcImpl(null, this.providerMock); + try + { + new ChatRoomIrcImpl(null, this.providerMock); + fail("Should have failed with IAE."); + } + catch (IllegalArgumentException e) + { + } } //@Test(expected = IllegalArgumentException.class) public void testConstructionNullProvider() { EasyMock.replay(this.providerMock, this.stackMock); - new ChatRoomIrcImpl("#test", null); + try + { + new ChatRoomIrcImpl("#test", null); + fail("Should have failed with IAE."); + } + catch (IllegalArgumentException e) + { + } } //@Test(expected = IllegalArgumentException.class) public void testEmptyName() { EasyMock.replay(this.providerMock, this.stackMock); - new ChatRoomIrcImpl("", this.providerMock); + try + { + new ChatRoomIrcImpl("", this.providerMock); + fail("Should have failed with IAE."); + } + catch (IllegalArgumentException e) + { + } } //@Test(expected = IllegalArgumentException.class) public void testIllegalNameBadPrefix() { EasyMock.replay(this.providerMock, this.stackMock); - new ChatRoomIrcImpl("!test", this.providerMock); + try + { + new ChatRoomIrcImpl("!test", this.providerMock); + fail("Should have failed with IAE."); + } + catch (IllegalArgumentException e) + { + } } //@Test(expected = IllegalArgumentException.class) public void testIllegalNameSpace() { EasyMock.replay(this.providerMock, this.stackMock); - new ChatRoomIrcImpl("#test test", this.providerMock); + try + { + new ChatRoomIrcImpl("#test test", this.providerMock); + fail("Should have failed with IAE."); + } + catch (IllegalArgumentException e) + { + } } //@Test(expected = IllegalArgumentException.class) public void testIllegalNameComma() { EasyMock.replay(this.providerMock, this.stackMock); - new ChatRoomIrcImpl("#test,test", this.providerMock); + try + { + new ChatRoomIrcImpl("#test,test", this.providerMock); + fail("Should have failed with IAE."); + } + catch (IllegalArgumentException e) + { + } } //@Test