Author: bdelacretaz
Date: Thu Jan 28 11:57:14 2016
New Revision: 1727315

URL: http://svn.apache.org/viewvc?rev=1727315&view=rev
Log:
SLING-5462 - mention this issue

Modified:
    
sling/trunk/bundles/jcr/base/src/test/java/org/apache/sling/jcr/base/RepositoryInitializersTest.java

Modified: 
sling/trunk/bundles/jcr/base/src/test/java/org/apache/sling/jcr/base/RepositoryInitializersTest.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/jcr/base/src/test/java/org/apache/sling/jcr/base/RepositoryInitializersTest.java?rev=1727315&r1=1727314&r2=1727315&view=diff
==============================================================================
--- 
sling/trunk/bundles/jcr/base/src/test/java/org/apache/sling/jcr/base/RepositoryInitializersTest.java
 (original)
+++ 
sling/trunk/bundles/jcr/base/src/test/java/org/apache/sling/jcr/base/RepositoryInitializersTest.java
 Thu Jan 28 11:57:14 2016
@@ -116,11 +116,8 @@ public class RepositoryInitializersTest
         
         assertStart(true);
         
-        // TODO this should really be 1,2,3 but it looks like
-        // the Sling OSGi mocks sort in the wrong order w.r.t the
-        // service ranking. As demonstrated by the integration test
-        // under launchpad/test-services which comes out in the correct
-        // order.
+        // TODO this should really be 1,2,3 but the Sling OSGi mocks sort 
+               // in the wrong order w.r.t service ranking, see SLING-5462
         assertTestInitializerProperty("3,2,1,");
         assertAdditionalRepositoryServices(1);
     }
@@ -134,7 +131,7 @@ public class RepositoryInitializersTest
         assertStart(true);
         
         // TODO see comment in inOrderInitializers, this should really
-        // be 3,2,1
+        // be 3,2,1, caused by SLING-5462
         assertTestInitializerProperty("1,2,3,");
         assertAdditionalRepositoryServices(1);
     }


Reply via email to