opendev/gerrit
Code Issues Proposed changes

AbstractDaemonTest: Add injected AllUsersName

Browse Source 
There are several tests that need AllUsersName, and they all get it
injected.

Move it up into AbstractDaemonTest as a protected member that can be
used by all tests.

Change-Id: Ic6341522133b90d441fa9324b2f0c455c765bc31
This commit is contained in:
David Pursehouse
2017-11-10 08:55:33 +09:00
parent 0a420520fe
commit 696e8bce96
12 changed files with 2 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
java/com/google/gerrit/acceptance/AbstractDaemonTest.java
View File

@@ -87,6 +87,7 @@ import com.google.gerrit.server.change.FileContentUtil;
import com.google.gerrit.server.change.RevisionResource;
import com.google.gerrit.server.change.Revisions;
import com.google.gerrit.server.config.AllProjectsName;
import com.google.gerrit.server.config.AllUsersName;
import com.google.gerrit.server.config.CanonicalWebUrl;
import com.google.gerrit.server.config.GerritServerConfig;
import com.google.gerrit.server.config.PluginConfigFactory;
@@ -209,6 +210,7 @@ public abstract class AbstractDaemonTest {
@Inject protected AccountCreator accountCreator;
@Inject protected Accounts accounts;
@Inject protected AllProjectsName allProjects;
@Inject protected AllUsersName allUsers;
@Inject protected BatchUpdate.Factory batchUpdateFactory;
@Inject protected ChangeData.Factory changeDataFactory;
@Inject protected ChangeFinder changeFinder;