opendev/gerrit
Code Issues Proposed changes

TestGroupBackendTest: Remove TestGroupBackend from DynamicSet after test

Browse Source 
Change-Id: I4c52fbbe769d2878fd3e9ad6e84895b1a0dab582
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin 2018-04-19 22:59:12 +02:00 committed by David Pursehouse
parent a649784abe
commit 9e63fd4ecc
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
javatests/com/google/gerrit/acceptance/TestGroupBackendTest.java
View File

@ -17,6 +17,7 @@ package com.google.gerrit.acceptance;
import static com.google.common.truth.Truth.assertThat;
import com.google.gerrit.extensions.registration.DynamicSet;
import com.google.gerrit.extensions.registration.RegistrationHandle;
import com.google.gerrit.reviewdb.client.AccountGroup;
import com.google.gerrit.server.account.GroupBackend;
import com.google.gerrit.server.account.UniversalGroupBackend;
@ -38,8 +39,12 @@ public class TestGroupBackendTest extends AbstractDaemonTest {
@Test
public void universalGroupBackendHandlesTestGroup() throws Exception {
groupBackends.add(testGroupBackend);
assertThat(universalGroupBackend.handles(testUUID)).isTrue();
RegistrationHandle registrationHandle = groupBackends.add(testGroupBackend);
try {
assertThat(universalGroupBackend.handles(testUUID)).isTrue();
} finally {
registrationHandle.remove();
}
}
@Test