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

baedke pushed a commit to branch issues/oak-10379
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


The following commit(s) were added to refs/heads/issues/oak-10379 by this push:
     new ba73848af7 OAK-10379: DynamicGroupsTest#testCrossIDPMembership() fails 
occasionally
ba73848af7 is described below

commit ba73848af7638b2de3139a44754b48445fe653ff
Author: Manfred Baedke <gre55...@adobe.com>
AuthorDate: Tue Aug 8 16:31:31 2023 +0200

    OAK-10379: DynamicGroupsTest#testCrossIDPMembership() fails occasionally
    
    Fixed timing issue.
---
 .../spi/security/authentication/external/impl/DynamicGroupsTest.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DynamicGroupsTest.java
 
b/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DynamicGroupsTest.java
index dd5937b566..3c80d3712a 100644
--- 
a/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DynamicGroupsTest.java
+++ 
b/oak-auth-external/src/test/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/DynamicGroupsTest.java
@@ -307,7 +307,9 @@ public class DynamicGroupsTest extends 
DynamicSyncContextTest {
 
         Authorizable a = um.getAuthorizable(PREVIOUS_SYNCED_ID);
         assertTrue(getIds(a.memberOf()).contains(local.getID()));
-        
+
+        // create new context to ensure that the internal timestamp is ahead 
of the last sync time (OAK-10379)
+        syncContext = new DynamicSyncContext(syncConfig, idp, userManager, 
valueFactory);
         // sync again to establish dynamic membership
         syncContext.setForceUserSync(true);
         syncContext.setForceGroupSync(true);

Reply via email to