Skip to content

Commit

Permalink
[sint] split off low-level tests
Browse files Browse the repository at this point in the history
  • Loading branch information
guusdk committed Nov 26, 2023
1 parent acbc4a5 commit 9a90e7b
Show file tree
Hide file tree
Showing 2 changed files with 121 additions and 73 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,121 @@
/**
*
* Copyright 2023 Guus der Kinderen
*
* 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.subscription;

import org.jivesoftware.smack.AbstractXMPPConnection;
import org.jivesoftware.smack.filter.AndFilter;
import org.jivesoftware.smack.filter.FromMatchesFilter;
import org.jivesoftware.smack.filter.PresenceTypeFilter;
import org.jivesoftware.smack.filter.StanzaExtensionFilter;
import org.jivesoftware.smack.filter.StanzaFilter;
import org.jivesoftware.smack.packet.Presence;
import org.jivesoftware.smack.packet.StandardExtensionElement;

import org.igniterealtime.smack.inttest.AbstractSmackLowLevelIntegrationTest;
import org.igniterealtime.smack.inttest.SmackIntegrationTestEnvironment;
import org.igniterealtime.smack.inttest.annotations.SmackIntegrationTest;
import org.igniterealtime.smack.inttest.util.IntegrationTestRosterUtil;
import org.igniterealtime.smack.inttest.util.SimpleResultSyncPoint;

/**
* Integration tests that verify that sent presence subscription requests are received as intended.
*
* @author Guus der Kinderen, [email protected]
*/
public class LowLevelSubscriptionIntegrationTest extends AbstractSmackLowLevelIntegrationTest {
public LowLevelSubscriptionIntegrationTest(SmackIntegrationTestEnvironment environment) {
super(environment);
}

/**
* This test verifies that a subscription request is received, in a scenario where the intended recipient was
* offline when the request was made.
*
* @param conOne Connection used to receive subscription request.
* @param conTwo Conenction used to send subscription request.
* @throws Exception on anything unexpected or undesired.
*/
@SmackIntegrationTest
public void testSubscriptionRequestOffline(final AbstractXMPPConnection conOne,
final AbstractXMPPConnection conTwo) throws Exception {
IntegrationTestRosterUtil.ensureBothAccountsAreNotInEachOthersRoster(conOne, conTwo);

final Presence subscriptionRequest = conTwo.getStanzaFactory().buildPresenceStanza()
.ofType(Presence.Type.subscribe)
.to(conOne.getUser().asBareJid())
.build();

conOne.disconnect();

conTwo.sendStanza(subscriptionRequest);

conOne.connect();

final SimpleResultSyncPoint received = new SimpleResultSyncPoint();

final StanzaFilter resultFilter = new AndFilter(
PresenceTypeFilter.SUBSCRIBE,
FromMatchesFilter.createBare(conTwo.getUser())
);

conOne.addAsyncStanzaListener(p -> received.signal(), resultFilter);

conOne.login();
received.waitForResult(timeout);
}

/**
* When a subscription request is made, the stanza can have additional extension elements. This test verifies that
* such extension elements are received, in a scenario where the intended recipient was offline when the request
* was made.
*
* @param conOne Connection used to receive subscription request.
* @param conTwo Conenction used to send subscription request.
* @see <a href="https://igniterealtime.atlassian.net/browse/OF-2244">Openfire issue OF-2244</a>
* @throws Exception on anything unexpected or undesired.
*/
@SmackIntegrationTest
public void testSubscriptionRequestOfflineWithExtension(final AbstractXMPPConnection conOne,
final AbstractXMPPConnection conTwo) throws Exception {
IntegrationTestRosterUtil.ensureBothAccountsAreNotInEachOthersRoster(conOne, conTwo);

final Presence subscriptionRequest = conTwo.getStanzaFactory().buildPresenceStanza()
.ofType(Presence.Type.subscribe)
.to(conOne.getUser().asBareJid())
.addExtension(new StandardExtensionElement("test", "org.example.test"))
.build();

conOne.disconnect();

conTwo.sendStanza(subscriptionRequest);

conOne.connect();

final SimpleResultSyncPoint received = new SimpleResultSyncPoint();

final StanzaFilter resultFilter = new AndFilter(
PresenceTypeFilter.SUBSCRIBE,
FromMatchesFilter.createBare(conTwo.getUser()),
new StanzaExtensionFilter("test", "org.example.test")
);

conOne.addAsyncStanzaListener(p -> received.signal(), resultFilter);

conOne.login();
received.waitForResult(timeout);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
*/
package org.jivesoftware.smack.subscription;

import org.jivesoftware.smack.AbstractXMPPConnection;
import org.jivesoftware.smack.filter.AndFilter;
import org.jivesoftware.smack.filter.FromMatchesFilter;
import org.jivesoftware.smack.filter.PresenceTypeFilter;
Expand Down Expand Up @@ -69,40 +68,6 @@ public void testSubscriptionRequest() throws Exception {
received.waitForResult(timeout);
}

/**
* This test verifies that a subscription request is received, in a scenario where the intended recipient was
* offline when the request was made.
*
* @throws Exception on anything unexpected or undesired.
*/
@SmackIntegrationTest
public void testSubscriptionRequestOffline() throws Exception {
IntegrationTestRosterUtil.ensureBothAccountsAreNotInEachOthersRoster(conOne, conTwo);

((AbstractXMPPConnection) conOne).disconnect();

final Presence subscriptionRequest = conTwo.getStanzaFactory().buildPresenceStanza()
.ofType(Presence.Type.subscribe)
.to(conOne.getUser())
.build();

conTwo.sendStanza(subscriptionRequest);

((AbstractXMPPConnection) conOne).connect();

final SimpleResultSyncPoint received = new SimpleResultSyncPoint();

final StanzaFilter resultFilter = new AndFilter(
PresenceTypeFilter.SUBSCRIBE,
FromMatchesFilter.createBare(conTwo.getUser())
);

conOne.addAsyncStanzaListener(p -> received.signal(), resultFilter);

((AbstractXMPPConnection) conOne).login();
received.waitForResult(timeout);
}

/**
* When a subscription request is made, the stanza can have additional extension elements. This test verifies that
* such extension elements are received.
Expand Down Expand Up @@ -132,42 +97,4 @@ public void testSubscriptionRequestWithExtension() throws Exception {
conTwo.sendStanza(subscriptionRequest);
received.waitForResult(timeout);
}

/**
* When a subscription request is made, the stanza can have additional extension elements. This test verifies that
* such extension elements are received, in a scenario where the intended recipient was offline when the request
* was made.
*
* @see <a href="https://igniterealtime.atlassian.net/browse/OF-2244">Openfire issue OF-2244</a>
* @throws Exception on anything unexpected or undesired.
*/
@SmackIntegrationTest
public void testSubscriptionRequestOfflineWithExtension() throws Exception {
IntegrationTestRosterUtil.ensureBothAccountsAreNotInEachOthersRoster(conOne, conTwo);

((AbstractXMPPConnection) conOne).disconnect();

final Presence subscriptionRequest = conTwo.getStanzaFactory().buildPresenceStanza()
.ofType(Presence.Type.subscribe)
.to(conOne.getUser())
.addExtension(new StandardExtensionElement("test", "org.example.test"))
.build();

conTwo.sendStanza(subscriptionRequest);

((AbstractXMPPConnection) conOne).connect();

final SimpleResultSyncPoint received = new SimpleResultSyncPoint();

final StanzaFilter resultFilter = new AndFilter(
PresenceTypeFilter.SUBSCRIBE,
FromMatchesFilter.createBare(conTwo.getUser()),
new StanzaExtensionFilter("test", "org.example.test")
);

conOne.addAsyncStanzaListener(p -> received.signal(), resultFilter);

((AbstractXMPPConnection) conOne).login();
received.waitForResult(timeout);
}
}

0 comments on commit 9a90e7b

Please sign in to comment.