From 9d0ea3cf70506bef7fcd0e83d5abd71a75661040 Mon Sep 17 00:00:00 2001 From: stawny Date: Tue, 24 Mar 2015 23:52:56 +0100 Subject: [PATCH] Resurrect integration tests: Chat Test --- .../java/org/jivesoftware/smack/ChatTest.java | 95 ------------ .../java/org/jivesoftware/smack/ChatTest.java | 135 ++++++++++++++++++ 2 files changed, 135 insertions(+), 95 deletions(-) delete mode 100644 smack-core/src/integration-test/java/org/jivesoftware/smack/ChatTest.java create mode 100644 smack-integration-test/src/main/java/org/jivesoftware/smack/ChatTest.java diff --git a/smack-core/src/integration-test/java/org/jivesoftware/smack/ChatTest.java b/smack-core/src/integration-test/java/org/jivesoftware/smack/ChatTest.java deleted file mode 100644 index f1681f116..000000000 --- a/smack-core/src/integration-test/java/org/jivesoftware/smack/ChatTest.java +++ /dev/null @@ -1,95 +0,0 @@ -/** - * - * Copyright 2004 Jive Software. - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -package org.jivesoftware.smack; - -import java.util.Date; - -import org.jivesoftware.smack.packet.Message; -import org.jivesoftware.smack.test.SmackTestCase; -import org.jivesoftware.smack.filter.ThreadFilter; - - -/** - * Tests the chat functionality. - * - * @author Gaston Dombiak - */ -public class ChatTest extends SmackTestCase { - - public ChatTest(String arg0) { - super(arg0); - } - - public void testProperties() { - try { - Chat newChat = getConnection(0).getChatManager().createChat(getFullJID(1), null); - PacketCollector collector = getConnection(1) - .createPacketCollector(new ThreadFilter(newChat.getThreadID())); - - Message msg = new Message(); - - msg.setSubject("Subject of the chat"); - msg.setBody("Body of the chat"); - msg.setProperty("favoriteColor", "red"); - msg.setProperty("age", 30); - msg.setProperty("distance", 30f); - msg.setProperty("weight", 30d); - msg.setProperty("male", true); - msg.setProperty("birthdate", new Date()); - newChat.sendMessage(msg); - - Message msg2 = (Message) collector.nextResult(2000); - - assertNotNull("No message was received", msg2); - assertEquals("Subjects are different", msg.getSubject(), msg2.getSubject()); - assertEquals("Bodies are different", msg.getBody(), msg2.getBody()); - assertEquals( - "favoriteColors are different", - msg.getProperty("favoriteColor"), - msg2.getProperty("favoriteColor")); - assertEquals( - "ages are different", - msg.getProperty("age"), - msg2.getProperty("age")); - assertEquals( - "distances are different", - msg.getProperty("distance"), - msg2.getProperty("distance")); - assertEquals( - "weights are different", - msg.getProperty("weight"), - msg2.getProperty("weight")); - assertEquals( - "males are different", - msg.getProperty("male"), - msg2.getProperty("male")); - assertEquals( - "birthdates are different", - msg.getProperty("birthdate"), - msg2.getProperty("birthdate")); - } - catch (XMPPException e) { - e.printStackTrace(); - fail(e.getMessage()); - } - } - - protected int getMaxConnections() { - return 2; - } -} diff --git a/smack-integration-test/src/main/java/org/jivesoftware/smack/ChatTest.java b/smack-integration-test/src/main/java/org/jivesoftware/smack/ChatTest.java new file mode 100644 index 000000000..0e17c5e76 --- /dev/null +++ b/smack-integration-test/src/main/java/org/jivesoftware/smack/ChatTest.java @@ -0,0 +1,135 @@ +/** + * + * Copyright 2015 Florian Schmaus + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.jivesoftware.smack; + +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertTrue; +import static org.jivesoftware.smackx.jiveproperties.JivePropertiesManager.addProperty; +import static org.jivesoftware.smackx.jiveproperties.JivePropertiesManager.getProperty; + +import java.util.Date; + +import org.junit.AfterClass; +import org.junit.BeforeClass; +import org.igniterealtime.smack.inttest.AbstractSmackIntegrationTest; +import org.igniterealtime.smack.inttest.SmackIntegrationTest; +import org.igniterealtime.smack.inttest.SmackIntegrationTestEnvironment; +import org.jivesoftware.smack.SmackException.NotConnectedException; +import org.jivesoftware.smack.chat.Chat; +import org.jivesoftware.smack.chat.ChatManager; +import org.jivesoftware.smack.chat.ChatManagerListener; +import org.jivesoftware.smack.filter.ThreadFilter; +import org.jivesoftware.smack.packet.Message; +import org.jivesoftware.smackx.jiveproperties.JivePropertiesManager; +import org.jxmpp.stringprep.XmppStringprepException; + +/** + * Tests for Chat Manager and for Chat Manager Listener. + * + * @author Stawicki Adam + */ +public class ChatTest extends AbstractSmackIntegrationTest { + + private final ChatManager chatManagerOne; + private boolean invoked; + + public ChatTest(SmackIntegrationTestEnvironment environment) { + super(environment); + chatManagerOne = ChatManager.getInstanceFor(conOne); + } + + @BeforeClass + public void setUp() { + JivePropertiesManager.setJavaObjectEnabled(true); + } + + @AfterClass + public void tearDown() { + JivePropertiesManager.setJavaObjectEnabled(false); + } + + @SmackIntegrationTest + public void testProperties() throws XmppStringprepException, NotConnectedException, Exception { + Chat newChat = chatManagerOne.createChat(conTwo.getUser()); + PacketCollector collector = conTwo.createPacketCollector(new ThreadFilter(newChat.getThreadID())); + + Message msg = new Message(); + + msg.setSubject("Subject of the chat"); + msg.setBody("Body of the chat"); + addProperty(msg, "favoriteColor", "red"); + addProperty(msg, "age", 30); + addProperty(msg, "distance", 30f); + addProperty(msg, "weight", 30d); + addProperty(msg, "male", true); + addProperty(msg, "birthdate", new Date()); + + newChat.sendMessage(msg); + + Message msg2 = (Message) collector.nextResult(2000); + assertNotNull("No message was received", msg2); + assertEquals("Subjects are different", msg.getSubject(), msg2.getSubject()); + assertEquals("Bodies are different", msg.getBody(), msg2.getBody()); + assertEquals( + "favoriteColors are different", + getProperty(msg, "favoriteColor"), + getProperty(msg2, "favoriteColor")); + assertEquals( + "ages are different", + getProperty(msg, "age"), + getProperty(msg2, "age")); + assertEquals( + "distances are different", + getProperty(msg, "distance"), + getProperty(msg2, "distance")); + assertEquals( + "weights are different", + getProperty(msg, "weight"), + getProperty(msg2, "weight")); + assertEquals( + "males are different", + getProperty(msg, "male"), + getProperty(msg2, "male")); + assertEquals( + "birthdates are different", + getProperty(msg, "birthdate"), + getProperty(msg2, "birthdate")); + } + + @SmackIntegrationTest + public void chatManagerTest() { + ChatManagerListener listener = new ChatManagerListener() { + + @Override + public void chatCreated(Chat chat, boolean createdLocally) { + invoked = true; + } + + }; + try { + chatManagerOne.addChatListener(listener); + chatManagerOne.createChat(conTwo.getUser()); + + assertTrue("TestChatManagerListener wasn't invoked", invoked); + } + finally { + chatManagerOne.removeChatListener(listener); + } + } +}