Only allow owner unsubscribing from a node if there are more than 1 owner

Author: Lloyd Watkin

src/main/java/org/buddycloud/channelserver/packetprocessor/iq/namespace/pubsub/set/UnsubscribeSet.java

@@ -108,7 +108,9 @@ public void process(Element elm, JID actorJID, IQ reqIQ, Element rsm)
 			return;
 		}
 
-		if (Affiliations.owner == existingAffiliation.getAffiliation()) {
+		if ((Affiliations.owner == existingAffiliation.getAffiliation()) &&
+			(channelManager.getNodeOwners(node).size() < 2)) {
+			
 			response.setType(Type.error);
 			PacketError pe = new PacketError(
 					org.xmpp.packet.PacketError.Condition.not_allowed,

src/test/java/org/buddycloud/channelserver/packetprocessor/iq/namespace/pubsub/set/UnsubscribeSetTest.java

@@ -1,5 +1,6 @@
 package src.test.java.org.buddycloud.channelserver.packetprocessor.iq.namespace.pubsub.set;
 
+import java.util.ArrayList;
 import java.util.Date;
 import java.util.concurrent.BlockingQueue;
 import java.util.concurrent.LinkedBlockingQueue;
@@ -66,12 +67,17 @@ public void setUp() throws Exception {
 	@Test
 	public void testCanNotUnsubscribeAsOnlyNodeOwner() throws Exception {
 
+		ArrayList<JID> owners = new ArrayList<JID>();
+		owners.add(jid);
+
 		Mockito.when(
 				channelManager.getUserSubscription(Mockito.anyString(),
 						Mockito.any(JID.class))).thenReturn(subscription);
 		Mockito.when(
 				channelManager.getUserAffiliation(Mockito.anyString(),
 						Mockito.any(JID.class))).thenReturn(affiliation);
+		Mockito.when(channelManager.getNodeOwners(Mockito.anyString()))
+				.thenReturn(owners);
 
 		unsubscribe.process(element, jid, request, null);
 
@@ -83,9 +89,10 @@ public void testCanNotUnsubscribeAsOnlyNodeOwner() throws Exception {
 
 		PacketError error = response.getError();
 		Assert.assertNotNull(error);
-		
+
 		Assert.assertEquals(PacketError.Type.cancel, error.getType());
-		Assert.assertEquals(PacketError.Condition.not_allowed, error.getCondition());
+		Assert.assertEquals(PacketError.Condition.not_allowed,
+				error.getCondition());
 
 	}