From d308196cd5de837eeae80a3d52b5c48f1cecceee Mon Sep 17 00:00:00 2001 From: Edwin Kempin Date: Thu, 3 Aug 2017 11:40:09 +0200 Subject: [PATCH] Test that account is removed from cache/index on deletion of user branch Change-Id: I28aec0d314c6a2b188e9c1b837197d6ae50bb58e Signed-off-by: Edwin Kempin --- .../com/google/gerrit/acceptance/api/accounts/AccountIT.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/accounts/AccountIT.java b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/accounts/AccountIT.java index 7477a3d28d..06fe021b69 100644 --- a/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/accounts/AccountIT.java +++ b/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/api/accounts/AccountIT.java @@ -44,6 +44,7 @@ import com.google.common.io.BaseEncoding; import com.google.common.util.concurrent.AtomicLongMap; import com.google.gerrit.acceptance.AbstractDaemonTest; import com.google.gerrit.acceptance.AccountCreator; +import com.google.gerrit.acceptance.GerritConfig; import com.google.gerrit.acceptance.PushOneCommit; import com.google.gerrit.acceptance.Sandboxed; import com.google.gerrit.acceptance.TestAccount; @@ -1019,6 +1020,7 @@ public class AccountIT extends AbstractDaemonTest { @Test @Sandboxed + @GerritConfig(name = "user.readAccountsFromGit", value = "true") public void deleteUserBranchWithAccessDatabaseCapability() throws Exception { allowGlobalCapabilities(REGISTERED_USERS, GlobalCapability.ACCESS_DATABASE); grant( @@ -1038,7 +1040,8 @@ public class AccountIT extends AbstractDaemonTest { assertThat(repo.exactRef(userRef)).isNull(); } - // TODO(ekempin): assert that account was deleted from cache and index + assertThat(accountCache.getOrNull(admin.id)).isNull(); + assertThat(accountQuery.byDefault(admin.id.toString())).isEmpty(); } @Test