Author: stillalex
Date: Thu May 3 13:22:23 2018
New Revision: 1830821
URL: http://svn.apache.org/viewvc?rev=1830821&view=rev
Log:
OAK-7462 Benchmark for SynchronizationMBean#syncAllUsers
- fixed assertions for the dynamic membership case
Modified:
jackrabbit/oak/trunk/oak-benchmarks/src/main/java/org/apache/jackrabbit/oak/benchmark/authentication/external/SyncAllUsersTest.java
Modified:
jackrabbit/oak/trunk/oak-benchmarks/src/main/java/org/apache/jackrabbit/oak/benchmark/authentication/external/SyncAllUsersTest.java
URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-benchmarks/src/main/java/org/apache/jackrabbit/oak/benchmark/authentication/external/SyncAllUsersTest.java?rev=1830821&r1=1830820&r2=1830821&view=diff
==============================================================================
---
jackrabbit/oak/trunk/oak-benchmarks/src/main/java/org/apache/jackrabbit/oak/benchmark/authentication/external/SyncAllUsersTest.java
(original)
+++
jackrabbit/oak/trunk/oak-benchmarks/src/main/java/org/apache/jackrabbit/oak/benchmark/authentication/external/SyncAllUsersTest.java
Thu May 3 13:22:23 2018
@@ -33,15 +33,17 @@ import org.apache.jackrabbit.oak.spi.sec
*/
public class SyncAllUsersTest extends AbstractExternalTest {
- private final int numberOfUsers;
- private final int numberOfGroups;
+ private final int expectedUpdates;
private SynchronizationMBean bean;
public SyncAllUsersTest(int numberOfUsers, int numberOfGroups, long
expTime, boolean dynamicMembership,
@Nonnull List<String> autoMembership) {
super(numberOfUsers, numberOfGroups, expTime, dynamicMembership,
autoMembership);
- this.numberOfUsers = numberOfUsers;
- this.numberOfGroups = numberOfGroups;
+ if (dynamicMembership) {
+ expectedUpdates = numberOfUsers;
+ } else {
+ expectedUpdates = numberOfUsers + numberOfGroups;
+ }
}
@Override
@@ -60,6 +62,6 @@ public class SyncAllUsersTest extends Ab
@Override
protected void runTest() throws Exception {
String[] ops = bean.syncAllUsers(true);
- assertEquals(numberOfUsers + numberOfGroups, ops.length);
+ assertEquals(expectedUpdates, ops.length);
}
}
\ No newline at end of file
