You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by bd...@apache.org on 2014/10/15 17:55:22 UTC

svn commit: r1632101 - in /sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices: events/EventsCounterImpl.java serversidetests/ResourceResolverTest.java

Author: bdelacretaz
Date: Wed Oct 15 15:55:21 2014
New Revision: 1632101

URL: http://svn.apache.org/r1632101
Log:
SLING-4061 - more logging to help troubleshoot

Modified:
    sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/events/EventsCounterImpl.java
    sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/serversidetests/ResourceResolverTest.java

Modified: sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/events/EventsCounterImpl.java
URL: http://svn.apache.org/viewvc/sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/events/EventsCounterImpl.java?rev=1632101&r1=1632100&r2=1632101&view=diff
==============================================================================
--- sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/events/EventsCounterImpl.java (original)
+++ sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/events/EventsCounterImpl.java Wed Oct 15 15:55:21 2014
@@ -73,7 +73,11 @@ public class EventsCounterImpl extends S
     
     public int getEventsCount(String topic) {
         final AtomicInteger counter = counters.get(topic);
-        return counter == null ? 0 : counter.get();
+        if(counter == null) {
+            log.debug("getEventsCount({}) returns 0, counter not found", topic);
+            return 0;
+        }
+        return counter.get();
     }
 
     @Override

Modified: sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/serversidetests/ResourceResolverTest.java
URL: http://svn.apache.org/viewvc/sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/serversidetests/ResourceResolverTest.java?rev=1632101&r1=1632100&r2=1632101&view=diff
==============================================================================
--- sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/serversidetests/ResourceResolverTest.java (original)
+++ sling/trunk/launchpad/test-services/src/main/java/org/apache/sling/launchpad/testservices/serversidetests/ResourceResolverTest.java Wed Oct 15 15:55:21 2014
@@ -96,6 +96,7 @@ public class ResourceResolverTest {
         }
         
         final int oldEventsCount = eventsCounter.getEventsCount(MAPPING_EVENT_TOPIC);
+        logger.debug("Waiting for event counter {} to change from current value {}", MAPPING_EVENT_TOPIC, oldEventsCount);
         session.save();
         final long timeout = System.currentTimeMillis() + updateTimeout;
         while(System.currentTimeMillis() < timeout) {
@@ -111,6 +112,7 @@ public class ResourceResolverTest {
             }
         }
         eventTimeoutTopic = MAPPING_EVENT_TOPIC;
+        logger.error("Timeout waiting for event counter {} to change from current value {}", MAPPING_EVENT_TOPIC, oldEventsCount);
         fail("Timeout waiting for " + MAPPING_EVENT_TOPIC + " event, after " + updateTimeout + " msec");
     }