(trying to) fix some problems with the SIMPLE test cases on cruise control

src/net/java/sip/communicator/impl/protocol/sip/OperationSetPresenceSipImpl.java

@@ -264,7 +264,9 @@ public OperationSetPresenceSipImpl(ProtocolProviderServiceSipImpl provider,
         // if we force the p2p mode, we start by not using a distant PA
         this.useDistantPA = !forceP2PMode;
         
-        this.subscriptionDuration = subscriptionExpiration;
+        if (subscriptionDuration > 0) {
+            this.subscriptionDuration = subscriptionExpiration;
+        }
         this.presenceEnabled = isPresenceEnabled;
     }
 
@@ -738,6 +740,7 @@ public void fireProviderMsgStatusChangeEvent(String oldValue)
 
             listener.providerStatusMessageChanged(evt);
         }
+        logger.debug("status dispatching done.");
     }
 
     /**

test/net/java/sip/communicator/slick/protocol/sip/TestOperationSetPresence.java

@@ -398,6 +398,15 @@ public void postTestSubscribe()
 
         subEvtCollector.collectedEvents.clear();
         
+        // wait to be sure that every responses for the subscribe have been
+        // received
+        Object lock = new Object();
+        synchronized(lock)
+        {
+            logger.info("Will wait all subscription events to be received");
+            lock.wait(3000);
+        }
+        
         // make the user agent tester change its states and make sure we are
         // notified
         logger.debug("Testing presence notifications.");
@@ -482,10 +491,9 @@ public void postTestSubscribe()
             // happens if the user is already subscribed
         }
         
-        Object lock = new Object();
         synchronized(lock)
         {
-            logger.info("Will wait all subscriptioin events to be received by" +
+            logger.info("Will wait all subscription events to be received by" +
                     " lib");
             lock.wait(3000);
             logger.info("Stopped waiting");