You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by be...@apache.org on 2009/04/24 22:00:15 UTC

svn commit: r768408 - in /mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler: PresenceHandlerBaseTestCase.java PresenceSubRequestOutHandlerTestCase.java

Author: berndf
Date: Fri Apr 24 20:00:15 2009
New Revision: 768408

URL: http://svn.apache.org/viewvc?rev=768408&view=rev
Log:
extract method to get message more elegantly

Modified:
    mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceHandlerBaseTestCase.java
    mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceSubRequestOutHandlerTestCase.java

Modified: mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceHandlerBaseTestCase.java
URL: http://svn.apache.org/viewvc/mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceHandlerBaseTestCase.java?rev=768408&r1=768407&r2=768408&view=diff
==============================================================================
--- mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceHandlerBaseTestCase.java (original)
+++ mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceHandlerBaseTestCase.java Fri Apr 24 20:00:15 2009
@@ -168,6 +168,15 @@
         StanzaReceiverRelay receiverRelay = (StanzaReceiverRelay) sessionContext.getServerRuntimeContext().getStanzaRelay();
         receiverRelay.resetAll();
     }
+
+    /**
+     * searches for the next message captured by the test session which was sent directly to the given user's resource 
+     * @param testUser
+     * @return NULL, if no stanza available matching the user's resource id
+     */
+    protected Stanza getNextDirectResponseFor(TestUser testUser) {
+        return sessionContext.getNextRecordedResponseForResource(testUser.getBoundResourceId());
+    }
 }
 
 class TestUser {

Modified: mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceSubRequestOutHandlerTestCase.java
URL: http://svn.apache.org/viewvc/mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceSubRequestOutHandlerTestCase.java?rev=768408&r1=768407&r2=768408&view=diff
==============================================================================
--- mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceSubRequestOutHandlerTestCase.java (original)
+++ mina/sandbox/vysper/trunk/src/test/java/org/apache/vysper/xmpp/modules/core/im/handler/PresenceSubRequestOutHandlerTestCase.java Fri Apr 24 20:00:15 2009
@@ -69,23 +69,23 @@
         assertStanzasReceivedDirectly(3);
 
         // roster push for 1 interested initiator of _same_ session
-        Stanza initiatorNotification = sessionContext.getNextRecordedResponseForResource(initiatingUser.getBoundResourceId());
+        Stanza initiatorNotification = getNextDirectResponseFor(initiatingUser);
         assertTrue(checkRosterPush(initiatorNotification, initiatingUser.getEntityFQ(), unrelatedUser.getEntity(), NONE, ASK_SUBSCRIBE));
 
         // no stanzas for not interested
-        assertNull(sessionContext.getNextRecordedResponseForResource(anotherAvailableUser.getBoundResourceId()));
+        assertNull(getNextDirectResponseFor(anotherAvailableUser));
 
         // roster 2 interested resources of _same_ session...
         
         // roster push for interested
-        Stanza interestedResourceNotification = sessionContext.getNextRecordedResponseForResource(anotherInterestedUser.getBoundResourceId());
+        Stanza interestedResourceNotification = getNextDirectResponseFor(anotherInterestedUser);
         assertTrue(checkRosterPush(interestedResourceNotification, new EntityImpl(initiatingUser.getEntity(), anotherInterestedUser.getBoundResourceId()), unrelatedUser.getEntity(), NONE, ASK_SUBSCRIBE));
-        assertNull(sessionContext.getNextRecordedResponseForResource(anotherInterestedUser.getBoundResourceId())); // no more stanzas
+        assertNull(getNextDirectResponseFor(anotherInterestedUser)); // no more stanzas;
 
         // roster push for interested but not avail
-        Stanza interestedNotYetAvailResourceNotification = sessionContext.getNextRecordedResponseForResource(anotherInterestedNotAvailUser.getBoundResourceId());
+        Stanza interestedNotYetAvailResourceNotification = getNextDirectResponseFor(anotherInterestedNotAvailUser);
         assertTrue(checkRosterPush(interestedNotYetAvailResourceNotification, new EntityImpl(initiatingUser.getEntity(), anotherInterestedNotAvailUser.getBoundResourceId()), unrelatedUser.getEntity(), NONE, ASK_SUBSCRIBE));
-        assertNull(sessionContext.getNextRecordedResponseForResource(anotherInterestedNotAvailUser.getBoundResourceId())); // no more stanzas;
+        assertNull(getNextDirectResponseFor(anotherInterestedNotAvailUser)); // no more stanzas;
 
         // sub request sent to contact
         assertTrue(checkPresence(unrelatedUser.getNextStanza(), PresenceStanzaType.SUBSCRIBE, initiatingUser.getEntity(), null));
@@ -111,13 +111,13 @@
         assertStanzasReceivedDirectly(3);
 
         // roster push for 1 interested initiator...
-        Stanza initiatorNotification = sessionContext.getNextRecordedResponseForResource(initiatingUser.getBoundResourceId());
+        Stanza initiatorNotification = getNextDirectResponseFor(initiatingUser);
         assertTrue(checkRosterPush(initiatorNotification, initiatingUser.getEntityFQ(), subscribed_FROM.getEntity(), FROM, ASK_SUBSCRIBE));
 
         // .. and 2 interested resources of _same_ session
-        Stanza anotherInterestedUserNotification = sessionContext.getNextRecordedResponseForResource(anotherInterestedUser.getBoundResourceId());
+        Stanza anotherInterestedUserNotification = getNextDirectResponseFor(anotherInterestedUser);
         assertTrue(checkRosterPush(anotherInterestedUserNotification, anotherInterestedUser.getEntityFQ(), subscribed_FROM.getEntity(), FROM, ASK_SUBSCRIBE));
-        Stanza anotherInterestedNotAvailUserNotification = sessionContext.getNextRecordedResponseForResource(anotherInterestedNotAvailUser.getBoundResourceId());
+        Stanza anotherInterestedNotAvailUserNotification = getNextDirectResponseFor(anotherInterestedNotAvailUser);
         assertTrue(checkRosterPush(anotherInterestedNotAvailUserNotification, anotherInterestedNotAvailUser.getEntityFQ(), subscribed_FROM.getEntity(), FROM, ASK_SUBSCRIBE));
 
         assertNull(sessionContext.getNextRecordedResponse()); // no more stanzas