svn commit: r1742125 - in /jackrabbit/oak/trunk/oak-auth-external/src: main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/ test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/

[angela]

Author: angela
Date: Tue May  3 15:03:37 2016
New Revision: 1742125

URL: http://svn.apache.org/viewvc?rev=1742125&view=rev
Log:
minor improvement:
- simplify usage of valuefactory in DefaultSyncHandlerTest
- fix typo in SynchronizationMBean javadoc

Modified:
    
jackrabbit/oak/trunk/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SynchronizationMBean.java
    
jackrabbit/oak/trunk/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DefaultSyncHandlerTest.java

Modified: 
jackrabbit/oak/trunk/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SynchronizationMBean.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SynchronizationMBean.java?rev=1742125&r1=1742124&r2=1742125&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SynchronizationMBean.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SynchronizationMBean.java
 Tue May  3 15:03:37 2016
@@ -150,7 +150,7 @@ public interface SynchronizationMBean {
 
     /**
      * Purges all orphaned users. this is similar to invoke {@link 
#syncUsers(String[], boolean)} with the list of
-     * orphaned users. Note tha this can eb an expensive operation since all 
potential users need to be examined.
+     * orphaned users. Note tha this can be an expensive operation since all 
potential users need to be examined.
      *
      * @return result messages.
      */

Modified: 
jackrabbit/oak/trunk/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DefaultSyncHandlerTest.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DefaultSyncHandlerTest.java?rev=1742125&r1=1742124&r2=1742125&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DefaultSyncHandlerTest.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DefaultSyncHandlerTest.java
 Tue May  3 15:03:37 2016
@@ -26,8 +26,6 @@ import javax.jcr.Value;
 import com.google.common.collect.Sets;
 import org.apache.jackrabbit.api.security.user.Authorizable;
 import org.apache.jackrabbit.api.security.user.UserManager;
-import org.apache.jackrabbit.oak.namepath.NamePathMapper;
-import org.apache.jackrabbit.oak.plugins.value.ValueFactoryImpl;
 import 
org.apache.jackrabbit.oak.spi.security.authentication.external.ExternalIdentity;
 import 
org.apache.jackrabbit.oak.spi.security.authentication.external.ExternalIdentityRef;
 import 
org.apache.jackrabbit.oak.spi.security.authentication.external.ExternalLoginModuleTestBase;
@@ -100,7 +98,7 @@ public class DefaultSyncHandlerTest exte
 
     @Test
     public void testCreateContext() throws Exception {
-        SyncContext ctx = syncHandler.createContext(idp, userManager, new 
ValueFactoryImpl(root, NamePathMapper.DEFAULT));
+        SyncContext ctx = syncHandler.createContext(idp, userManager, 
getValueFactory());
         assertTrue(ctx instanceof DefaultSyncContext);
     }