[sling-org-apache-sling-discovery-base] 17/20: SLING-5251 : add config parameter useSyncTokenService that enables usage of a SyncTokenService if available. This increases synchronization QoS when for topology changes

[ro...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

rombert pushed a commit to annotated tag org.apache.sling.discovery.base-1.1.0
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-discovery-base.git

commit e17113161f82b861e8dd0154e65d30b8e02a4023
Author: Stefan Egli <st...@apache.org>
AuthorDate: Tue Nov 10 08:36:21 2015 +0000

    SLING-5251 : add config parameter useSyncTokenService that enables usage of a SyncTokenService if available. This increases synchronization QoS when for topology changes
    
    git-svn-id: https://svn.apache.org/repos/asf/sling/trunk/bundles/extensions/discovery/base@1713589 13f79535-47bb-0310-9956-ffa450edef68
---
 .../sling/discovery/base/its/AbstractTopologyEventTest.java       | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/test/java/org/apache/sling/discovery/base/its/AbstractTopologyEventTest.java b/src/test/java/org/apache/sling/discovery/base/its/AbstractTopologyEventTest.java
index 0280c15..82094b8 100644
--- a/src/test/java/org/apache/sling/discovery/base/its/AbstractTopologyEventTest.java
+++ b/src/test/java/org/apache/sling/discovery/base/its/AbstractTopologyEventTest.java
@@ -33,6 +33,7 @@ import org.apache.sling.discovery.base.its.setup.VirtualInstanceBuilder;
 import org.apache.sling.discovery.base.its.setup.mock.AssertingTopologyEventListener;
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -152,7 +153,12 @@ public abstract class AbstractTopologyEventTest {
         assertEquals(0, l1.getUnexpectedCount());
         assertEquals(2, l1.getEvents().size());
         assertEquals(0, l2.getUnexpectedCount());
-        assertEquals(1, l2.getEvents().size());
+        // with the switch to use the SyncTokenService in discovery.impl tests
+        // by default, the following check is no longer possible:
+//        assertEquals(1, l2.getEvents().size());
+        // (this is due to the fact that synching requires some more time
+        // and we're a bit early at this stage - the below same check
+        // is the only one that we can do here really - and that one must work)
         assertEquals(0, l1Two.getUnexpectedCount());
         assertEquals(2, l1Two.getEvents().size());
 

-- 
To stop receiving notification emails like this one, please contact
"commits@sling.apache.org" <co...@sling.apache.org>.