blob: a66486915d33a443c028d732b43c4794469cf4ad [file] [log] [blame]
// Copyright (C) 2018 The Android Open Source Project
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package com.google.gerrit.acceptance.api.group;
import static com.google.common.truth.Truth.assertWithMessage;
import static com.google.gerrit.truth.ListSubject.assertThat;
import static com.google.gerrit.truth.OptionalSubject.assertThat;
import com.google.common.collect.ImmutableSet;
import com.google.gerrit.common.errors.NoSuchGroupException;
import com.google.gerrit.extensions.api.GerritApi;
import com.google.gerrit.extensions.common.GroupInfo;
import com.google.gerrit.extensions.restapi.RestApiException;
import com.google.gerrit.reviewdb.client.AccountGroup;
import com.google.gerrit.server.ServerInitiated;
import com.google.gerrit.server.account.GroupCache;
import com.google.gerrit.server.group.InternalGroup;
import com.google.gerrit.server.group.db.GroupsUpdate;
import com.google.gerrit.server.group.db.InternalGroupUpdate;
import com.google.gerrit.server.group.testing.InternalGroupSubject;
import com.google.gerrit.server.index.group.GroupIndexer;
import com.google.gerrit.server.query.group.InternalGroupQuery;
import com.google.gerrit.testing.InMemoryTestEnvironment;
import com.google.gerrit.truth.ListSubject;
import com.google.gerrit.truth.OptionalSubject;
import com.google.gwtorm.server.OrmException;
import com.google.inject.Inject;
import com.google.inject.Provider;
import java.io.IOException;
import java.util.List;
import java.util.Optional;
import org.eclipse.jgit.errors.ConfigInvalidException;
import org.junit.Rule;
import org.junit.Test;
public class GroupIndexerIT {
@Rule public InMemoryTestEnvironment testEnvironment = new InMemoryTestEnvironment();
@Inject private GroupIndexer groupIndexer;
@Inject private GerritApi gApi;
@Inject private GroupCache groupCache;
@Inject @ServerInitiated private GroupsUpdate groupsUpdate;
@Inject private Provider<InternalGroupQuery> groupQueryProvider;
@Test
public void indexingUpdatesTheIndex() throws Exception {
AccountGroup.UUID groupUuid = createGroup("users");
AccountGroup.UUID subgroupUuid = new AccountGroup.UUID("contributors");
updateGroupWithoutCacheOrIndex(
groupUuid,
newGroupUpdate()
.setSubgroupModification(subgroups -> ImmutableSet.of(subgroupUuid))
.build());
groupIndexer.index(groupUuid);
List<InternalGroup> parentGroups = groupQueryProvider.get().bySubgroup(subgroupUuid);
assertThatGroups(parentGroups).onlyElement().groupUuid().isEqualTo(groupUuid);
}
@Test
public void indexCannotBeCorruptedByStaleCache() throws Exception {
AccountGroup.UUID groupUuid = createGroup("verifiers");
loadGroupToCache(groupUuid);
AccountGroup.UUID subgroupUuid = new AccountGroup.UUID("contributors");
updateGroupWithoutCacheOrIndex(
groupUuid,
newGroupUpdate()
.setSubgroupModification(subgroups -> ImmutableSet.of(subgroupUuid))
.build());
groupIndexer.index(groupUuid);
List<InternalGroup> parentGroups = groupQueryProvider.get().bySubgroup(subgroupUuid);
assertThatGroups(parentGroups).onlyElement().groupUuid().isEqualTo(groupUuid);
}
@Test
public void indexingUpdatesStaleUuidCache() throws Exception {
AccountGroup.UUID groupUuid = createGroup("verifiers");
loadGroupToCache(groupUuid);
updateGroupWithoutCacheOrIndex(groupUuid, newGroupUpdate().setDescription("Modified").build());
groupIndexer.index(groupUuid);
Optional<InternalGroup> updatedGroup = groupCache.get(groupUuid);
assertThatGroup(updatedGroup).value().description().isEqualTo("Modified");
}
@Test
public void reindexingStaleGroupUpdatesTheIndex() throws Exception {
AccountGroup.UUID groupUuid = createGroup("users");
AccountGroup.UUID subgroupUuid = new AccountGroup.UUID("contributors");
updateGroupWithoutCacheOrIndex(
groupUuid,
newGroupUpdate()
.setSubgroupModification(subgroups -> ImmutableSet.of(subgroupUuid))
.build());
groupIndexer.reindexIfStale(groupUuid);
List<InternalGroup> parentGroups = groupQueryProvider.get().bySubgroup(subgroupUuid);
assertThatGroups(parentGroups).onlyElement().groupUuid().isEqualTo(groupUuid);
}
@Test
public void notStaleGroupIsNotReindexed() throws Exception {
AccountGroup.UUID groupUuid = createGroup("verifiers");
updateGroupWithoutCacheOrIndex(groupUuid, newGroupUpdate().setDescription("Modified").build());
groupIndexer.index(groupUuid);
boolean reindexed = groupIndexer.reindexIfStale(groupUuid);
assertWithMessage("Group should not have been reindexed").that(reindexed).isFalse();
}
@Test
public void indexStalenessIsNotDerivedFromCacheStaleness() throws Exception {
AccountGroup.UUID groupUuid = createGroup("verifiers");
updateGroupWithoutCacheOrIndex(groupUuid, newGroupUpdate().setDescription("Modified").build());
reloadGroupToCache(groupUuid);
boolean reindexed = groupIndexer.reindexIfStale(groupUuid);
assertWithMessage("Group should have been reindexed").that(reindexed).isTrue();
}
private AccountGroup.UUID createGroup(String name) throws RestApiException {
GroupInfo group = gApi.groups().create(name).get();
return new AccountGroup.UUID(group.id);
}
private void reloadGroupToCache(AccountGroup.UUID groupUuid) {
groupCache.evict(groupUuid);
loadGroupToCache(groupUuid);
}
private void loadGroupToCache(AccountGroup.UUID groupUuid) {
groupCache.get(groupUuid);
}
private static InternalGroupUpdate.Builder newGroupUpdate() {
return InternalGroupUpdate.builder();
}
private void updateGroupWithoutCacheOrIndex(
AccountGroup.UUID groupUuid, InternalGroupUpdate groupUpdate)
throws OrmException, NoSuchGroupException, IOException, ConfigInvalidException {
groupsUpdate.updateGroupInNoteDb(groupUuid, groupUpdate);
}
private static OptionalSubject<InternalGroupSubject, InternalGroup> assertThatGroup(
Optional<InternalGroup> updatedGroup) {
return assertThat(updatedGroup, InternalGroupSubject::assertThat);
}
private static ListSubject<InternalGroupSubject, InternalGroup> assertThatGroups(
List<InternalGroup> parentGroups) {
return assertThat(parentGroups, InternalGroupSubject::assertThat);
}
}