Merge "Make InvalidTransitionException a subclass of Exception"
diff --git a/.gitignore b/.gitignore
index dc4fc0a..16ed26a 100644
--- a/.gitignore
+++ b/.gitignore
@@ -3,3 +3,4 @@
.project
/.settings
/bazel-*
+.DS_Store
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/InitIts.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/InitIts.java
index ac0136c..1cd2b74 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/InitIts.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/InitIts.java
@@ -21,23 +21,24 @@
import com.google.gerrit.pgm.init.api.ConsoleUI;
import com.google.gerrit.pgm.init.api.InitStep;
import com.google.gerrit.pgm.init.api.Section;
-
+import java.io.IOException;
+import java.util.EnumSet;
import org.eclipse.jgit.errors.ConfigInvalidException;
import org.eclipse.jgit.lib.Config;
-import java.io.IOException;
-import java.util.EnumSet;
-
public class InitIts implements InitStep {
- public static String COMMENT_LINK_SECTION = "commentlink";
+ protected static final String COMMENT_LINK_SECTION = "commentlink";
public static enum TrueFalseEnum {
- TRUE, FALSE;
+ TRUE,
+ FALSE
}
public static enum ItsIntegration {
- ENABLED, DISABLED, ENFORCED;
+ ENABLED,
+ DISABLED,
+ ENFORCED
}
private final String pluginName;
@@ -46,7 +47,10 @@
private final AllProjectsConfig allProjectsConfig;
private final AllProjectsNameOnInitProvider allProjects;
- public InitIts(String pluginName, String itsDisplayName, ConsoleUI ui,
+ public InitIts(
+ String pluginName,
+ String itsDisplayName,
+ ConsoleUI ui,
AllProjectsConfig allProjectsConfig,
AllProjectsNameOnInitProvider allProjects) {
this.pluginName = pluginName;
@@ -57,8 +61,7 @@
}
@Override
- public void run() throws IOException, ConfigInvalidException {
- }
+ public void run() throws IOException, ConfigInvalidException {}
@Override
public void postRun() throws IOException, ConfigInvalidException {
@@ -76,7 +79,8 @@
itsintegration = ItsIntegration.DISABLED;
}
itsintegration =
- ui.readEnum(itsintegration,
+ ui.readEnum(
+ itsintegration,
EnumSet.allOf(ItsIntegration.class),
"Issue tracker integration for all projects?");
switch (itsintegration) {
@@ -92,8 +96,8 @@
cfg.unset("plugin", pluginName, "enabled");
break;
default:
- throw new IOException("Unsupported value for issue track integration: "
- + itsintegration.name());
+ throw new IOException(
+ "Unsupported value for issue track integration: " + itsintegration.name());
}
allProjectsConfig.save(pluginName, "Initialize " + itsDisplayName + " Integration");
}
@@ -101,8 +105,9 @@
private void configureBranches(Config cfg) {
String[] branches = cfg.getStringList("plugin", pluginName, "branch");
if (branches.length > 1) {
- ui.message("The issue tracker integration is configured for multiple branches."
- + " Please adapt the configuration in the 'project.config' file of the '%s' project.\n",
+ ui.message(
+ "The issue tracker integration is configured for multiple branches."
+ + " Please adapt the configuration in the 'project.config' file of the '%s' project.\n",
allProjects.get());
return;
}
@@ -114,14 +119,18 @@
boolean validRef;
do {
- String v = ui.readString(branch, "Branches for which the issue tracker integration"
- + " should be enabled (ref, ref pattern or regular expression)");
+ String v =
+ ui.readString(
+ branch,
+ "Branches for which the issue tracker integration"
+ + " should be enabled (ref, ref pattern or regular expression)");
validRef = RefConfigSection.isValid(v);
if (validRef) {
branch = v;
} else {
ui.message(
- "'%s' is not valid. Please specify a valid ref, ref pattern or regular expression\n", v);
+ "'%s' is not valid. Please specify a valid ref, ref pattern or regular expression\n",
+ v);
}
} while (!validRef);
@@ -133,7 +142,7 @@
}
public boolean enterSSLVerify(Section section) {
- return TrueFalseEnum.TRUE == section.select("Verify SSL Certificates",
- "sslVerify", TrueFalseEnum.TRUE);
+ return TrueFalseEnum.TRUE
+ == section.select("Verify SSL Certificates", "sslVerify", TrueFalseEnum.TRUE);
}
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfig.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfig.java
index 665aef1..99fe80c 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfig.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfig.java
@@ -14,10 +14,8 @@
package com.googlesource.gerrit.plugins.its.base.its;
-import com.google.common.base.Function;
-import com.google.common.base.Optional;
-import com.google.common.base.Predicate;
-import com.google.common.collect.FluentIterable;
+import static java.util.stream.Collectors.toList;
+
import com.google.gerrit.common.data.RefConfigSection;
import com.google.gerrit.extensions.annotations.PluginName;
import com.google.gerrit.extensions.api.projects.CommentLinkInfo;
@@ -38,18 +36,15 @@
import com.google.gerrit.server.project.ProjectState;
import com.google.gerrit.server.project.RefPatternMatcher;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.validation.ItsAssociationPolicy;
-
+import java.util.Collections;
+import java.util.List;
+import java.util.Optional;
+import java.util.regex.Pattern;
import org.eclipse.jgit.lib.Config;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.util.Collections;
-import java.util.List;
-import java.util.regex.Pattern;
-
-
public class ItsConfig {
private static final String PLUGIN = "plugin";
@@ -61,20 +56,18 @@
private final Config gerritConfig;
private static final ThreadLocal<Project.NameKey> currentProjectName =
- new ThreadLocal<Project.NameKey>() {
- @Override
- protected Project.NameKey initialValue() {
- return null;
- }
- };
+ ThreadLocal.withInitial(() -> null);
public static void setCurrentProjectName(Project.NameKey projectName) {
currentProjectName.set(projectName);
}
@Inject
- public ItsConfig(@PluginName String pluginName, ProjectCache projectCache,
- PluginConfigFactory pluginCfgFactory, @GerritServerConfig Config gerritConfig) {
+ public ItsConfig(
+ @PluginName String pluginName,
+ ProjectCache projectCache,
+ PluginConfigFactory pluginCfgFactory,
+ @GerritServerConfig Config gerritConfig) {
this.pluginName = pluginName;
this.projectCache = projectCache;
this.pluginCfgFactory = pluginCfgFactory;
@@ -103,7 +96,7 @@
RefUpdatedEvent e = (RefUpdatedEvent) event;
return isEnabled(e.getProjectNameKey(), e.getRefName());
} else {
- log.debug("Event " + event + " not recognised and ignored");
+ log.debug("Event {} not recognised and ignored", event);
return false;
}
}
@@ -111,25 +104,28 @@
public boolean isEnabled(Project.NameKey projectNK, String refName) {
ProjectState projectState = projectCache.get(projectNK);
if (projectState == null) {
- log.error("Failed to check if " + pluginName + " is enabled for project "
- + projectNK.get() + ": Project " + projectNK.get() + " not found");
+ log.error(
+ "Failed to check if {} is enabled for project {}: Project not found",
+ pluginName,
+ projectNK.get());
return false;
}
- if(isEnforcedByAnyParentProject(refName, projectState)) {
+ if (isEnforcedByAnyParentProject(refName, projectState)) {
return true;
}
- return !"false".equals(pluginCfgFactory.getFromProjectConfigWithInheritance(
- projectState, pluginName).getString("enabled", "false"))
+ return !"false"
+ .equals(
+ pluginCfgFactory
+ .getFromProjectConfigWithInheritance(projectState, pluginName)
+ .getString("enabled", "false"))
&& isEnabledForBranch(projectState, refName);
}
- private boolean isEnforcedByAnyParentProject(String refName,
- ProjectState projectState) {
+ private boolean isEnforcedByAnyParentProject(String refName, ProjectState projectState) {
for (ProjectState parentState : projectState.treeInOrder()) {
- PluginConfig parentCfg =
- pluginCfgFactory.getFromProjectConfig(parentState, pluginName);
+ PluginConfig parentCfg = pluginCfgFactory.getFromProjectConfig(parentState, pluginName);
if ("enforced".equals(parentCfg.getString("enabled", "false"))
&& isEnabledForBranch(parentState, refName)) {
return true;
@@ -140,8 +136,9 @@
private boolean isEnabledForBranch(ProjectState project, String refName) {
String[] refPatterns =
- pluginCfgFactory.getFromProjectConfigWithInheritance(project,
- pluginName).getStringList("branch");
+ pluginCfgFactory
+ .getFromProjectConfigWithInheritance(project, pluginName)
+ .getStringList("branch");
if (refPatterns.length == 0) {
return true;
}
@@ -177,48 +174,35 @@
/**
* Gets the regular expression used to identify issue ids.
- * <p>
- * The index of the group that holds the issue id is
- * {@link #getIssuePatternGroupIndex()}.
*
- * @return the regular expression, or {@code null}, if there is no pattern
- * to match issue ids.
+ * <p>The index of the group that holds the issue id is {@link #getIssuePatternGroupIndex()}.
+ *
+ * @return the regular expression, or {@code null}, if there is no pattern to match issue ids.
*/
public Pattern getIssuePattern() {
Optional<String> match =
- FluentIterable
- .from(getCommentLinkInfo(getCommentLinkName()))
- .filter(new Predicate<CommentLinkInfo>() {
- @Override
- public boolean apply(CommentLinkInfo input) {
- return input.match != null && !input.match.trim().isEmpty();
- }
- })
- .transform(new Function<CommentLinkInfo, String>() {
- @Override
- public String apply(CommentLinkInfo input) {
- return input.match;
- }
- })
- .last();
+ getCommentLinkInfo(getCommentLinkName())
+ .stream()
+ .filter(input -> input.match != null && !input.match.trim().isEmpty())
+ .map(input -> input.match)
+ .reduce((a, b) -> b);
- String defPattern = gerritConfig.getString("commentlink", getCommentLinkName(),
- "match");
+ String defPattern = gerritConfig.getString("commentlink", getCommentLinkName(), "match");
if (!match.isPresent() && defPattern == null) {
return null;
}
- return Pattern.compile(match.or(defPattern));
+ return Pattern.compile(match.orElse(defPattern));
}
/**
* Gets the index of the group in the issue pattern that holds the issue id.
- * <p>
- * The corresponding issue pattern is {@link #getIssuePattern()}
*
- * @return the group index for {@link #getIssuePattern()} that holds the
- * issue id. The group index is guaranteed to be a valid group index.
+ * <p>The corresponding issue pattern is {@link #getIssuePattern()}
+ *
+ * @return the group index for {@link #getIssuePattern()} that holds the issue id. The group index
+ * is guaranteed to be a valid group index.
*/
public int getIssuePatternGroupIndex() {
Pattern pattern = getIssuePattern();
@@ -237,36 +221,33 @@
*/
public ItsAssociationPolicy getItsAssociationPolicy() {
ItsAssociationPolicy legacyItsAssociationPolicy =
- gerritConfig.getEnum("commentlink", getCommentLinkName(),
- "association", ItsAssociationPolicy.OPTIONAL);
+ gerritConfig.getEnum(
+ "commentlink", getCommentLinkName(), "association", ItsAssociationPolicy.OPTIONAL);
return getPluginConfigEnum("association", legacyItsAssociationPolicy);
}
private String getPluginConfigString(String key) {
- return getCurrentPluginConfig().getString(key,
- gerritConfig.getString(PLUGIN, pluginName, key));
+ return getCurrentPluginConfig().getString(key, gerritConfig.getString(PLUGIN, pluginName, key));
}
private int getPluginConfigInt(String key, int defaultValue) {
- return getCurrentPluginConfig().getInt(key,
- gerritConfig.getInt(PLUGIN, pluginName, key, defaultValue));
+ return getCurrentPluginConfig()
+ .getInt(key, gerritConfig.getInt(PLUGIN, pluginName, key, defaultValue));
}
private <T extends Enum<?>> T getPluginConfigEnum(String key, T defaultValue) {
- return getCurrentPluginConfig().getEnum(key,
- gerritConfig.getEnum(PLUGIN, pluginName, key, defaultValue));
+ return getCurrentPluginConfig()
+ .getEnum(key, gerritConfig.getEnum(PLUGIN, pluginName, key, defaultValue));
}
private PluginConfig getCurrentPluginConfig() {
NameKey projectName = currentProjectName.get();
if (projectName != null) {
try {
- return pluginCfgFactory.getFromProjectConfigWithInheritance(
- projectName, pluginName);
+ return pluginCfgFactory.getFromProjectConfigWithInheritance(projectName, pluginName);
} catch (NoSuchProjectException e) {
- log.error("Cannot access " + projectName + " configuration for plugin "
- + pluginName, e);
+ log.error("Cannot access " + projectName + " configuration for plugin " + pluginName, e);
}
}
return new PluginConfig(pluginName, new Config());
@@ -275,15 +256,11 @@
private List<CommentLinkInfo> getCommentLinkInfo(final String commentlinkName) {
NameKey projectName = currentProjectName.get();
if (projectName != null) {
- List<CommentLinkInfo> commentlinks =
- projectCache.get(projectName).getCommentLinks();
- return FluentIterable.from(commentlinks)
- .filter(new Predicate<CommentLinkInfo>() {
- @Override
- public boolean apply(CommentLinkInfo input) {
- return input.name.equals(commentlinkName);
- }
- }).toList();
+ List<CommentLinkInfo> commentlinks = projectCache.get(projectName).getCommentLinks();
+ return commentlinks
+ .stream()
+ .filter(input -> input.name.equals(commentlinkName))
+ .collect(toList());
}
return Collections.emptyList();
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsFacade.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsFacade.java
index 53e3ced..9d7b3fa 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsFacade.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsFacade.java
@@ -17,9 +17,7 @@
import java.io.IOException;
import java.net.URL;
-/**
- * A simple facade to an issue tracking system (its)
- */
+/** A simple facade to an issue tracking system (its) */
public interface ItsFacade {
public enum Check {
@@ -27,20 +25,15 @@
ACCESS
}
- public String healthCheck(Check check)
- throws IOException;
+ public String healthCheck(Check check) throws IOException;
- public void addRelatedLink(String issueId, URL relatedUrl, String description)
- throws IOException;
+ public void addRelatedLink(String issueId, URL relatedUrl, String description) throws IOException;
- public void addComment(String issueId, String comment)
- throws IOException;
+ public void addComment(String issueId, String comment) throws IOException;
- public void performAction(String issueId, String actionName)
- throws IOException;
+ public void performAction(String issueId, String actionName) throws IOException;
- public boolean exists(final String issueId)
- throws IOException;
+ public boolean exists(final String issueId) throws IOException;
public String createLinkForWebui(String url, String text);
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java
index 17bcf6b..37f1851 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/ItsHookEnabledConfigEntry.java
@@ -18,17 +18,18 @@
import com.google.gerrit.server.config.PluginConfigFactory;
import com.google.gerrit.server.config.ProjectConfigEntry;
import com.google.gerrit.server.project.ProjectState;
-
import java.util.Arrays;
public class ItsHookEnabledConfigEntry extends ProjectConfigEntry {
private final String pluginName;
private final PluginConfigFactory pluginCfgFactory;
- public ItsHookEnabledConfigEntry(String pluginName,
- PluginConfigFactory pluginCfgFactory) {
- super("Enable " + pluginName + " integration", "false",
- Arrays.asList(new String[] {"false", "true", "enforced"}), true);
+ public ItsHookEnabledConfigEntry(String pluginName, PluginConfigFactory pluginCfgFactory) {
+ super(
+ "Enable " + pluginName + " integration",
+ "false",
+ Arrays.asList("false", "true", "enforced"),
+ true);
this.pluginName = pluginName;
this.pluginCfgFactory = pluginCfgFactory;
}
@@ -36,8 +37,7 @@
@Override
public boolean isEditable(ProjectState project) {
for (ProjectState parentState : project.parents()) {
- PluginConfig parentCfg =
- pluginCfgFactory.getFromProjectConfig(parentState, pluginName);
+ PluginConfig parentCfg = pluginCfgFactory.getFromProjectConfig(parentState, pluginName);
if ("enforced".equals(parentCfg.getString("enabled"))) {
return false;
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java
index 1d563e7..de4acd9 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/its/NoopItsFacade.java
@@ -14,16 +14,12 @@
package com.googlesource.gerrit.plugins.its.base.its;
+import java.io.IOException;
+import java.net.URL;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.io.IOException;
-import java.net.URL;
-
-/**
- * An ITS facade doing nothing, it's configured when no ITS are referenced in
- * config
- */
+/** An ITS facade doing nothing, it's configured when no ITS are referenced in config */
public class NoopItsFacade implements ItsFacade {
private Logger log = LoggerFactory.getLogger(NoopItsFacade.class);
@@ -39,8 +35,7 @@
public void addRelatedLink(String issueId, URL relatedUrl, String description)
throws IOException {
if (log.isDebugEnabled()) {
- log.debug("addRelatedLink({},{},{})", new Object[] {issueId, relatedUrl,
- description});
+ log.debug("addRelatedLink({},{},{})", issueId, relatedUrl, description);
}
}
@@ -53,8 +48,7 @@
}
@Override
- public void performAction(String issueId, String actionName)
- throws IOException {
+ public void performAction(String issueId, String actionName) throws IOException {
if (log.isDebugEnabled()) {
log.debug("performAction({},{})", issueId, actionName);
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/CommitMessageFetcher.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/CommitMessageFetcher.java
index 76a1a5a..905a2a3 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/CommitMessageFetcher.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/CommitMessageFetcher.java
@@ -3,7 +3,7 @@
import com.google.gerrit.reviewdb.client.Project.NameKey;
import com.google.gerrit.server.git.GitRepositoryManager;
import com.google.inject.Inject;
-
+import java.io.IOException;
import org.eclipse.jgit.lib.ObjectId;
import org.eclipse.jgit.lib.Repository;
import org.eclipse.jgit.revwalk.RevCommit;
@@ -11,11 +11,8 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.io.IOException;
-
public class CommitMessageFetcher {
- private static final Logger log = LoggerFactory.getLogger(
- CommitMessageFetcher.class);
+ private static final Logger log = LoggerFactory.getLogger(CommitMessageFetcher.class);
private final GitRepositoryManager repoManager;
@@ -38,8 +35,9 @@
try {
ret = fetch(projectName, commitId);
} catch (IOException e) {
- log.error("Could not fetch commit message for commit " + commitId +
- " of project " + projectName, e);
+ log.error(
+ "Could not fetch commit message for commit " + commitId + " of project " + projectName,
+ e);
}
return ret;
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractor.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractor.java
index 2809a83..5c344f1 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractor.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractor.java
@@ -11,22 +11,18 @@
import com.google.gerrit.reviewdb.client.PatchSet;
import com.google.inject.ImplementedBy;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsConfig;
-
-import org.apache.commons.lang.StringUtils;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
import java.util.EnumSet;
import java.util.Map;
import java.util.Set;
import java.util.regex.Matcher;
import java.util.regex.Pattern;
+import org.apache.commons.lang.StringUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
public class IssueExtractor {
- private static final Logger log = LoggerFactory.getLogger(
- IssueExtractor.class);
+ private static final Logger log = LoggerFactory.getLogger(IssueExtractor.class);
private final CommitMessageFetcher commitMessageFetcher;
private final PatchSetDb db;
@@ -83,7 +79,7 @@
Pattern pattern = itsConfig.getIssuePattern();
if (pattern == null) return new String[] {};
- log.debug("Matching '" + haystack + "' against " + pattern.pattern());
+ log.debug("Matching '{}' against {}", haystack, pattern.pattern());
Set<String> issues = Sets.newHashSet();
Matcher matcher = pattern.matcher(haystack);
@@ -100,22 +96,18 @@
}
/**
- * Helper funcion for {@link #getIssueIds(String, String)}.
- * <p>
- * Adds a text's issues for a given occurrence to the map returned by
- * {@link #getIssueIds(String, String)}.
+ * Helper function for {@link #getIssueIds(String, String)}.
+ *
+ * <p>Adds a text's issues for a given occurrence to the map returned by {@link
+ * #getIssueIds(String, String)}.
*
* @param text The text to extract issues from.
* @param occurrence The occurrence the issues get added at in {@code map}.
* @param map The map that the issues should get added to.
*/
- private void addIssuesOccurrence(String text, String occurrence, Map<String,Set<String>> map) {
+ private void addIssuesOccurrence(String text, String occurrence, Map<String, Set<String>> map) {
for (String issue : getIssueIds(text)) {
- Set<String> occurrences = map.get(issue);
- if (occurrences == null) {
- occurrences = Sets.newLinkedHashSet();
- map.put(issue, occurrences);
- }
+ Set<String> occurrences = map.computeIfAbsent(issue, k -> Sets.newLinkedHashSet());
occurrences.add(occurrence);
}
}
@@ -125,17 +117,14 @@
*
* @param projectName The project to fetch {@code commitId} from.
* @param commitId The commit id to fetch issues for.
- * @return A mapping, whose keys are issue ids and whose values is a set of
- * places where the issue occurs. Each issue occurs at least in
- * "somewhere". Issues from the first line get tagged with an occurrence
- * "subject". Issues in the last block get tagged with "footer". Issues
- * occurring between "subject" and "footer" get tagged with "body".
+ * @return A mapping, whose keys are issue ids and whose values is a set of places where the issue
+ * occurs. Each issue occurs at least in "somewhere". Issues from the first line get tagged
+ * with an occurrence "subject". Issues in the last block get tagged with "footer". Issues
+ * occurring between "subject" and "footer" get tagged with "body".
*/
- public Map<String,Set<String>> getIssueIds(String projectName,
- String commitId) {
- Map<String,Set<String>> ret = Maps.newHashMap();
- String commitMessage = commitMessageFetcher.fetchGuarded(projectName,
- commitId);
+ public Map<String, Set<String>> getIssueIds(String projectName, String commitId) {
+ Map<String, Set<String>> ret = Maps.newHashMap();
+ String commitMessage = commitMessageFetcher.fetchGuarded(projectName, commitId);
addIssuesOccurrence(commitMessage, "somewhere", ret);
@@ -145,12 +134,12 @@
addIssuesOccurrence(lines[0], "subject", ret);
// Determining footer line numbers
- int currentLine = lines.length-1;
- while (currentLine >=0 && lines[currentLine].isEmpty()) {
+ int currentLine = lines.length - 1;
+ while (currentLine >= 0 && lines[currentLine].isEmpty()) {
currentLine--;
}
int footerEnd = currentLine + 1;
- while (currentLine >=0 && !lines[currentLine].isEmpty()) {
+ while (currentLine >= 0 && !lines[currentLine].isEmpty()) {
currentLine--;
}
int footerStart = currentLine + 1;
@@ -168,7 +157,7 @@
// No footer could be found. So all lines after the first one (that's
// the subject) is the body.
//body = String[] templateParameters =
- // Arrays.copyOfRange(allParameters, 1, allParameters.length);
+ // Arrays.copyOfRange(allParameters, 1, allParameters.length);
if (lines.length > 0) {
body = StringUtils.join(lines, "\n", 1, lines.length);
}
@@ -207,30 +196,30 @@
/**
* Gets issues for a commit with new issue occurrences marked as "added".
- * <p>
- * Fetches the patch set's immediate ancestor and compares issue occurrences
- * between them. Any new occurrence gets marked as "added." So if for
- * example in patch sets 1, and 2 issue 23 occurs in the subject, while in
- * patch set the issue occurs in the body, then patch set 2 has occurrences
- * "somewhere", and "subject" for issue 23. Patch set 3 has occurrences
- * "somewhere", "body", and "body-added" for issue 23.
+ *
+ * <p>Fetches the patch set's immediate ancestor and compares issue occurrences between them. Any
+ * new occurrence gets marked as "added." So if for example in patch sets 1, and 2 issue 23 occurs
+ * in the subject, while in patch set the issue occurs in the body, then patch set 2 has
+ * occurrences "somewhere", and "subject" for issue 23. Patch set 3 has occurrences "somewhere",
+ * "body", and "body-added" for issue 23.
*
* @param projectName The project to fetch {@code commitId} from.
* @param commitId The commit id to fetch issues for.
- * @param patchSetId The patch set for the {@code commitId}. If it is null,
- * no occurrence can be marked as "-added".
- * @return A mapping, whose keys are issue ids and whose values is a set of
- * places where the issue occurs. Each issue occurs at least in
- * "somewhere". Issues from the first line get tagged with an occurrence
- * "subject". Issues in the last block get tagged with "footer". Issues
- * occurring between "subject" and "footer" get tagged with "body".
+ * @param patchSetId The patch set for the {@code commitId}. If it is null, no occurrence can be
+ * marked as "-added".
+ * @return A mapping, whose keys are issue ids and whose values is a set of places where the issue
+ * occurs. Each issue occurs at least in "somewhere". Issues from the first line get tagged
+ * with an occurrence "subject". Issues in the last block get tagged with "footer". Issues
+ * occurring between "subject" and "footer" get tagged with "body".
*/
- public Map<String, Set<String>> getIssueIds(String projectName, String commitId, PatchSet.Id patchSetId) {
+ public Map<String, Set<String>> getIssueIds(
+ String projectName, String commitId, PatchSet.Id patchSetId) {
Map<String, Set<String>> current = getIssueIds(projectName, commitId);
if (patchSetId != null) {
Map<String, Set<String>> previous = Maps.newHashMap();
if (patchSetId.get() != 1) {
- PatchSet.Id previousPatchSetId = new PatchSet.Id(patchSetId.getParentKey(), patchSetId.get() - 1);
+ PatchSet.Id previousPatchSetId =
+ new PatchSet.Id(patchSetId.getParentKey(), patchSetId.get() - 1);
String previousPatchSet = db.getRevision(previousPatchSetId);
if (previousPatchSet != null) {
previous = getIssueIds(projectName, previousPatchSet);
@@ -248,7 +237,7 @@
newOccurrences.removeAll(previousOccurrences);
}
for (String occurrence : newOccurrences) {
- currentOccurrences.add( "added@" + occurrence);
+ currentOccurrences.add("added@" + occurrence);
}
}
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractor.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractor.java
index a306240..2593c16 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractor.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractor.java
@@ -21,16 +21,11 @@
import com.google.gerrit.server.data.PatchSetAttribute;
import com.google.gerrit.server.data.RefUpdateAttribute;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
import java.util.Set;
-/**
- * Extractor to translate the various {@code *Attribute}s to
- * {@link Property Properties}.
- */
+/** Extractor to translate the various {@code *Attribute}s to {@link Property Properties}. */
public class PropertyAttributeExtractor {
private Property.Factory propertyFactory;
private ItsFacade its;
@@ -41,25 +36,18 @@
this.propertyFactory = propertyFactory;
}
- public Set<Property> extractFrom(AccountAttribute accountAttribute,
- String prefix) {
+ public Set<Property> extractFrom(AccountAttribute accountAttribute, String prefix) {
Set<Property> properties = Sets.newHashSet();
if (accountAttribute != null) {
// deprecated, to be removed soon. migrate to ones without dash.
- properties.add(propertyFactory.create(prefix + "-email",
- accountAttribute.email));
- properties.add(propertyFactory.create(prefix + "-username",
- accountAttribute.username));
- properties.add(propertyFactory.create(prefix + "-name",
- accountAttribute.name));
+ properties.add(propertyFactory.create(prefix + "-email", accountAttribute.email));
+ properties.add(propertyFactory.create(prefix + "-username", accountAttribute.username));
+ properties.add(propertyFactory.create(prefix + "-name", accountAttribute.name));
// New style configs for vm and soy
- properties.add(propertyFactory.create(prefix + "Email",
- accountAttribute.email));
- properties.add(propertyFactory.create(prefix + "Username",
- accountAttribute.username));
- properties.add(propertyFactory.create(prefix + "Name",
- accountAttribute.name));
+ properties.add(propertyFactory.create(prefix + "Email", accountAttribute.email));
+ properties.add(propertyFactory.create(prefix + "Username", accountAttribute.username));
+ properties.add(propertyFactory.create(prefix + "Name", accountAttribute.name));
}
return properties;
}
@@ -74,20 +62,19 @@
// deprecated, to be removed soon. migrate to ones without dash.
properties.add(propertyFactory.create("commit-message", changeAttribute.commitMessage));
properties.add(propertyFactory.create("change-id", changeAttribute.id));
- properties.add(propertyFactory.create("change-number",
- String.valueOf(changeAttribute.number)));
+ properties.add(propertyFactory.create("change-number", String.valueOf(changeAttribute.number)));
properties.add(propertyFactory.create("change-url", changeAttribute.url));
// New style configs for vm and soy
properties.add(propertyFactory.create("commitMessage", changeAttribute.commitMessage));
properties.add(propertyFactory.create("changeId", changeAttribute.id));
- properties.add(propertyFactory.create("changeNumber",
- String.valueOf(changeAttribute.number)));
+ properties.add(propertyFactory.create("changeNumber", String.valueOf(changeAttribute.number)));
properties.add(propertyFactory.create("changeUrl", changeAttribute.url));
// Soy specfic config though will work with Velocity too
- properties.add(propertyFactory.create("formatChangeUrl",
- its.createLinkForWebui(changeAttribute.url, changeAttribute.url)));
+ properties.add(
+ propertyFactory.create(
+ "formatChangeUrl", its.createLinkForWebui(changeAttribute.url, changeAttribute.url)));
String status = null;
if (changeAttribute.status != null) {
@@ -98,69 +85,59 @@
return properties;
}
- public Set<Property>extractFrom(PatchSetAttribute patchSetAttribute) {
+ public Set<Property> extractFrom(PatchSetAttribute patchSetAttribute) {
Set<Property> properties = Sets.newHashSet();
- properties.add(propertyFactory.create("revision",
- patchSetAttribute.revision));
+ properties.add(propertyFactory.create("revision", patchSetAttribute.revision));
// deprecated, to be removed soon. migrate to ones without dash.
- properties.add(propertyFactory.create("patch-set-number",
- String.valueOf(patchSetAttribute.number)));
+ properties.add(
+ propertyFactory.create("patch-set-number", String.valueOf(patchSetAttribute.number)));
// New style configs for vm and soy
- properties.add(propertyFactory.create("patchSetNumber",
- String.valueOf(patchSetAttribute.number)));
+ properties.add(
+ propertyFactory.create("patchSetNumber", String.valueOf(patchSetAttribute.number)));
properties.add(propertyFactory.create("ref", patchSetAttribute.ref));
// deprecated, to be removed soon. migrate to ones without dash.
- properties.add(propertyFactory.create("created-on",
- patchSetAttribute.createdOn.toString()));
+ properties.add(propertyFactory.create("created-on", patchSetAttribute.createdOn.toString()));
// New style configs for vm and soy
- properties.add(propertyFactory.create("createdOn",
- patchSetAttribute.createdOn.toString()));
+ properties.add(propertyFactory.create("createdOn", patchSetAttribute.createdOn.toString()));
- properties.add(propertyFactory.create("parents",
- patchSetAttribute.parents.toString()));
- properties.add(propertyFactory.create("deletions",
- Integer.toString(patchSetAttribute.sizeDeletions)));
- properties.add(propertyFactory.create("insertions",
- Integer.toString(patchSetAttribute.sizeInsertions)));
- properties.addAll(extractFrom(patchSetAttribute.uploader,
- "uploader"));
- properties.addAll(extractFrom(patchSetAttribute.author,
- "author"));
+ properties.add(propertyFactory.create("parents", patchSetAttribute.parents.toString()));
+ properties.add(
+ propertyFactory.create("deletions", Integer.toString(patchSetAttribute.sizeDeletions)));
+ properties.add(
+ propertyFactory.create("insertions", Integer.toString(patchSetAttribute.sizeInsertions)));
+ properties.addAll(extractFrom(patchSetAttribute.uploader, "uploader"));
+ properties.addAll(extractFrom(patchSetAttribute.author, "author"));
return properties;
}
- public Set<Property>extractFrom(RefUpdateAttribute refUpdateAttribute) {
+ public Set<Property> extractFrom(RefUpdateAttribute refUpdateAttribute) {
Set<Property> properties = Sets.newHashSet();
- properties.add(propertyFactory.create("revision",
- refUpdateAttribute.newRev));
+ properties.add(propertyFactory.create("revision", refUpdateAttribute.newRev));
// deprecated, to be removed soon. migrate to ones without dash.
- properties.add(propertyFactory.create("revision-old",
- refUpdateAttribute.oldRev));
+ properties.add(propertyFactory.create("revision-old", refUpdateAttribute.oldRev));
// New style configs for vm and soy
- properties.add(propertyFactory.create("revisionOld",
- refUpdateAttribute.oldRev));
- properties.add(propertyFactory.create("project",
- refUpdateAttribute.project));
- properties.add(propertyFactory.create("ref",
- refUpdateAttribute.refName));
+ properties.add(propertyFactory.create("revisionOld", refUpdateAttribute.oldRev));
+ properties.add(propertyFactory.create("project", refUpdateAttribute.project));
+ properties.add(propertyFactory.create("ref", refUpdateAttribute.refName));
return properties;
}
- public Set<Property>extractFrom(ApprovalAttribute approvalAttribute) {
+ public Set<Property> extractFrom(ApprovalAttribute approvalAttribute) {
Set<Property> properties = Sets.newHashSet();
// deprecated, to be removed soon. migrate to ones without dash.
- properties.add(propertyFactory.create("approval-" +
- approvalAttribute.type, approvalAttribute.value));
+ properties.add(
+ propertyFactory.create("approval-" + approvalAttribute.type, approvalAttribute.value));
// New style configs for vm and soy
- properties.add(propertyFactory.create("approval" +
- approvalAttribute.type.replace("-", ""), approvalAttribute.value));
+ properties.add(
+ propertyFactory.create(
+ "approval" + approvalAttribute.type.replace("-", ""), approvalAttribute.value));
return properties;
}
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractor.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractor.java
index 72a3089..3442cbf 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractor.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractor.java
@@ -32,19 +32,13 @@
import com.google.gerrit.server.events.PatchSetEvent;
import com.google.gerrit.server.events.RefUpdatedEvent;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
-import org.eclipse.jgit.lib.ObjectId;
-
import java.util.Collections;
import java.util.Map;
import java.util.Set;
+import org.eclipse.jgit.lib.ObjectId;
-/**
- * Extractor to translate an {@link ChangeEvent} to
- * {@link Property Properties}.
- */
+/** Extractor to translate an {@link ChangeEvent} to {@link Property Properties}. */
public class PropertyExtractor {
private IssueExtractor issueExtractor;
private Property.Factory propertyFactory;
@@ -52,7 +46,8 @@
private final String pluginName;
@Inject
- PropertyExtractor(IssueExtractor issueExtractor,
+ PropertyExtractor(
+ IssueExtractor issueExtractor,
Property.Factory propertyFactory,
PropertyAttributeExtractor propertyAttributeExtractor,
@PluginName String pluginName) {
@@ -64,56 +59,49 @@
/**
* creates a patch id for change id string and patchset id string.
+ *
* @param changeId String representation of the patch sets {@code Change.Id@}
* @param patchId String representation of the patch sets {@code Patchset.Id@}
- * @return PatchSet.Id for the specified patch set. If the String to int
- * conversion fails for any of the parameters, null is returned.
+ * @return PatchSet.Id for the specified patch set. If the String to int conversion fails for any
+ * of the parameters, null is returned.
*/
private PatchSet.Id newPatchSetId(String changeId, String patchId) {
try {
- return new PatchSet.Id(new Change.Id(Integer.parseInt(changeId)),
- Integer.parseInt(patchId));
+ return new PatchSet.Id(new Change.Id(Integer.parseInt(changeId)), Integer.parseInt(patchId));
} catch (NumberFormatException e) {
return null;
}
}
- private Map<String,Set<String>> extractFrom(PatchSetEvent event,
- Set<Property> common) {
+ private Map<String, Set<String>> extractFrom(PatchSetEvent event, Set<Property> common) {
common.add(propertyFactory.create("event-type", event.type));
ChangeAttribute change = event.change.get();
PatchSetAttribute patchSet = event.patchSet.get();
common.addAll(propertyAttributeExtractor.extractFrom(change));
common.addAll(propertyAttributeExtractor.extractFrom(patchSet));
PatchSet.Id patchSetId =
- newPatchSetId(Integer.toString(change.number),
- Integer.toString(patchSet.number));
- return issueExtractor.getIssueIds(change.project,
- patchSet.revision, patchSetId);
+ newPatchSetId(Integer.toString(change.number), Integer.toString(patchSet.number));
+ return issueExtractor.getIssueIds(change.project, patchSet.revision, patchSetId);
}
- private Map<String,Set<String>> extractFrom(ChangeAbandonedEvent event,
- Set<Property> common) {
+ private Map<String, Set<String>> extractFrom(ChangeAbandonedEvent event, Set<Property> common) {
common.addAll(propertyAttributeExtractor.extractFrom(event.abandoner.get(), "abandoner"));
common.add(propertyFactory.create("reason", event.reason));
return extractFrom((PatchSetEvent) event, common);
}
- private Map<String,Set<String>> extractFrom(ChangeMergedEvent event,
- Set<Property> common) {
+ private Map<String, Set<String>> extractFrom(ChangeMergedEvent event, Set<Property> common) {
common.addAll(propertyAttributeExtractor.extractFrom(event.submitter.get(), "submitter"));
return extractFrom((PatchSetEvent) event, common);
}
- private Map<String,Set<String>> extractFrom(ChangeRestoredEvent event,
- Set<Property> common) {
+ private Map<String, Set<String>> extractFrom(ChangeRestoredEvent event, Set<Property> common) {
common.addAll(propertyAttributeExtractor.extractFrom(event.restorer.get(), "restorer"));
common.add(propertyFactory.create("reason", event.reason));
return extractFrom((PatchSetEvent) event, common);
}
- private Map<String,Set<String>> extractFrom(RefUpdatedEvent event,
- Set<Property> common) {
+ private Map<String, Set<String>> extractFrom(RefUpdatedEvent event, Set<Property> common) {
common.add(propertyFactory.create("event-type", event.type));
common.addAll(propertyAttributeExtractor.extractFrom(event.submitter.get(), "submitter"));
common.addAll(propertyAttributeExtractor.extractFrom(event.refUpdate.get()));
@@ -121,23 +109,19 @@
if (ObjectId.zeroId().name().equals(refUpdated.newRev)) {
return Collections.emptyMap();
}
- return issueExtractor.getIssueIds(event.getProjectNameKey().get(),
- refUpdated.newRev);
+ return issueExtractor.getIssueIds(event.getProjectNameKey().get(), refUpdated.newRev);
}
- private Map<String,Set<String>> extractFrom(PatchSetCreatedEvent event,
- Set<Property> common) {
+ private Map<String, Set<String>> extractFrom(PatchSetCreatedEvent event, Set<Property> common) {
common.addAll(propertyAttributeExtractor.extractFrom(event.uploader.get(), "uploader"));
return extractFrom((PatchSetEvent) event, common);
}
- private Map<String,Set<String>> extractFrom(CommentAddedEvent event,
- Set<Property> common) {
+ private Map<String, Set<String>> extractFrom(CommentAddedEvent event, Set<Property> common) {
common.addAll(propertyAttributeExtractor.extractFrom(event.author.get(), "commenter"));
if (event.approvals != null) {
for (ApprovalAttribute approvalAttribute : event.approvals.get()) {
- common.addAll(propertyAttributeExtractor.extractFrom(
- approvalAttribute));
+ common.addAll(propertyAttributeExtractor.extractFrom(approvalAttribute));
}
}
common.add(propertyFactory.create("comment", event.comment));
@@ -147,10 +131,10 @@
/**
* A set of property sets extracted from an event.
*
- * As events may relate to more that a single issue, and properties sets are
- * should be tied to a single issue, returning {@code Set<Property>} is not
- * sufficient, and we need to return {@code Set<Set<Property>>}. Using this
- * approach, a PatchSetCreatedEvent for a patch set with commit message:
+ * <p>As events may relate to more that a single issue, and properties sets are should be tied to
+ * a single issue, returning {@code Set<Property>} is not sufficient, and we need to return {@code
+ * Set<Set<Property>>}. Using this approach, a PatchSetCreatedEvent for a patch set with commit
+ * message:
*
* <pre>
* (bug 4711) Fix treatment of special characters in title
@@ -176,16 +160,15 @@
* event: patchset-created
* </pre>
*
- * Thereby, sites can choose to cause different actions for different issues
- * associated to the same event. So in the above example, a comment
- * "mentioned in change 123" may be added for issue 42, and a comment
- * "fixed by change 123” may be added for issue 4711.
+ * Thereby, sites can choose to cause different actions for different issues associated to the
+ * same event. So in the above example, a comment "mentioned in change 123" may be added for issue
+ * 42, and a comment "fixed by change 123” may be added for issue 4711.
*
* @param event The event to extract property sets from.
* @return sets of property sets extracted from the event.
*/
public Set<Set<Property>> extractFrom(Event event) {
- Map<String,Set<String>> associations = null;
+ Map<String, Set<String>> associations = null;
Set<Set<Property>> ret = Sets.newHashSet();
Set<Property> common = Sets.newHashSet();
@@ -212,7 +195,7 @@
properties.add(property);
property = propertyFactory.create("its-name", pluginName);
properties.add(property);
- for (String occurrence: associations.get(issue)) {
+ for (String occurrence : associations.get(issue)) {
property = propertyFactory.create("association", occurrence);
properties.add(property);
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsAssociationPolicy.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsAssociationPolicy.java
index ea363a8..4e08b62 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsAssociationPolicy.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsAssociationPolicy.java
@@ -15,5 +15,7 @@
package com.googlesource.gerrit.plugins.its.base.validation;
public enum ItsAssociationPolicy {
- MANDATORY, SUGGESTED, OPTIONAL;
+ MANDATORY,
+ SUGGESTED,
+ OPTIONAL
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateComment.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateComment.java
index 3d8816f..969eeb0 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateComment.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateComment.java
@@ -21,35 +21,27 @@
import com.google.gerrit.server.git.validators.CommitValidationListener;
import com.google.gerrit.server.git.validators.CommitValidationMessage;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsConfig;
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.util.IssueExtractor;
-
+import java.io.IOException;
+import java.util.Collections;
+import java.util.List;
import org.eclipse.jgit.revwalk.RevCommit;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.io.IOException;
-import java.util.Collections;
-import java.util.List;
-
public class ItsValidateComment implements CommitValidationListener {
- private static final Logger log = LoggerFactory
- .getLogger(ItsValidateComment.class);
+ private static final Logger log = LoggerFactory.getLogger(ItsValidateComment.class);
- @Inject
- private ItsFacade client;
+ @Inject private ItsFacade client;
- @Inject @PluginName
- private String pluginName;
+ @Inject @PluginName private String pluginName;
- @Inject
- private ItsConfig itsConfig;
+ @Inject private ItsConfig itsConfig;
- @Inject
- private IssueExtractor issueExtractor;
+ @Inject private IssueExtractor issueExtractor;
private List<CommitValidationMessage> validCommit(RevCommit commit)
throws CommitValidationException {
@@ -70,8 +62,7 @@
try {
exists = client.exists(issueId);
} catch (IOException e) {
- synopsis = "Failed to check whether or not issue " + issueId
- + " exists";
+ synopsis = "Failed to check whether or not issue " + issueId + " exists";
log.warn(synopsis, e);
details = e.toString();
ret.add(commitValidationFailure(synopsis, details));
@@ -129,20 +120,18 @@
return ret;
}
- private CommitValidationMessage commitValidationFailure(
- String synopsis, String details) throws CommitValidationException {
- CommitValidationMessage ret =
- new CommitValidationMessage(synopsis + "\n" + details, false);
+ private CommitValidationMessage commitValidationFailure(String synopsis, String details)
+ throws CommitValidationException {
+ CommitValidationMessage ret = new CommitValidationMessage(synopsis + "\n" + details, false);
if (itsConfig.getItsAssociationPolicy() == ItsAssociationPolicy.MANDATORY) {
- throw new CommitValidationException(synopsis,
- Collections.singletonList(ret));
+ throw new CommitValidationException(synopsis, Collections.singletonList(ret));
}
return ret;
}
@Override
- public List<CommitValidationMessage> onCommitReceived(
- CommitReceivedEvent receiveEvent) throws CommitValidationException {
+ public List<CommitValidationMessage> onCommitReceived(CommitReceivedEvent receiveEvent)
+ throws CommitValidationException {
ItsConfig.setCurrentProjectName(receiveEvent.getProjectNameKey());
if (itsConfig.isEnabled(receiveEvent.getProjectNameKey(), receiveEvent.getRefName())) {
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionController.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionController.java
index 718d778..6ef9143 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionController.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionController.java
@@ -17,18 +17,16 @@
import com.google.gerrit.server.events.Event;
import com.google.gerrit.server.events.EventListener;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsConfig;
import com.googlesource.gerrit.plugins.its.base.util.PropertyExtractor;
-
import java.util.Collection;
import java.util.Set;
/**
* Controller that takes actions according to {@code ChangeEvents@}.
*
- * The taken actions are typically Its related (e.g.: adding an Its comment, or
- * changing an issue's status).
+ * <p>The taken actions are typically Its related (e.g.: adding an Its comment, or changing an
+ * issue's status).
*/
public class ActionController implements EventListener {
private final PropertyExtractor propertyExtractor;
@@ -37,8 +35,11 @@
private final ItsConfig itsConfig;
@Inject
- public ActionController(PropertyExtractor propertyExtractor,
- RuleBase ruleBase, ActionExecutor actionExecutor, ItsConfig itsConfig) {
+ public ActionController(
+ PropertyExtractor propertyExtractor,
+ RuleBase ruleBase,
+ ActionExecutor actionExecutor,
+ ItsConfig itsConfig) {
this.propertyExtractor = propertyExtractor;
this.ruleBase = ruleBase;
this.actionExecutor = actionExecutor;
@@ -51,11 +52,9 @@
return;
}
- Set<Set<Property>> propertiesCollections =
- propertyExtractor.extractFrom(event);
+ Set<Set<Property>> propertiesCollections = propertyExtractor.extractFrom(event);
for (Set<Property> properties : propertiesCollections) {
- Collection<ActionRequest> actions =
- ruleBase.actionRequestsFor(properties);
+ Collection<ActionRequest> actions = ruleBase.actionRequestsFor(properties);
if (!actions.isEmpty()) {
for (Property property : properties) {
if ("issue".equals(property.getKey())) {
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequest.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequest.java
index 0394e09..dec671e 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequest.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequest.java
@@ -17,14 +17,13 @@
import com.google.gerrit.common.Nullable;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
-
import java.util.Arrays;
/**
* An action to take for an {@code ChangeEvent}.
*
- * Actions are typically related to an Its (e.g.:adding an Its comment, or
- * changing an issue's status).
+ * <p>Actions are typically related to an Its (e.g.:adding an Its comment, or changing an issue's
+ * status).
*/
public class ActionRequest {
private final String unparsed;
@@ -47,8 +46,7 @@
/**
* Gets the name of the requested action.
*
- * @return The name of the requested action, if a name has been given.
- * "" otherwise.
+ * @return The name of the requested action, if a name has been given. "" otherwise.
*/
public String getName() {
String ret = "";
@@ -62,8 +60,7 @@
* Gets the name of the requested action.
*
* @param i The number of the parameter to extract. 1 is the first parameter.
- * @return The name of the requested parameter, if the requested parameter
- * exists. "" otherwise.
+ * @return The name of the requested parameter, if the requested parameter exists. "" otherwise.
*/
public String getParameter(int i) {
String ret = "";
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Condition.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Condition.java
index bf9f9df..6f8df84 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Condition.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Condition.java
@@ -20,25 +20,23 @@
import com.google.gerrit.common.Nullable;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
-
import com.googlesource.gerrit.plugins.its.base.workflow.action.Action;
-
import java.util.Collections;
import java.util.List;
import java.util.Set;
/**
* A condition as used in {@link Rule}, as precondition to {@link Action}s.
- * <p>
- * A condition consists of a key and an associated set of values.
- * <p>
- * For positive conditions (see constructor), the condition is said to match a
- * set of properties, if this set contains at least one property that matches
- * the rules key and whose value matches at least one of the rule's value.
- * <p>
- * For negated conditions (see constructor), the condition is said to match a
- * set of properties, if this set does not contain any property that matches
- * the rules key and whose value matches at least one of the rule's value.
+ *
+ * <p>A condition consists of a key and an associated set of values.
+ *
+ * <p>For positive conditions (see constructor), the condition is said to match a set of properties,
+ * if this set contains at least one property that matches the rules key and whose value matches at
+ * least one of the rule's value.
+ *
+ * <p>For negated conditions (see constructor), the condition is said to match a set of properties,
+ * if this set does not contain any property that matches the rules key and whose value matches at
+ * least one of the rule's value.
*/
public class Condition {
private final String key;
@@ -46,29 +44,26 @@
private final boolean negated;
public interface Factory {
- Condition create(@Assisted("key") String key,
- @Assisted("values") String values);
+ Condition create(@Assisted("key") String key, @Assisted("values") String values);
}
/**
* Constructs a condition.
+ *
* @param key The key to use for values.
- * @param values A comma separated list of values to associate to the key. If
- * the first value is not "!", it's a positive condition. If the first
- * value is "!", the "!" is removed from the values and the condition is
- * considered a negated condition.
+ * @param values A comma separated list of values to associate to the key. If the first value is
+ * not "!", it's a positive condition. If the first value is "!", the "!" is removed from the
+ * values and the condition is considered a negated condition.
*/
@Inject
- public Condition(@Assisted("key") String key,
- @Nullable @Assisted("values") String values) {
+ public Condition(@Assisted("key") String key, @Nullable @Assisted("values") String values) {
this.key = key;
Set<String> modifyableValues;
boolean modifyableNegated = false;
if (values == null) {
modifyableValues = Collections.emptySet();
} else {
- List<String> valueList = Lists.newArrayList(
- Splitter.on(',').trimResults().split(values));
+ List<String> valueList = Lists.newArrayList(Splitter.on(',').trimResults().split(values));
if (!valueList.isEmpty() && "!".equals(valueList.get(0))) {
modifyableNegated = true;
valueList.remove(0);
@@ -87,17 +82,15 @@
* Checks whether or not the Condition matches the given set of properties
*
* @param properties The set of properties to match against.
- * @return For positive conditions, true iff properties contains at least
- * one property that matches the rules key and whose value matches at
- * least one of the rule's value. For negated conditions, true iff
- * properties does not contain any property that matches the rules key
- * and whose value matches at least one of the rule's value.
+ * @return For positive conditions, true iff properties contains at least one property that
+ * matches the rules key and whose value matches at least one of the rule's value. For negated
+ * conditions, true iff properties does not contain any property that matches the rules key
+ * and whose value matches at least one of the rule's value.
*/
public boolean isMetBy(Iterable<Property> properties) {
for (Property property : properties) {
String propertyKey = property.getKey();
- if ((key == null && propertyKey == null)
- || (key != null && key.equals(propertyKey))) {
+ if ((key == null && propertyKey == null) || (key != null && key.equals(propertyKey))) {
if (values.contains(property.getValue())) {
return !negated;
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Property.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Property.java
index 98a8117..eb1f087 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Property.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Property.java
@@ -21,20 +21,18 @@
/**
* A property to match against {@code Condition}s.
*
- * A property is a simple key value pair.
+ * <p>A property is a simple key value pair.
*/
public class Property {
public interface Factory {
- Property create(@Assisted("key") String key,
- @Assisted("value") String value);
+ Property create(@Assisted("key") String key, @Assisted("value") String value);
}
private final String key;
private final String value;
@Inject
- public Property(@Assisted("key") String key,
- @Nullable @Assisted("value") String value) {
+ public Property(@Assisted("key") String key, @Nullable @Assisted("value") String value) {
this.key = key;
this.value = value;
}
@@ -71,8 +69,7 @@
@Override
public int hashCode() {
- return (key == null ? 0 : key.hashCode()) * 31 +
- (value == null ? 0 : value.hashCode());
+ return (key == null ? 0 : key.hashCode()) * 31 + (value == null ? 0 : value.hashCode());
}
@Override
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Rule.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Rule.java
index 4ec6be3..3efa269 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Rule.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/Rule.java
@@ -18,15 +18,12 @@
import com.google.common.collect.Sets;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
-
import java.util.Collection;
import java.util.Collections;
import java.util.List;
import java.util.Set;
-/**
- * A single rule that associates {@code Action}s to {@code Condition}s.
- */
+/** A single rule that associates {@code Action}s to {@code Condition}s. */
public class Rule {
private final String name;
private List<ActionRequest> actionRequests;
@@ -68,14 +65,13 @@
/**
* Gets this rule's the action requests for a given set of properties.
*
- * If the given set of properties meets all of the rule's conditions, the
- * rule's actions are returned. Otherwise the empty collection is returned.
+ * <p>If the given set of properties meets all of the rule's conditions, the rule's actions are
+ * returned. Otherwise the empty collection is returned.
*
* @param properties The properties to check against the rule's conditions.
* @return The actions that should get fired.
*/
- public Collection<ActionRequest> actionRequestsFor(
- Iterable<Property> properties) {
+ public Collection<ActionRequest> actionRequestsFor(Iterable<Property> properties) {
for (Condition condition : conditions) {
if (!condition.isMetBy(properties)) {
return Collections.emptyList();
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBase.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBase.java
index 2f0e1a2..41dc9fa 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBase.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBase.java
@@ -14,48 +14,35 @@
package com.googlesource.gerrit.plugins.its.base.workflow;
-
import com.google.common.collect.Lists;
import com.google.gerrit.extensions.annotations.PluginName;
import com.google.gerrit.server.config.SitePath;
import com.google.inject.Inject;
-
+import java.io.File;
+import java.io.IOException;
+import java.nio.file.Path;
+import java.util.Collection;
import org.eclipse.jgit.errors.ConfigInvalidException;
import org.eclipse.jgit.storage.file.FileBasedConfig;
import org.eclipse.jgit.util.FS;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.io.File;
-import java.io.IOException;
-import java.nio.file.Path;
-import java.util.Collection;
-
-/**
- * Collection and matcher agains {@link Rule}s.
- */
+/** Collection and matcher agains {@link Rule}s. */
public class RuleBase {
private static final Logger log = LoggerFactory.getLogger(RuleBase.class);
- /**
- * File beginning (relative to site) to load rules from
- */
- private static final String ITS_CONFIG_FILE_START = "etc" +
- File.separatorChar + "its" + File.separator + "actions";
+ /** File beginning (relative to site) to load rules from */
+ private static final String ITS_CONFIG_FILE_START =
+ "etc" + File.separatorChar + "its" + File.separator + "actions";
- /**
- * File end to load rules from
- */
+ /** File end to load rules from */
private static final String ITS_CONFIG_FILE_END = ".config";
- /**
- * The section for rules within rulebases
- */
+ /** The section for rules within rulebases */
private static final String RULE_SECTION = "rule";
- /**
- * The key for actions within rulebases
- */
+ /** The key for actions within rulebases */
private static final String ACTION_KEY = "action";
private final Path sitePath;
@@ -71,7 +58,9 @@
}
@Inject
- public RuleBase(@SitePath Path sitePath, Rule.Factory ruleFactory,
+ public RuleBase(
+ @SitePath Path sitePath,
+ Rule.Factory ruleFactory,
Condition.Factory conditionFactory,
ActionRequest.Factory actionRequestFactory,
@PluginName String pluginName) {
@@ -85,8 +74,8 @@
/**
* Adds rules from a file to the the RuleBase.
- * <p>
- * If the given file does not exist, it is silently ignored
+ *
+ * <p>If the given file does not exist, it is silently ignored
*
* @param ruleFile File from which to read the rules
*/
@@ -105,7 +94,7 @@
Rule rule = ruleFactory.create(subsection);
Collection<String> keys = cfg.getNames(RULE_SECTION, subsection);
for (String key : keys) {
- String values[] = cfg.getStringList(RULE_SECTION, subsection, key);
+ String[] values = cfg.getStringList(RULE_SECTION, subsection, key);
if (ACTION_KEY.equals(key)) {
for (String value : values) {
ActionRequest actionRequest = actionRequestFactory.create(value);
@@ -123,9 +112,7 @@
}
}
- /**
- * Loads the rules for the RuleBase.
- */
+ /** Loads the rules for the RuleBase. */
private void reloadRules() {
rules = Lists.newArrayList();
@@ -137,34 +124,40 @@
// "actions.config" (with trailing "s", we (for now) load files from both
// locations, but consider "actions.config" (with trailing "s" the
// canonical place.
- File faultyNameRuleFile = new File(sitePath.toFile(), "etc" + File.separatorChar
- + "its" + File.separator + "action.config");
+ File faultyNameRuleFile =
+ new File(
+ sitePath.toFile(),
+ "etc" + File.separatorChar + "its" + File.separator + "action.config");
if (faultyNameRuleFile.exists()) {
- log.warn("Loading rules from deprecated 'etc/its/action.config' (No "
- + "trailing 's' in 'action'). Please migrate to "
- + "'etc/its/actions.config' (Trailing 's' in 'actions').");
+ log.warn(
+ "Loading rules from deprecated 'etc/its/action.config' (No "
+ + "trailing 's' in 'action'). Please migrate to "
+ + "'etc/its/actions.config' (Trailing 's' in 'actions').");
addRulesFromFile(faultyNameRuleFile);
}
// Add global rules
- File globalRuleFile = new File(sitePath.toFile(), ITS_CONFIG_FILE_START +
- ITS_CONFIG_FILE_END);
+ File globalRuleFile = new File(sitePath.toFile(), ITS_CONFIG_FILE_START + ITS_CONFIG_FILE_END);
addRulesFromFile(globalRuleFile);
// Add its-specific rules
- File itsSpecificRuleFile = new File(sitePath.toFile(), ITS_CONFIG_FILE_START + "-" +
- pluginName + ITS_CONFIG_FILE_END);
+ File itsSpecificRuleFile =
+ new File(sitePath.toFile(), ITS_CONFIG_FILE_START + "-" + pluginName + ITS_CONFIG_FILE_END);
addRulesFromFile(itsSpecificRuleFile);
if (!globalRuleFile.exists() && !itsSpecificRuleFile.exists()) {
try {
- log.warn("Neither global rule file "
- + globalRuleFile.getCanonicalPath() + " nor Its specific rule file"
- + itsSpecificRuleFile.getCanonicalPath() + " exist. Please configure "
- + "rules.");
+ log.warn(
+ "Neither global rule file "
+ + globalRuleFile.getCanonicalPath()
+ + " nor Its specific rule file"
+ + itsSpecificRuleFile.getCanonicalPath()
+ + " exist. Please configure "
+ + "rules.");
} catch (IOException e) {
- log.warn("Neither global rule file nor Its specific rule files exist. "
- + "Please configure rules.");
+ log.warn(
+ "Neither global rule file nor Its specific rule files exist. "
+ + "Please configure rules.");
}
}
}
@@ -175,8 +168,7 @@
* @param properties The properties to search actions for.
* @return Requests for the actions that should be fired.
*/
- public Collection<ActionRequest> actionRequestsFor(
- Iterable<Property> properties) {
+ public Collection<ActionRequest> actionRequestsFor(Iterable<Property> properties) {
Collection<ActionRequest> ret = Lists.newLinkedList();
for (Rule rule : rules) {
ret.addAll(rule.actionRequestsFor(properties));
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/Action.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/Action.java
index 24feee5..b8b2102 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/Action.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/Action.java
@@ -16,13 +16,10 @@
import com.googlesource.gerrit.plugins.its.base.workflow.ActionRequest;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
import java.io.IOException;
import java.util.Set;
-/**
- * Interface for actions on an issue tracking system
- */
+/** Interface for actions on an issue tracking system */
public interface Action {
/**
@@ -32,6 +29,6 @@
* @param actionRequest The request to execute.
* @param properties The properties for the execution.
*/
- public void execute(String issue, ActionRequest actionRequest,
- Set<Property> properties) throws IOException;
+ public void execute(String issue, ActionRequest actionRequest, Set<Property> properties)
+ throws IOException;
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddComment.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddComment.java
index 8ca9c59..16855d3 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddComment.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddComment.java
@@ -16,20 +16,17 @@
import com.google.common.base.Strings;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.workflow.ActionRequest;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
-import org.apache.commons.lang.StringUtils;
-
import java.io.IOException;
import java.util.Set;
+import org.apache.commons.lang.StringUtils;
/**
* Adds a fixed comment to an issue.
*
- * The action requests parameters get concatenated and get added to the issue.
+ * <p>The action requests parameters get concatenated and get added to the issue.
*/
public class AddComment implements Action {
public interface Factory {
@@ -44,8 +41,8 @@
}
@Override
- public void execute(String issue, ActionRequest actionRequest,
- Set<Property> properties) throws IOException {
+ public void execute(String issue, ActionRequest actionRequest, Set<Property> properties)
+ throws IOException {
String[] parameters = actionRequest.getParameters();
String comment = StringUtils.join(parameters, " ");
if (!Strings.isNullOrEmpty(comment)) {
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddSoyComment.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddSoyComment.java
index f65d6c6..bbad6e9 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddSoyComment.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddSoyComment.java
@@ -28,15 +28,11 @@
import java.io.File;
import java.io.IOException;
import java.io.Reader;
-import java.io.StringWriter;
import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Path;
-import java.util.Arrays;
import java.util.HashMap;
-import java.util.Map;
import java.util.Set;
-import org.apache.commons.lang.StringUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -66,7 +62,7 @@
this.its = its;
}
- private HashMap getSoyContext(Set<Property> properties) {
+ private HashMap<String, Object> getSoyContext(Set<Property> properties) {
HashMap<String, Object> soyContext = new HashMap<>();
for (Property property : properties) {
String key = property.getKey();
@@ -81,13 +77,17 @@
return soyContext;
}
- private String soyTemplate(SoyFileSet.Builder builder, String template, SanitizedContent.ContentKind kind, Set<Property> properties) {
+ private String soyTemplate(
+ SoyFileSet.Builder builder,
+ String template,
+ SanitizedContent.ContentKind kind,
+ Set<Property> properties) {
Path templateDir = sitePath.resolve(ITS_TEMPLATE_DIR);
Path templatePath = templateDir.resolve(template + ".soy");
String content;
try (Reader r = Files.newBufferedReader(templatePath, StandardCharsets.UTF_8)) {
- content = CharStreams.toString(r);
+ content = CharStreams.toString(r);
} catch (IOException err) {
throw new ProvisionException(
"Failed to read template file " + templatePath.toAbsolutePath().toString(), err);
@@ -95,7 +95,7 @@
builder.add(content, templatePath.toAbsolutePath().toString());
- HashMap context = getSoyContext(properties);
+ HashMap<String, Object> context = getSoyContext(properties);
SoyTofu.Renderer renderer =
builder
@@ -107,7 +107,8 @@
return renderer.render();
}
- protected String soyTextTemplate(SoyFileSet.Builder builder, String template, Set<Property> properties) {
+ protected String soyTextTemplate(
+ SoyFileSet.Builder builder, String template, Set<Property> properties) {
return soyTemplate(builder, template, SanitizedContent.ContentKind.TEXT, properties);
}
@@ -118,7 +119,7 @@
String template = null;
String templateName = actionRequest.getParameter(1);
if (templateName.isEmpty()) {
- log.error("No template name given in " + actionRequest);
+ log.error("No template name given in {}", actionRequest);
} else {
template = templateName;
}
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardComment.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardComment.java
index b033603..eb8c4a6 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardComment.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardComment.java
@@ -17,11 +17,9 @@
import com.google.common.base.Strings;
import com.google.common.collect.Maps;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.workflow.ActionRequest;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
import java.io.IOException;
import java.util.Map;
import java.util.Set;
@@ -29,8 +27,7 @@
/**
* Adds a short predefined comments to an issue.
*
- * Comments are added for merging, abandoning, restoring of changes and adding
- * of patch sets.
+ * <p>Comments are added for merging, abandoning, restoring of changes and adding of patch sets.
*/
public class AddStandardComment implements Action {
public interface Factory {
@@ -54,16 +51,14 @@
return ret;
}
- private String getCommentChangeEvent(String Action, String prefix,
- Map<String, String> map) {
+ private String getCommentChangeEvent(String action, String prefix, Map<String, String> map) {
String ret = "";
- String changeNumber = Strings.nullToEmpty(map.get("change-number"));
- changeNumber = Strings.nullToEmpty(map.get("changeNumber"));
+ String changeNumber = getValueFromMap(map, "", "change-number", "changeNumber");
if (!changeNumber.isEmpty()) {
changeNumber += " ";
}
- ret += "Change " + changeNumber + Action;
- String submitter = formatPerson(prefix, map);
+ ret += "Change " + changeNumber + action;
+ String submitter = getValueFromMap(map, prefix, "-name", "Name", "-username", "Username");
if (!submitter.isEmpty()) {
ret += " by " + submitter;
}
@@ -75,14 +70,23 @@
if (!reason.isEmpty()) {
ret += "\n\nReason:\n" + reason;
}
- String url = Strings.nullToEmpty(map.get("change-url"));
- url = Strings.nullToEmpty(map.get("changeUrl"));
+ String url = getValueFromMap(map, "", "change-url", "changeUrl");
if (!url.isEmpty()) {
ret += "\n\n" + its.createLinkForWebui(url, url);
}
return ret;
}
+ private String getValueFromMap(Map<String, String> map, String keyPrefix, String... keyOptions) {
+ for (String key : keyOptions) {
+ String ret = Strings.nullToEmpty(map.get(keyPrefix + key));
+ if (!ret.isEmpty()) {
+ return ret;
+ }
+ }
+ return "";
+ }
+
private String getCommentChangeAbandoned(Map<String, String> map) {
return getCommentChangeEvent("abandoned", "abandoner", map);
}
@@ -96,19 +100,17 @@
}
private String getCommentPatchSetCreated(Map<String, String> map) {
- return getCommentChangeEvent("had a related patch set uploaded",
- "uploader", map);
+ return getCommentChangeEvent("had a related patch set uploaded", "uploader", map);
}
@Override
- public void execute(String issue, ActionRequest actionRequest,
- Set<Property> properties) throws IOException {
+ public void execute(String issue, ActionRequest actionRequest, Set<Property> properties)
+ throws IOException {
String comment = "";
Map<String, String> map = Maps.newHashMap();
for (Property property : properties) {
String current = property.getValue();
- if (!Strings.isNullOrEmpty(current))
- {
+ if (!Strings.isNullOrEmpty(current)) {
String key = property.getKey();
String old = Strings.nullToEmpty(map.get(key));
if (!old.isEmpty()) {
diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEvent.java b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEvent.java
index d66dbce..3a46202 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEvent.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEvent.java
@@ -15,34 +15,34 @@
package com.googlesource.gerrit.plugins.its.base.workflow.action;
import com.google.inject.Inject;
-
import com.googlesource.gerrit.plugins.its.base.workflow.ActionRequest;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
import java.io.IOException;
import java.util.Set;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
/**
* Dumps the event's properties to the log.
*
- * This event helps when developing rules as available properties become
- * visible.
+ * <p>This event helps when developing rules as available properties become visible.
*/
public class LogEvent implements Action {
private static final Logger log = LoggerFactory.getLogger(LogEvent.class);
- private enum Level { ERROR, WARN, INFO, DEBUG }
+ private enum Level {
+ ERROR,
+ WARN,
+ INFO,
+ DEBUG
+ }
public interface Factory {
LogEvent create();
}
@Inject
- public LogEvent() {
- }
+ public LogEvent() {}
private void logProperty(Level level, Property property) {
String message = property.toString();
@@ -65,8 +65,8 @@
}
@Override
- public void execute(String issue, ActionRequest actionRequest,
- Set<Property> properties) throws IOException {
+ public void execute(String issue, ActionRequest actionRequest, Set<Property> properties)
+ throws IOException {
String levelParameter = actionRequest.getParameter(1);
if (levelParameter != null) {
levelParameter = levelParameter.toLowerCase();
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfigTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfigTest.java
index f0f3fe2..6561d00 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfigTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/its/ItsConfigTest.java
@@ -35,13 +35,10 @@
import com.google.gerrit.server.project.ProjectState;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.validation.ItsAssociationPolicy;
-
-import org.eclipse.jgit.lib.Config;
-
import java.util.Arrays;
+import org.eclipse.jgit.lib.Config;
public class ItsConfigTest extends LoggingMockingTestCase {
private Injector injector;
@@ -50,14 +47,13 @@
private PluginConfigFactory pluginConfigFactory;
private Config serverConfig;
- public void setupIsEnabled(String enabled, String parentEnabled,
- String[] branches) {
+ public void setupIsEnabled(String enabled, String parentEnabled, String[] branches) {
ProjectState projectState = createMock(ProjectState.class);
- expect(projectCache.get(new Project.NameKey("testProject")))
- .andReturn(projectState).anyTimes();
+ expect(projectCache.get(new Project.NameKey("testProject"))).andReturn(projectState).anyTimes();
expect(projectCache.get(new Project.NameKey("parentProject")))
- .andReturn(projectState).anyTimes();
+ .andReturn(projectState)
+ .anyTimes();
Iterable<ProjectState> parents;
if (parentEnabled == null) {
@@ -67,21 +63,21 @@
PluginConfig parentPluginConfig = createMock(PluginConfig.class);
- expect(pluginConfigFactory.getFromProjectConfig(
- parentProjectState, "ItsTestName")).andReturn(parentPluginConfig);
+ expect(pluginConfigFactory.getFromProjectConfig(parentProjectState, "ItsTestName"))
+ .andReturn(parentPluginConfig);
- expect(parentPluginConfig.getString("enabled", "false")).andReturn(parentEnabled)
- .anyTimes();
+ expect(parentPluginConfig.getString("enabled", "false")).andReturn(parentEnabled).anyTimes();
PluginConfig parentPluginConfigWI = createMock(PluginConfig.class);
- expect(pluginConfigFactory.getFromProjectConfigWithInheritance(
- parentProjectState, "ItsTestName")).andReturn(parentPluginConfigWI)
+ expect(
+ pluginConfigFactory.getFromProjectConfigWithInheritance(
+ parentProjectState, "ItsTestName"))
+ .andReturn(parentPluginConfigWI)
.anyTimes();
- String[] parentBranches = { "refs/heads/testBranch" };
- expect(parentPluginConfigWI.getStringList("branch"))
- .andReturn(parentBranches).anyTimes();
+ String[] parentBranches = {"refs/heads/testBranch"};
+ expect(parentPluginConfigWI.getStringList("branch")).andReturn(parentBranches).anyTimes();
parents = Arrays.asList(parentProjectState, projectState);
}
@@ -89,21 +85,21 @@
PluginConfig pluginConfig = createMock(PluginConfig.class);
- expect(pluginConfigFactory.getFromProjectConfig(
- projectState, "ItsTestName")).andReturn(pluginConfig).anyTimes();
+ expect(pluginConfigFactory.getFromProjectConfig(projectState, "ItsTestName"))
+ .andReturn(pluginConfig)
+ .anyTimes();
expect(pluginConfig.getString("enabled", "false")).andReturn(enabled).anyTimes();
PluginConfig pluginConfigWI = createMock(PluginConfig.class);
- expect(pluginConfigFactory.getFromProjectConfigWithInheritance(
- projectState, "ItsTestName")).andReturn(pluginConfigWI).anyTimes();
-
- expect(pluginConfigWI.getString("enabled", "false"))
- .andReturn(enabled).anyTimes();
-
- expect(pluginConfigWI.getStringList("branch")).andReturn(branches)
+ expect(pluginConfigFactory.getFromProjectConfigWithInheritance(projectState, "ItsTestName"))
+ .andReturn(pluginConfigWI)
.anyTimes();
+
+ expect(pluginConfigWI.getString("enabled", "false")).andReturn(enabled).anyTimes();
+
+ expect(pluginConfigWI.getStringList("branch")).andReturn(branches).anyTimes();
}
public void testIsEnabledRefNoParentNoBranchEnabled() {
@@ -290,8 +286,7 @@
String[] branches = {};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -304,8 +299,7 @@
String[] branches = {"refs/heads/testBranch"};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -318,8 +312,7 @@
String[] branches = {"^refs/heads/test.*"};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -332,8 +325,7 @@
String[] branches = {"^refs/heads/foo.*"};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -346,8 +338,7 @@
String[] branches = {"refs/heads/foo", "refs/heads/testBranch"};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -360,8 +351,7 @@
String[] branches = {"^refs/heads/foo.*", "^refs/heads/test.*"};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -374,8 +364,7 @@
String[] branches = {"refs/heads/testBranch", "refs/heads/foo.*"};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -384,12 +373,11 @@
assertTrue(itsConfig.isEnabled(event));
}
-public void testIsEnabledEventMultiBranchMixedMatchRegExp() {
+ public void testIsEnabledEventMultiBranchMixedMatchRegExp() {
String[] branches = {"refs/heads/foo", "^refs/heads/test.*"};
setupIsEnabled("true", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -402,8 +390,7 @@
String[] branches = {"^refs/heads/testBranch"};
setupIsEnabled("false", null, branches);
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -416,8 +403,7 @@
String[] branches = {};
setupIsEnabled("true", null, branches);
- CommentAddedEvent event =
- new CommentAddedEvent(testChange("testProject", "testBranch"));
+ CommentAddedEvent event = new CommentAddedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -430,8 +416,7 @@
String[] branches = {};
setupIsEnabled("true", null, branches);
- ChangeMergedEvent event =
- new ChangeMergedEvent(testChange("testProject", "testBranch"));
+ ChangeMergedEvent event = new ChangeMergedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -444,8 +429,7 @@
String[] branches = {};
setupIsEnabled("true", null, branches);
- ChangeAbandonedEvent event =
- new ChangeAbandonedEvent(testChange("testProject", "testBranch"));
+ ChangeAbandonedEvent event = new ChangeAbandonedEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -458,8 +442,7 @@
String[] branches = {};
setupIsEnabled("true", null, branches);
- ChangeRestoredEvent event =
- new ChangeRestoredEvent(testChange("testProject", "testBranch"));
+ ChangeRestoredEvent event = new ChangeRestoredEvent(testChange("testProject", "testBranch"));
ItsConfig itsConfig = createItsConfig();
@@ -500,272 +483,321 @@
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
replayMocks();
- assertNull("Pattern for null match is not null",
- itsConfig.getIssuePattern());
+ assertNull("Pattern for null match is not null", itsConfig.getIssuePattern());
}
public void testGetIssuePatternNullMatchWCommentLink() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn("foo").atLeastOnce();
- expect(serverConfig.getString("commentlink", "foo", "match"))
- .andReturn(null).atLeastOnce();
+ .andReturn("foo")
+ .atLeastOnce();
+ expect(serverConfig.getString("commentlink", "foo", "match")).andReturn(null).atLeastOnce();
replayMocks();
- assertNull("Pattern for null match is not null",
- itsConfig.getIssuePattern());
+ assertNull("Pattern for null match is not null", itsConfig.getIssuePattern());
}
public void testGetIssuePattern() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn("TestPattern").atLeastOnce();
+ .andReturn("TestPattern")
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and generated pattern are not equal",
- "TestPattern", itsConfig.getIssuePattern().pattern());
+ assertEquals(
+ "Expected and generated pattern are not equal",
+ "TestPattern",
+ itsConfig.getIssuePattern().pattern());
}
public void testGetIssuePatternWCommentLink() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn("foo").atLeastOnce();
+ .andReturn("foo")
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "foo", "match"))
- .andReturn("TestPattern").atLeastOnce();
+ .andReturn("TestPattern")
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and generated pattern are not equal",
- "TestPattern", itsConfig.getIssuePattern().pattern());
-
+ assertEquals(
+ "Expected and generated pattern are not equal",
+ "TestPattern",
+ itsConfig.getIssuePattern().pattern());
}
public void testGetIssuePatternGroupIndexGroupDefault() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn("(foo)(bar)(baz)").atLeastOnce();
+ .andReturn("(foo)(bar)(baz)")
+ .atLeastOnce();
expect(serverConfig.getInt("plugin", "ItsTestName", "commentlinkGroupIndex", 1))
- .andReturn(1).atLeastOnce();
+ .andReturn(1)
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and actual group index do not match",
- 1, itsConfig.getIssuePatternGroupIndex());
+ assertEquals(
+ "Expected and actual group index do not match", 1, itsConfig.getIssuePatternGroupIndex());
}
public void testGetIssuePatternGroupIndexGroupDefaultGroupless() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn("foo").atLeastOnce();
+ .andReturn("foo")
+ .atLeastOnce();
expect(serverConfig.getInt("plugin", "ItsTestName", "commentlinkGroupIndex", 1))
- .andReturn(1).atLeastOnce();
+ .andReturn(1)
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and actual group index do not match",
- 0, itsConfig.getIssuePatternGroupIndex());
+ assertEquals(
+ "Expected and actual group index do not match", 0, itsConfig.getIssuePatternGroupIndex());
}
public void testGetIssuePatternGroupIndexGroup1() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn("(foo)(bar)(baz)").atLeastOnce();
+ .andReturn("(foo)(bar)(baz)")
+ .atLeastOnce();
expect(serverConfig.getInt("plugin", "ItsTestName", "commentlinkGroupIndex", 1))
- .andReturn(1).atLeastOnce();
+ .andReturn(1)
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and actual group index do not match",
- 1, itsConfig.getIssuePatternGroupIndex());
+ assertEquals(
+ "Expected and actual group index do not match", 1, itsConfig.getIssuePatternGroupIndex());
}
public void testGetIssuePatternGroupIndexGroup3() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn("(foo)(bar)(baz)").atLeastOnce();
+ .andReturn("(foo)(bar)(baz)")
+ .atLeastOnce();
expect(serverConfig.getInt("plugin", "ItsTestName", "commentlinkGroupIndex", 1))
- .andReturn(3).atLeastOnce();
+ .andReturn(3)
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and actual group index do not match",
- 3, itsConfig.getIssuePatternGroupIndex());
+ assertEquals(
+ "Expected and actual group index do not match", 3, itsConfig.getIssuePatternGroupIndex());
}
public void testGetIssuePatternGroupIndexGroupTooHigh() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn("(foo)(bar)(baz)").atLeastOnce();
+ .andReturn("(foo)(bar)(baz)")
+ .atLeastOnce();
expect(serverConfig.getInt("plugin", "ItsTestName", "commentlinkGroupIndex", 1))
- .andReturn(5).atLeastOnce();
+ .andReturn(5)
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and actual group index do not match",
- 1, itsConfig.getIssuePatternGroupIndex());
+ assertEquals(
+ "Expected and actual group index do not match", 1, itsConfig.getIssuePatternGroupIndex());
}
public void testGetIssuePatternGroupIndexGroupTooHighGroupless() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
+ .andReturn(null)
+ .atLeastOnce();
expect(serverConfig.getString("commentlink", "ItsTestName", "match"))
- .andReturn("foo").atLeastOnce();
+ .andReturn("foo")
+ .atLeastOnce();
expect(serverConfig.getInt("plugin", "ItsTestName", "commentlinkGroupIndex", 1))
- .andReturn(5).atLeastOnce();
+ .andReturn(5)
+ .atLeastOnce();
replayMocks();
- assertEquals("Expected and actual group index do not match",
- 0, itsConfig.getIssuePatternGroupIndex());
+ assertEquals(
+ "Expected and actual group index do not match", 0, itsConfig.getIssuePatternGroupIndex());
}
public void testGetItsAssociationPolicyOptional() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
- expect(serverConfig.getEnum("commentlink", "ItsTestName", "association",
- ItsAssociationPolicy.OPTIONAL))
+ .andReturn(null)
+ .atLeastOnce();
+ expect(
+ serverConfig.getEnum(
+ "commentlink", "ItsTestName", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.OPTIONAL)
.atLeastOnce();
- expect(serverConfig.getEnum("plugin", "ItsTestName", "association",
- ItsAssociationPolicy.OPTIONAL))
+ expect(
+ serverConfig.getEnum(
+ "plugin", "ItsTestName", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.OPTIONAL)
.atLeastOnce();
replayMocks();
- assertEquals("Expected and generated associated policy do not match",
- ItsAssociationPolicy.OPTIONAL, itsConfig.getItsAssociationPolicy());
+ assertEquals(
+ "Expected and generated associated policy do not match",
+ ItsAssociationPolicy.OPTIONAL,
+ itsConfig.getItsAssociationPolicy());
}
public void testGetItsAssociationPolicyOptionalWCommentLink() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn("foo").atLeastOnce();
- expect(serverConfig.getEnum("commentlink", "foo", "association",
- ItsAssociationPolicy.OPTIONAL))
+ .andReturn("foo")
+ .atLeastOnce();
+ expect(serverConfig.getEnum("commentlink", "foo", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.OPTIONAL)
.atLeastOnce();
- expect(serverConfig.getEnum("plugin", "ItsTestName", "association",
- ItsAssociationPolicy.OPTIONAL))
+ expect(
+ serverConfig.getEnum(
+ "plugin", "ItsTestName", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.OPTIONAL)
.atLeastOnce();
replayMocks();
- assertEquals("Expected and generated associated policy do not match",
- ItsAssociationPolicy.OPTIONAL, itsConfig.getItsAssociationPolicy());
+ assertEquals(
+ "Expected and generated associated policy do not match",
+ ItsAssociationPolicy.OPTIONAL,
+ itsConfig.getItsAssociationPolicy());
}
public void testGetItsAssociationPolicySuggested() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
- expect(serverConfig.getEnum("commentlink", "ItsTestName", "association",
- ItsAssociationPolicy.OPTIONAL))
+ .andReturn(null)
+ .atLeastOnce();
+ expect(
+ serverConfig.getEnum(
+ "commentlink", "ItsTestName", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
- expect(serverConfig.getEnum("plugin", "ItsTestName", "association",
- ItsAssociationPolicy.SUGGESTED))
+ expect(
+ serverConfig.getEnum(
+ "plugin", "ItsTestName", "association", ItsAssociationPolicy.SUGGESTED))
.andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
replayMocks();
- assertEquals("Expected and generated associated policy do not match",
- ItsAssociationPolicy.SUGGESTED, itsConfig.getItsAssociationPolicy());
+ assertEquals(
+ "Expected and generated associated policy do not match",
+ ItsAssociationPolicy.SUGGESTED,
+ itsConfig.getItsAssociationPolicy());
}
public void testGetItsAssociationPolicySuggestedWCommentLink() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn("foo").atLeastOnce();
- expect(serverConfig.getEnum("plugin", "ItsTestName", "association",
- ItsAssociationPolicy.SUGGESTED))
+ .andReturn("foo")
+ .atLeastOnce();
+ expect(
+ serverConfig.getEnum(
+ "plugin", "ItsTestName", "association", ItsAssociationPolicy.SUGGESTED))
.andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
- expect(serverConfig.getEnum("commentlink", "foo", "association",
- ItsAssociationPolicy.OPTIONAL))
+ expect(serverConfig.getEnum("commentlink", "foo", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
replayMocks();
- assertEquals("Expected and generated associated policy do not match",
- ItsAssociationPolicy.SUGGESTED, itsConfig.getItsAssociationPolicy());
+ assertEquals(
+ "Expected and generated associated policy do not match",
+ ItsAssociationPolicy.SUGGESTED,
+ itsConfig.getItsAssociationPolicy());
}
public void testGetItsAssociationPolicyMandatory() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn(null).atLeastOnce();
- expect(serverConfig.getEnum("commentlink", "ItsTestName", "association",
- ItsAssociationPolicy.OPTIONAL))
+ .andReturn(null)
+ .atLeastOnce();
+ expect(
+ serverConfig.getEnum(
+ "commentlink", "ItsTestName", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.MANDATORY)
.atLeastOnce();
- expect(serverConfig.getEnum("plugin", "ItsTestName", "association",
- ItsAssociationPolicy.MANDATORY))
+ expect(
+ serverConfig.getEnum(
+ "plugin", "ItsTestName", "association", ItsAssociationPolicy.MANDATORY))
.andReturn(ItsAssociationPolicy.MANDATORY)
.atLeastOnce();
replayMocks();
- assertEquals("Expected and generated associated policy do not match",
- ItsAssociationPolicy.MANDATORY, itsConfig.getItsAssociationPolicy());
+ assertEquals(
+ "Expected and generated associated policy do not match",
+ ItsAssociationPolicy.MANDATORY,
+ itsConfig.getItsAssociationPolicy());
}
public void testGetItsAssociationPolicyMandatoryWCommentLink() {
ItsConfig itsConfig = createItsConfig();
expect(serverConfig.getString("plugin", "ItsTestName", "commentlink"))
- .andReturn("foo").atLeastOnce();
- expect(serverConfig.getEnum("commentlink", "foo", "association",
- ItsAssociationPolicy.OPTIONAL))
+ .andReturn("foo")
+ .atLeastOnce();
+ expect(serverConfig.getEnum("commentlink", "foo", "association", ItsAssociationPolicy.OPTIONAL))
.andReturn(ItsAssociationPolicy.MANDATORY)
.atLeastOnce();
- expect(serverConfig.getEnum("plugin", "ItsTestName", "association",
- ItsAssociationPolicy.MANDATORY))
+ expect(
+ serverConfig.getEnum(
+ "plugin", "ItsTestName", "association", ItsAssociationPolicy.MANDATORY))
.andReturn(ItsAssociationPolicy.MANDATORY)
.atLeastOnce();
replayMocks();
- assertEquals("Expected and generated associated policy do not match",
- ItsAssociationPolicy.MANDATORY, itsConfig.getItsAssociationPolicy());
+ assertEquals(
+ "Expected and generated associated policy do not match",
+ ItsAssociationPolicy.MANDATORY,
+ itsConfig.getItsAssociationPolicy());
}
private ItsConfig createItsConfig() {
@@ -787,12 +819,10 @@
pluginConfigFactory = createMock(PluginConfigFactory.class);
bind(PluginConfigFactory.class).toInstance(pluginConfigFactory);
- bind(String.class).annotatedWith(PluginName.class)
- .toInstance("ItsTestName");
+ bind(String.class).annotatedWith(PluginName.class).toInstance("ItsTestName");
serverConfig = createMock(Config.class);
- bind(Config.class).annotatedWith(GerritServerConfig.class)
- .toInstance(serverConfig);
+ bind(Config.class).annotatedWith(GerritServerConfig.class).toInstance(serverConfig);
}
}
}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/LoggingMockingTestCase.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/LoggingMockingTestCase.java
index 6af549e..b1d98e2 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/LoggingMockingTestCase.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/LoggingMockingTestCase.java
@@ -19,20 +19,17 @@
import com.google.gerrit.reviewdb.client.Branch;
import com.google.gerrit.reviewdb.client.Change;
import com.google.gerrit.reviewdb.client.Project;
-
import com.googlesource.gerrit.plugins.its.base.testutil.log.LogUtil;
-
+import java.sql.Timestamp;
+import java.util.Iterator;
import org.apache.log4j.Level;
import org.apache.log4j.spi.LoggingEvent;
import org.junit.After;
-import java.sql.Timestamp;
-import java.util.Iterator;
-
public abstract class LoggingMockingTestCase extends MockingTestCase {
- protected final Change.Key testChangeKey = new Change.Key(
- "Ic19f7bf6c8b4685c363a8204c32d827ffda52ec0");
+ protected final Change.Key testChangeKey =
+ new Change.Key("Ic19f7bf6c8b4685c363a8204c32d827ffda52ec0");
protected final Change.Id testChangeId = new Change.Id(1);
protected final Account.Id testAccountId = new Account.Id(1);
@@ -49,10 +46,9 @@
}
}
}
- assertNotNull("Could not find log message containing '" + needle + "'",
- hit);
- assertTrue("Could not remove log message containing '" + needle + "'",
- loggedEvents.remove(hit));
+ assertNotNull("Could not find log message containing '" + needle + "'", hit);
+ assertTrue(
+ "Could not remove log message containing '" + needle + "'", loggedEvents.remove(hit));
}
protected final void assertLogMessageContains(String needle) {
@@ -65,15 +61,14 @@
while (hit == null && iter.hasNext()) {
LoggingEvent event = iter.next();
- if (event.getThrowableInformation().getThrowable().toString()
- .contains(needle)) {
+ if (event.getThrowableInformation().getThrowable().toString().contains(needle)) {
hit = event;
}
}
- assertNotNull("Could not find log message with a Throwable containing '"
- + needle + "'", hit);
- assertTrue("Could not remove log message with a Throwable containing '"
- + needle + "'", loggedEvents.remove(hit));
+ assertNotNull("Could not find log message with a Throwable containing '" + needle + "'", hit);
+ assertTrue(
+ "Could not remove log message with a Throwable containing '" + needle + "'",
+ loggedEvents.remove(hit));
}
// As the PowerMock runner does not pass through runTest, we inject log
@@ -101,7 +96,7 @@
// and we can improve later to allow tests to specify which loggers are
// to check.
String logName = this.getClass().getCanonicalName();
- logName = logName.substring(0, logName.length()-4);
+ logName = logName.substring(0, logName.length() - 4);
LogUtil.logToCollection(logName, loggedEvents);
}
@@ -116,7 +111,10 @@
}
protected Change testChange(String project, String branch) {
- return new Change(testChangeKey, testChangeId, testAccountId,
+ return new Change(
+ testChangeKey,
+ testChangeId,
+ testAccountId,
new Branch.NameKey(new Project.NameKey(project), branch),
new Timestamp(System.currentTimeMillis()));
}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/MockingTestCase.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/MockingTestCase.java
index 2f045ef..d717024 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/MockingTestCase.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/MockingTestCase.java
@@ -14,8 +14,9 @@
package com.googlesource.gerrit.plugins.its.base.testutil;
+import java.util.ArrayList;
+import java.util.Collection;
import junit.framework.TestCase;
-
import org.easymock.Capture;
import org.easymock.EasyMock;
import org.easymock.IMocksControl;
@@ -24,12 +25,7 @@
import org.powermock.api.easymock.PowerMock;
import org.powermock.modules.junit4.PowerMockRunner;
-import java.util.ArrayList;
-import java.util.Collection;
-
-/**
- * Test case with some support for automatically verifying mocks.
- */
+/** Test case with some support for automatically verifying mocks. */
public abstract class MockingTestCase extends TestCase {
private Collection<Object> mocks;
private Collection<IMocksControl> mockControls;
@@ -50,8 +46,9 @@
/**
* Create and register a mock.
*
- * Creates a mock and registers it in the list of created mocks, so it gets
- * treated automatically upon {@code replay} and {@code verify};
+ * <p>Creates a mock and registers it in the list of created mocks, so it gets treated
+ * automatically upon {@code replay} and {@code verify};
+ *
* @param toMock The class to create a mock for.
* @return The mock instance.
*/
@@ -62,11 +59,11 @@
/**
* Create a mock for a mock control and register a mock.
*
- * Creates a mock and registers it in the list of created mocks, so it gets
- * treated automatically upon {@code replay} and {@code verify};
+ * <p>Creates a mock and registers it in the list of created mocks, so it gets treated
+ * automatically upon {@code replay} and {@code verify};
+ *
* @param toMock The class to create a mock for.
- * @param control The mock control to create the mock on. If null, do not use
- * a specific control.
+ * @param control The mock control to create the mock on. If null, do not use a specific control.
* @return The mock instance.
*/
protected final <T> T createMock(Class<T> toMock, IMocksControl control) {
@@ -78,17 +75,14 @@
} else {
mock = EasyMock.createMock(toMock);
}
- assertTrue("Adding " + toMock.getName() + " mock failed",
- mocks.add(mock));
+ assertTrue("Adding " + toMock.getName() + " mock failed", mocks.add(mock));
} else {
mock = control.createMock(toMock);
}
return mock;
}
- /**
- * Set all registered mocks to replay
- */
+ /** Set all registered mocks to replay */
protected final void replayMocks() {
assertFalse("Mocks have already been set to replay", mocksReplayed);
if (usePowerMock) {
@@ -105,17 +99,17 @@
/**
* Verify all registered mocks
*
- * This method is called automatically at the end of a test. Nevertheless,
- * it is safe to also call it beforehand, if this better meets the
- * verification part of a test.
+ * <p>This method is called automatically at the end of a test. Nevertheless, it is safe to also
+ * call it beforehand, if this better meets the verification part of a test.
*/
// As the PowerMock runner does not pass through runTest, we inject mock
// verification through @After
@After
public final void verifyMocks() {
if (!mocks.isEmpty() || !mockControls.isEmpty()) {
- assertTrue("Created mocks have not been set to replay. Call replayMocks "
- + "within the test", mocksReplayed);
+ assertTrue(
+ "Created mocks have not been set to replay. Call replayMocks " + "within the test",
+ mocksReplayed);
if (usePowerMock) {
PowerMock.verifyAll();
} else {
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/CollectionAppender.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/CollectionAppender.java
index a533f8c..c122676 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/CollectionAppender.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/CollectionAppender.java
@@ -15,16 +15,12 @@
package com.googlesource.gerrit.plugins.its.base.testutil.log;
import com.google.common.collect.Lists;
-
+import java.util.Collection;
+import java.util.LinkedList;
import org.apache.log4j.AppenderSkeleton;
import org.apache.log4j.spi.LoggingEvent;
-import java.util.Collection;
-import java.util.LinkedList;
-
-/**
- * Log4j appender that logs into a list
- */
+/** Log4j appender that logs into a list */
public class CollectionAppender extends AppenderSkeleton {
private Collection<LoggingEvent> events;
@@ -43,14 +39,13 @@
@Override
protected void append(LoggingEvent event) {
- if (! events.add(event)) {
+ if (!events.add(event)) {
throw new RuntimeException("Could not append event " + event);
}
}
@Override
- public void close() {
- }
+ public void close() {}
public Collection<LoggingEvent> getLoggedEvents() {
return Lists.newLinkedList(events);
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/LogUtil.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/LogUtil.java
index c5a6cb3..6cd633d 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/LogUtil.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/testutil/log/LogUtil.java
@@ -14,15 +14,14 @@
package com.googlesource.gerrit.plugins.its.base.testutil.log;
+import java.util.Collection;
import org.apache.log4j.LogManager;
import org.apache.log4j.Logger;
import org.apache.log4j.spi.LoggingEvent;
-import java.util.Collection;
-
public class LogUtil {
- public static CollectionAppender logToCollection(String logName,
- Collection<LoggingEvent> collection) {
+ public static CollectionAppender logToCollection(
+ String logName, Collection<LoggingEvent> collection) {
Logger log = LogManager.getLogger(logName);
CollectionAppender listAppender = new CollectionAppender(collection);
log.removeAllAppenders();
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractorTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractorTest.java
index 3e8721d..80cbfd3 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractorTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/util/IssueExtractorTest.java
@@ -21,26 +21,22 @@
import com.google.gerrit.reviewdb.client.Change;
import com.google.gerrit.reviewdb.client.PatchSet;
import com.google.gerrit.reviewdb.client.RevId;
-
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsConfig;
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.util.IssueExtractor.PatchSetDb;
-
-import org.junit.runner.RunWith;
-import org.powermock.core.classloader.annotations.PrepareForTest;
-import org.powermock.modules.junit4.PowerMockRunner;
-
import java.util.Arrays;
import java.util.List;
import java.util.Map;
import java.util.Set;
import java.util.regex.Pattern;
+import org.junit.runner.RunWith;
+import org.powermock.core.classloader.annotations.PrepareForTest;
+import org.powermock.modules.junit4.PowerMockRunner;
@RunWith(PowerMockRunner.class)
-@PrepareForTest({PatchSet.class,RevId.class})
+@PrepareForTest({PatchSet.class, RevId.class})
public class IssueExtractorTest extends LoggingMockingTestCase {
private Injector injector;
private ItsConfig itsConfig;
@@ -50,25 +46,23 @@
public void testIssueIdsNullPattern() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(null)
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(null).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Test");
+ String[] ret = issueExtractor.getIssueIds("Test");
assertEquals("Number of found ids do not match", 0, ret.length);
}
public void testIssueIdsNoMatch() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Test");
+ String[] ret = issueExtractor.getIssueIds("Test");
assertEquals("Number of found ids do not match", 0, ret.length);
assertLogMessageContains("Matching");
@@ -77,13 +71,12 @@
public void testIssueIdsEmptyGroup() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(X*)(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(X*)(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("bug#4711");
+ String[] ret = issueExtractor.getIssueIds("bug#4711");
assertEquals("Number of found ids do not match", 0, ret.length);
assertLogMessageContains("Matching");
@@ -92,13 +85,12 @@
public void testIssueIdsFullMatch() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("bug#4711");
+ String[] ret = issueExtractor.getIssueIds("bug#4711");
assertEquals("Number of found ids do not match", 1, ret.length);
assertEquals("First found issue id do not match", "4711", ret[0]);
@@ -108,13 +100,12 @@
public void testIssueIdsMatch() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Foo bug#4711 bar");
+ String[] ret = issueExtractor.getIssueIds("Foo bug#4711 bar");
assertEquals("Number of found ids do not match", 1, ret.length);
assertEquals("Found issue id does not match", "4711", ret[0]);
@@ -124,13 +115,12 @@
public void testIssueIdsGrouplessMatch() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#\\d+"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#\\d+")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(0).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Foo bug#4711 bar");
+ String[] ret = issueExtractor.getIssueIds("Foo bug#4711 bar");
assertEquals("Number of found ids do not match", 1, ret.length);
assertEquals("Found issue id does not match", "bug#4711", ret[0]);
@@ -140,13 +130,12 @@
public void testIssueIdsMultiGroupMatchGroup1() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern())
- .andReturn(Pattern.compile("bug#(\\d)(\\d+)")).atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d)(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Foo bug#4711 bar");
+ String[] ret = issueExtractor.getIssueIds("Foo bug#4711 bar");
assertEquals("Number of found ids do not match", 1, ret.length);
assertEquals("Found issue id does not match", "4", ret[0]);
@@ -156,13 +145,12 @@
public void testIssueIdsMultiGroupMatchGroup2() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern())
- .andReturn(Pattern.compile("bug#(\\d)(\\d+)")).atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d)(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(2).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Foo bug#4711 bar");
+ String[] ret = issueExtractor.getIssueIds("Foo bug#4711 bar");
assertEquals("Number of found ids do not match", 1, ret.length);
assertEquals("Found issue id does not match", "711", ret[0]);
@@ -172,13 +160,12 @@
public void testIssueIdsMulipleMatches() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Foo bug#4711 bug#42 bar bug#123");
+ String[] ret = issueExtractor.getIssueIds("Foo bug#4711 bug#42 bar bug#123");
assertEquals("Number of found ids do not match", 3, ret.length);
List<String> retList = Arrays.asList(ret);
assertTrue("4711 not among the extracted ids", retList.contains("4711"));
@@ -191,14 +178,12 @@
public void testIssueIdsMulipleMatchesWithDuplicates() {
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
replayMocks();
- String ret[] = issueExtractor.getIssueIds("Foo bug#4711 bug#42 bar\n" +
- "bug#123 baz bug#42");
+ String[] ret = issueExtractor.getIssueIds("Foo bug#4711 bug#42 bar\n" + "bug#123 baz bug#42");
assertEquals("Number of found ids do not match", 3, ret.length);
List<String> retList = Arrays.asList(ret);
assertTrue("4711 not among the extracted ids", retList.contains("4711"));
@@ -209,23 +194,21 @@
}
public void testIssueIdsCommitSingleIssue() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "bug#42\n" +
- "\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("bug#42\n" + "\n" + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("42", Sets.newHashSet("somewhere", "subject"));
assertEquals("Extracted issues do not match", expected, actual);
@@ -237,23 +220,24 @@
}
public void testIssueIdsCommitMultipleIssues() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "bug#42, and bug#4711\n" +
- "\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "bug#42, and bug#4711\n"
+ + "\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("4711", Sets.newHashSet("somewhere", "subject"));
assertEquals("Extracted issues do not match", expected, actual);
@@ -266,23 +250,24 @@
}
public void testIssueIdsCommitMultipleIssuesMultipleTimes() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "bug#42, bug#4711, bug#4711, bug#42, and bug#4711\n" +
- "\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "bug#42, bug#4711, bug#4711, bug#42, and bug#4711\n"
+ + "\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("4711", Sets.newHashSet("somewhere", "subject"));
assertEquals("Extracted issues do not match", expected, actual);
@@ -295,25 +280,26 @@
}
public void testIssueIdsCommitSingleIssueBody() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject does not reference a bug\n" +
- "Body references bug#42\n" +
- "\n" +
- "Footer: does not reference a bug\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject does not reference a bug\n"
+ + "Body references bug#42\n"
+ + "\n"
+ + "Footer: does not reference a bug\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("42", Sets.newHashSet("somewhere", "body"));
assertEquals("Extracted issues do not match", expected, actual);
@@ -326,27 +312,27 @@
}
public void testIssueIdsCommitSingleIssueFooter() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject does not reference a bug\n" +
- "Body does not reference a bug\n" +
- "\n" +
- "Footer: references bug#42\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject does not reference a bug\n"
+ + "Body does not reference a bug\n"
+ + "\n"
+ + "Footer: references bug#42\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
- expected.put("42", Sets.newHashSet("somewhere", "footer",
- "footer-Footer"));
+ Map<String, Set<String>> expected = Maps.newHashMap();
+ expected.put("42", Sets.newHashSet("somewhere", "footer", "footer-Footer"));
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -358,29 +344,30 @@
}
public void testIssueIdsCommitMultipleIssuesFooter() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject does not reference a bug\n" +
- "Body does not reference a bug\n" +
- "\n" +
- "KeyA: references bug#42\n" +
- "KeyB: does not reference bug\n" +
- "KeyC: references bug#176\n" +
- "Unkeyed reference to bug#4711\n" +
- "Change-Id: I1234567891123456789212345678931234567894\n" +
- "KeyZ: references bug#256" );
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject does not reference a bug\n"
+ + "Body does not reference a bug\n"
+ + "\n"
+ + "KeyA: references bug#42\n"
+ + "KeyB: does not reference bug\n"
+ + "KeyC: references bug#176\n"
+ + "Unkeyed reference to bug#4711\n"
+ + "Change-Id: I1234567891123456789212345678931234567894\n"
+ + "KeyZ: references bug#256");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("42", Sets.newHashSet("somewhere", "footer", "footer-KeyA"));
expected.put("176", Sets.newHashSet("somewhere", "footer", "footer-KeyC"));
expected.put("256", Sets.newHashSet("somewhere", "footer", "footer-KeyZ"));
@@ -399,26 +386,27 @@
}
public void testIssueIdsCommitDifferentParts() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject references bug#42.\n" +
- "Body references bug#16.\n" +
- "Body also references bug#176.\n" +
- "\n" +
- "Bug: bug#4711 in footer\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject references bug#42.\n"
+ + "Body references bug#16.\n"
+ + "Body also references bug#176.\n"
+ + "\n"
+ + "Bug: bug#4711 in footer\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body"));
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("176", Sets.newHashSet("somewhere", "body"));
@@ -434,26 +422,27 @@
}
public void testIssueIdsCommitDifferentPartsEmptySubject() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "\n" +
- "Body references bug#16.\n" +
- "Body also references bug#176.\n" +
- "\n" +
- "Bug: bug#4711 in footer\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "\n"
+ + "Body references bug#16.\n"
+ + "Body also references bug#176.\n"
+ + "\n"
+ + "Bug: bug#4711 in footer\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body"));
expected.put("176", Sets.newHashSet("somewhere", "body"));
expected.put("4711", Sets.newHashSet("somewhere", "footer", "footer-Bug"));
@@ -468,26 +457,27 @@
}
public void testIssueIdsCommitDifferentPartsLinePastFooter() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject references bug#42.\n" +
- "Body references bug#16.\n" +
- "Body also references bug#176.\n" +
- "\n" +
- "Bug: bug#4711 in footer\n" +
- "Change-Id: I1234567891123456789212345678931234567894\n");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject references bug#42.\n"
+ + "Body references bug#16.\n"
+ + "Body also references bug#176.\n"
+ + "\n"
+ + "Bug: bug#4711 in footer\n"
+ + "Change-Id: I1234567891123456789212345678931234567894\n");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body"));
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("176", Sets.newHashSet("somewhere", "body"));
@@ -503,27 +493,28 @@
}
public void testIssueIdsCommitDifferentPartsLinesPastFooter() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject references bug#42.\n" +
- "Body references bug#16.\n" +
- "Body also references bug#176.\n" +
- "\n" +
- "Bug: bug#4711 in footer\n" +
- "Change-Id: I1234567891123456789212345678931234567894\n" +
- "\n");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject references bug#42.\n"
+ + "Body references bug#16.\n"
+ + "Body also references bug#176.\n"
+ + "\n"
+ + "Bug: bug#4711 in footer\n"
+ + "Change-Id: I1234567891123456789212345678931234567894\n"
+ + "\n");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body"));
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("176", Sets.newHashSet("somewhere", "body"));
@@ -539,23 +530,24 @@
}
public void testIssueIdsCommitDifferentPartsNoFooter() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject references bug#42.\n" +
- "Body references bug#16.\n" +
- "Body also references bug#176.");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject references bug#42.\n"
+ + "Body references bug#16.\n"
+ + "Body also references bug#176.");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body"));
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("176", Sets.newHashSet("somewhere", "body"));
@@ -567,23 +559,24 @@
}
public void testIssueIdsCommitDifferentPartsNoFooterTrailingLine() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject references bug#42.\n" +
- "Body references bug#16.\n" +
- "Body also references bug#176.\n");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject references bug#42.\n"
+ + "Body references bug#16.\n"
+ + "Body also references bug#176.\n");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body"));
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("176", Sets.newHashSet("somewhere", "body"));
@@ -595,24 +588,25 @@
}
public void testIssueIdsCommitDifferentPartsNoFooterTrailingLines() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject references bug#42.\n" +
- "Body references bug#16.\n" +
- "Body also references bug#176.\n" +
- "\n");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject references bug#42.\n"
+ + "Body references bug#16.\n"
+ + "Body also references bug#176.\n"
+ + "\n");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body"));
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("176", Sets.newHashSet("somewhere", "body"));
@@ -624,20 +618,21 @@
}
public void testIssueIdsCommitEmpty() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn("");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -646,96 +641,98 @@
}
public void testIssueIdsCommitBlankLine() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn("\n");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("\n");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
}
public void testIssueIdsCommitBlankLines() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn("\n\n");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("\n\n");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
}
public void testIssueIdsCommitMoreBlankLines() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn("\n\n\n");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("\n\n\n");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
}
public void testIssueIdsCommitMixed() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "Subject bug#42, bug#1984, and bug#16\n" +
- "\n" +
- "bug#4711 in body,\n" +
- "along with bug#1984, and bug#5150.\n" +
- "bug#4711 in body again, along with bug#16\n" +
- "\n" +
- "Bug: bug#176, bug#1984, and bug#5150\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "Subject bug#42, bug#1984, and bug#16\n"
+ + "\n"
+ + "bug#4711 in body,\n"
+ + "along with bug#1984, and bug#5150.\n"
+ + "bug#4711 in body again, along with bug#16\n"
+ + "\n"
+ + "Bug: bug#176, bug#1984, and bug#5150\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894");
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds("testProject", "1234567891123456789212345678931234567894");
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "subject", "body"));
expected.put("42", Sets.newHashSet("somewhere", "subject"));
expected.put("176", Sets.newHashSet("somewhere", "footer", "footer-Bug"));
- expected.put("1984", Sets.newHashSet("somewhere", "subject", "body",
- "footer", "footer-Bug"));
+ expected.put("1984", Sets.newHashSet("somewhere", "subject", "body", "footer", "footer-Bug"));
expected.put("4711", Sets.newHashSet("somewhere", "body"));
- expected.put("5150", Sets.newHashSet("somewhere", "body", "footer",
- "footer-Bug"));
+ expected.put("5150", Sets.newHashSet("somewhere", "body", "footer", "footer-Bug"));
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -747,21 +744,23 @@
}
public void testIssueIdsCommitWAddedEmptyFirst() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn("");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("");
replayMocks();
PatchSet.Id patchSetId = new PatchSet.Id(new Change.Id(4), 1);
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", patchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", patchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -770,32 +769,29 @@
}
public void testIssueIdsCommitWAddedSingleSubjectIssueFirst() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
Change.Id changeId = createMock(Change.Id.class);
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "bug#42\n" +
- "\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("bug#42\n" + "\n" + "Change-Id: I1234567891123456789212345678931234567894");
PatchSet.Id currentPatchSetId = createMock(PatchSet.Id.class);
expect(currentPatchSetId.get()).andReturn(1).anyTimes();
- expect(currentPatchSetId.getParentKey()).andReturn(changeId)
- .anyTimes();
+ expect(currentPatchSetId.getParentKey()).andReturn(changeId).anyTimes();
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", currentPatchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", currentPatchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
- expected.put("42", Sets.newHashSet("somewhere", "subject",
- "added@somewhere", "added@subject"));
+ Map<String, Set<String>> expected = Maps.newHashMap();
+ expected.put("42", Sets.newHashSet("somewhere", "subject", "added@somewhere", "added@subject"));
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -806,43 +802,40 @@
}
public void testIssueIdsCommitWAddedSingleSubjectIssueSecondEmpty() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
Change.Id changeId = createMock(Change.Id.class);
// Call for current patch set
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "bug#42\n" +
- "\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("bug#42\n" + "\n" + "Change-Id: I1234567891123456789212345678931234567894");
// Call for previous patch set
PatchSet.Id previousPatchSetId = new PatchSet.Id(changeId, 1);
- expect(db.getRevision(previousPatchSetId)).andReturn("9876543211987654321298765432139876543214");
+ expect(db.getRevision(previousPatchSetId))
+ .andReturn("9876543211987654321298765432139876543214");
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "9876543211987654321298765432139876543214")).andReturn(
- "subject\n" +
- "\n" +
- "Change-Id: I9876543211987654321298765432139876543214");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "9876543211987654321298765432139876543214"))
+ .andReturn("subject\n" + "\n" + "Change-Id: I9876543211987654321298765432139876543214");
PatchSet.Id currentPatchSetId = createMock(PatchSet.Id.class);
expect(currentPatchSetId.get()).andReturn(2).anyTimes();
- expect(currentPatchSetId.getParentKey()).andReturn(changeId)
- .anyTimes();
+ expect(currentPatchSetId.getParentKey()).andReturn(changeId).anyTimes();
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", currentPatchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", currentPatchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
- expected.put("42", Sets.newHashSet("somewhere", "subject",
- "added@somewhere", "added@subject"));
+ Map<String, Set<String>> expected = Maps.newHashMap();
+ expected.put("42", Sets.newHashSet("somewhere", "subject", "added@somewhere", "added@subject"));
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -858,41 +851,39 @@
}
public void testIssueIdsCommitWAddedSingleSubjectIssueSecondSame() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
Change.Id changeId = createMock(Change.Id.class);
// Call for current patch set
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "bug#42\n" +
- "\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("bug#42\n" + "\n" + "Change-Id: I1234567891123456789212345678931234567894");
// Call for previous patch set
PatchSet.Id previousPatchSetId = new PatchSet.Id(changeId, 1);
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "9876543211987654321298765432139876543214")).andReturn(
- "bug#42\n" +
- "\n" +
- "Change-Id: I9876543211987654321298765432139876543214");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "9876543211987654321298765432139876543214"))
+ .andReturn("bug#42\n" + "\n" + "Change-Id: I9876543211987654321298765432139876543214");
- expect(db.getRevision(previousPatchSetId)).andReturn("9876543211987654321298765432139876543214");
+ expect(db.getRevision(previousPatchSetId))
+ .andReturn("9876543211987654321298765432139876543214");
PatchSet.Id currentPatchSetId = createMock(PatchSet.Id.class);
expect(currentPatchSetId.get()).andReturn(2).anyTimes();
- expect(currentPatchSetId.getParentKey()).andReturn(changeId)
- .anyTimes();
+ expect(currentPatchSetId.getParentKey()).andReturn(changeId).anyTimes();
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", currentPatchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", currentPatchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("42", Sets.newHashSet("somewhere", "subject"));
assertEquals("Extracted issues do not match", expected, actual);
@@ -909,42 +900,43 @@
}
public void testIssueIdsCommitWAddedSingleSubjectIssueSecondBody() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
Change.Id changeId = createMock(Change.Id.class);
// Call for current patch set
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "bug#42\n" +
- "\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn("bug#42\n" + "\n" + "Change-Id: I1234567891123456789212345678931234567894");
// Call for previous patch set
PatchSet.Id previousPatchSetId = new PatchSet.Id(changeId, 1);
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "9876543211987654321298765432139876543214")).andReturn(
- "subject\n" +
- "bug#42\n" +
- "\n" +
- "Change-Id: I9876543211987654321298765432139876543214");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "9876543211987654321298765432139876543214"))
+ .andReturn(
+ "subject\n"
+ + "bug#42\n"
+ + "\n"
+ + "Change-Id: I9876543211987654321298765432139876543214");
- expect(db.getRevision(previousPatchSetId)).andReturn("9876543211987654321298765432139876543214");
+ expect(db.getRevision(previousPatchSetId))
+ .andReturn("9876543211987654321298765432139876543214");
PatchSet.Id currentPatchSetId = createMock(PatchSet.Id.class);
expect(currentPatchSetId.get()).andReturn(2).anyTimes();
- expect(currentPatchSetId.getParentKey()).andReturn(changeId)
- .anyTimes();
+ expect(currentPatchSetId.getParentKey()).andReturn(changeId).anyTimes();
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", currentPatchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", currentPatchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("42", Sets.newHashSet("somewhere", "subject", "added@subject"));
assertEquals("Extracted issues do not match", expected, actual);
@@ -961,44 +953,46 @@
}
public void testIssueIdsCommitWAddedSingleSubjectIssueSecondFooter() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
Change.Id changeId = createMock(Change.Id.class);
// Call for current patch set
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "subject\n" +
- "\n" +
- "Bug: bug#42\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "subject\n"
+ + "\n"
+ + "Bug: bug#42\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
// Call for previous patch set
PatchSet.Id previousPatchSetId = new PatchSet.Id(changeId, 1);
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "9876543211987654321298765432139876543214")).andReturn(
- "bug#42\n" +
- "\n" +
- "Change-Id: I9876543211987654321298765432139876543214");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "9876543211987654321298765432139876543214"))
+ .andReturn("bug#42\n" + "\n" + "Change-Id: I9876543211987654321298765432139876543214");
- expect(db.getRevision(previousPatchSetId)).andReturn("9876543211987654321298765432139876543214");
+ expect(db.getRevision(previousPatchSetId))
+ .andReturn("9876543211987654321298765432139876543214");
PatchSet.Id currentPatchSetId = createMock(PatchSet.Id.class);
expect(currentPatchSetId.get()).andReturn(2).anyTimes();
- expect(currentPatchSetId.getParentKey()).andReturn(changeId)
- .anyTimes();
+ expect(currentPatchSetId.getParentKey()).andReturn(changeId).anyTimes();
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", currentPatchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", currentPatchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
- expected.put("42", Sets.newHashSet("somewhere", "footer", "added@footer",
- "footer-Bug", "added@footer-Bug"));
+ Map<String, Set<String>> expected = Maps.newHashMap();
+ expected.put(
+ "42",
+ Sets.newHashSet("somewhere", "footer", "added@footer", "footer-Bug", "added@footer-Bug"));
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -1015,48 +1009,60 @@
}
public void testIssueIdsCommitWAddedSubjectFooter() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
Change.Id changeId = createMock(Change.Id.class);
// Call for current patch set
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "subject bug#42\n" +
- "\n" +
- "body bug#42\n" +
- "\n" +
- "Bug: bug#42\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "subject bug#42\n"
+ + "\n"
+ + "body bug#42\n"
+ + "\n"
+ + "Bug: bug#42\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
// Call for previous patch set
PatchSet.Id previousPatchSetId = new PatchSet.Id(changeId, 1);
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "9876543211987654321298765432139876543214")).andReturn(
- "subject\n" +
- "bug#42\n" +
- "\n" +
- "Change-Id: I9876543211987654321298765432139876543214");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "9876543211987654321298765432139876543214"))
+ .andReturn(
+ "subject\n"
+ + "bug#42\n"
+ + "\n"
+ + "Change-Id: I9876543211987654321298765432139876543214");
- expect(db.getRevision(previousPatchSetId)).andReturn("9876543211987654321298765432139876543214");
+ expect(db.getRevision(previousPatchSetId))
+ .andReturn("9876543211987654321298765432139876543214");
PatchSet.Id currentPatchSetId = createMock(PatchSet.Id.class);
expect(currentPatchSetId.get()).andReturn(2).anyTimes();
- expect(currentPatchSetId.getParentKey()).andReturn(changeId)
- .anyTimes();
+ expect(currentPatchSetId.getParentKey()).andReturn(changeId).anyTimes();
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", currentPatchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", currentPatchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
- expected.put("42", Sets.newHashSet("somewhere", "subject", "added@subject",
- "body", "footer", "added@footer", "footer-Bug",
- "added@footer-Bug"));
+ Map<String, Set<String>> expected = Maps.newHashMap();
+ expected.put(
+ "42",
+ Sets.newHashSet(
+ "somewhere",
+ "subject",
+ "added@subject",
+ "body",
+ "footer",
+ "added@footer",
+ "footer-Bug",
+ "added@footer-Bug"));
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -1073,50 +1079,62 @@
}
public void testIssueIdsCommitWAddedMultiple() {
- expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)"))
- .atLeastOnce();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).atLeastOnce();
expect(itsConfig.getIssuePatternGroupIndex()).andReturn(1).atLeastOnce();
Change.Id changeId = createMock(Change.Id.class);
// Call for current patch set
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "1234567891123456789212345678931234567894")).andReturn(
- "subject bug#42\n" +
- "\n" +
- "body bug#42 bug#16\n" +
- "\n" +
- "Bug: bug#42\n" +
- "Change-Id: I1234567891123456789212345678931234567894");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "1234567891123456789212345678931234567894"))
+ .andReturn(
+ "subject bug#42\n"
+ + "\n"
+ + "body bug#42 bug#16\n"
+ + "\n"
+ + "Bug: bug#42\n"
+ + "Change-Id: I1234567891123456789212345678931234567894");
// Call for previous patch set
PatchSet.Id previousPatchSetId = new PatchSet.Id(changeId, 1);
- expect(commitMessageFetcher.fetchGuarded("testProject",
- "9876543211987654321298765432139876543214")).andReturn(
- "subject\n" +
- "bug#42 bug#4711\n" +
- "\n" +
- "Bug: bug#16\n" +
- "Change-Id: I9876543211987654321298765432139876543214");
+ expect(
+ commitMessageFetcher.fetchGuarded(
+ "testProject", "9876543211987654321298765432139876543214"))
+ .andReturn(
+ "subject\n"
+ + "bug#42 bug#4711\n"
+ + "\n"
+ + "Bug: bug#16\n"
+ + "Change-Id: I9876543211987654321298765432139876543214");
- expect(db.getRevision(previousPatchSetId)).andReturn("9876543211987654321298765432139876543214");
+ expect(db.getRevision(previousPatchSetId))
+ .andReturn("9876543211987654321298765432139876543214");
PatchSet.Id currentPatchSetId = createMock(PatchSet.Id.class);
expect(currentPatchSetId.get()).andReturn(2).anyTimes();
- expect(currentPatchSetId.getParentKey()).andReturn(changeId)
- .anyTimes();
+ expect(currentPatchSetId.getParentKey()).andReturn(changeId).anyTimes();
replayMocks();
IssueExtractor issueExtractor = injector.getInstance(IssueExtractor.class);
- Map<String,Set<String>> actual = issueExtractor.getIssueIds("testProject",
- "1234567891123456789212345678931234567894", currentPatchSetId);
+ Map<String, Set<String>> actual =
+ issueExtractor.getIssueIds(
+ "testProject", "1234567891123456789212345678931234567894", currentPatchSetId);
- Map<String,Set<String>> expected = Maps.newHashMap();
+ Map<String, Set<String>> expected = Maps.newHashMap();
expected.put("16", Sets.newHashSet("somewhere", "body", "added@body"));
- expected.put("42", Sets.newHashSet("somewhere", "subject", "added@subject",
- "body", "footer", "added@footer", "footer-Bug",
- "added@footer-Bug"));
+ expected.put(
+ "42",
+ Sets.newHashSet(
+ "somewhere",
+ "subject",
+ "added@subject",
+ "body",
+ "footer",
+ "added@footer",
+ "footer-Bug",
+ "added@footer-Bug"));
assertEquals("Extracted issues do not match", expected, actual);
assertLogMessageContains("Matching");
@@ -1153,4 +1171,4 @@
bind(PatchSetDb.class).toInstance(db);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractorTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractorTest.java
index 41f8921..756c1a1 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractorTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyAttributeExtractorTest.java
@@ -26,11 +26,9 @@
import com.google.gerrit.server.data.RefUpdateAttribute;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
import java.util.Set;
public class PropertyAttributeExtractorTest extends LoggingMockingTestCase {
@@ -42,8 +40,7 @@
public void testAccountAttributeNull() {
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(null, "prefix");
@@ -60,34 +57,27 @@
// deprecated, to be removed soon. migrate to ones without dash.
Property propertyEmail2 = createMock(Property.class);
- expect(propertyFactory.create("prefix-email", "testEmail"))
- .andReturn(propertyEmail2);
+ expect(propertyFactory.create("prefix-email", "testEmail")).andReturn(propertyEmail2);
Property propertyName2 = createMock(Property.class);
- expect(propertyFactory.create("prefix-name", "testName"))
- .andReturn(propertyName2);
+ expect(propertyFactory.create("prefix-name", "testName")).andReturn(propertyName2);
Property propertyUsername2 = createMock(Property.class);
- expect(propertyFactory.create("prefix-username", "testUsername"))
- .andReturn(propertyUsername2);
+ expect(propertyFactory.create("prefix-username", "testUsername")).andReturn(propertyUsername2);
// New style configs for vm and soy
- Property propertyEmail= createMock(Property.class);
- expect(propertyFactory.create("prefixEmail", "testEmail"))
- .andReturn(propertyEmail);
+ Property propertyEmail = createMock(Property.class);
+ expect(propertyFactory.create("prefixEmail", "testEmail")).andReturn(propertyEmail);
Property propertyName = createMock(Property.class);
- expect(propertyFactory.create("prefixName", "testName"))
- .andReturn(propertyName);
+ expect(propertyFactory.create("prefixName", "testName")).andReturn(propertyName);
Property propertyUsername = createMock(Property.class);
- expect(propertyFactory.create("prefixUsername", "testUsername"))
- .andReturn(propertyUsername);
+ expect(propertyFactory.create("prefixUsername", "testUsername")).andReturn(propertyUsername);
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(accountAttribute, "prefix");
@@ -119,36 +109,28 @@
changeAttribute.commitMessage = "Commit Message";
Property propertyProject = createMock(Property.class);
- expect(propertyFactory.create("project", "testProject"))
- .andReturn(propertyProject);
+ expect(propertyFactory.create("project", "testProject")).andReturn(propertyProject);
Property propertyBranch = createMock(Property.class);
- expect(propertyFactory.create("branch", "testBranch"))
- .andReturn(propertyBranch);
+ expect(propertyFactory.create("branch", "testBranch")).andReturn(propertyBranch);
Property propertyTopic = createMock(Property.class);
- expect(propertyFactory.create("topic", "testTopic"))
- .andReturn(propertyTopic);
+ expect(propertyFactory.create("topic", "testTopic")).andReturn(propertyTopic);
Property propertySubject = createMock(Property.class);
- expect(propertyFactory.create("subject", "testSubject"))
- .andReturn(propertySubject);
+ expect(propertyFactory.create("subject", "testSubject")).andReturn(propertySubject);
Property propertyId2 = createMock(Property.class);
- expect(propertyFactory.create("change-id", "testId"))
- .andReturn(propertyId2);
+ expect(propertyFactory.create("change-id", "testId")).andReturn(propertyId2);
Property propertyId = createMock(Property.class);
- expect(propertyFactory.create("changeId", "testId"))
- .andReturn(propertyId);
+ expect(propertyFactory.create("changeId", "testId")).andReturn(propertyId);
Property propertyNumber2 = createMock(Property.class);
- expect(propertyFactory.create("change-number", "4711"))
- .andReturn(propertyNumber2);
+ expect(propertyFactory.create("change-number", "4711")).andReturn(propertyNumber2);
Property propertyNumber = createMock(Property.class);
- expect(propertyFactory.create("changeNumber", "4711"))
- .andReturn(propertyNumber);
+ expect(propertyFactory.create("changeNumber", "4711")).andReturn(propertyNumber);
Property propertyUrl2 = createMock(Property.class);
expect(propertyFactory.create("change-url", "http://www.example.org/test"))
@@ -159,36 +141,29 @@
.andReturn(propertyUrl);
Property propertyStatus = createMock(Property.class);
- expect(propertyFactory.create("status", null))
- .andReturn(propertyStatus);
+ expect(propertyFactory.create("status", null)).andReturn(propertyStatus);
Property propertyEmail = createMock(Property.class);
- expect(propertyFactory.create("ownerEmail", "testEmail"))
- .andReturn(propertyEmail);
+ expect(propertyFactory.create("ownerEmail", "testEmail")).andReturn(propertyEmail);
Property propertyName = createMock(Property.class);
- expect(propertyFactory.create("ownerName", "testName"))
- .andReturn(propertyName);
+ expect(propertyFactory.create("ownerName", "testName")).andReturn(propertyName);
Property propertyUsername = createMock(Property.class);
- expect(propertyFactory.create("ownerUsername", "testUsername"))
- .andReturn(propertyUsername);
+ expect(propertyFactory.create("ownerUsername", "testUsername")).andReturn(propertyUsername);
Property propertyCommitMessage = createMock(Property.class);
expect(propertyFactory.create("commitMessage", "Commit Message"))
.andReturn(propertyCommitMessage);
Property propertyEmail2 = createMock(Property.class);
- expect(propertyFactory.create("owner-email", "testEmail"))
- .andReturn(propertyEmail2);
+ expect(propertyFactory.create("owner-email", "testEmail")).andReturn(propertyEmail2);
Property propertyName2 = createMock(Property.class);
- expect(propertyFactory.create("owner-name", "testName"))
- .andReturn(propertyName2);
+ expect(propertyFactory.create("owner-name", "testName")).andReturn(propertyName2);
Property propertyUsername2 = createMock(Property.class);
- expect(propertyFactory.create("owner-username", "testUsername"))
- .andReturn(propertyUsername2);
+ expect(propertyFactory.create("owner-username", "testUsername")).andReturn(propertyUsername2);
Property propertyCommitMessage2 = createMock(Property.class);
expect(propertyFactory.create("commit-message", "Commit Message"))
@@ -203,8 +178,7 @@
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(changeAttribute);
@@ -251,76 +225,61 @@
changeAttribute.commitMessage = "Commit Message";
Property propertyProject = createMock(Property.class);
- expect(propertyFactory.create("project", "testProject"))
- .andReturn(propertyProject);
+ expect(propertyFactory.create("project", "testProject")).andReturn(propertyProject);
Property propertyBranch = createMock(Property.class);
- expect(propertyFactory.create("branch", "testBranch"))
- .andReturn(propertyBranch);
+ expect(propertyFactory.create("branch", "testBranch")).andReturn(propertyBranch);
Property propertyTopic = createMock(Property.class);
- expect(propertyFactory.create("topic", "testTopic"))
- .andReturn(propertyTopic);
+ expect(propertyFactory.create("topic", "testTopic")).andReturn(propertyTopic);
Property propertySubject = createMock(Property.class);
- expect(propertyFactory.create("subject", "testSubject"))
- .andReturn(propertySubject);
+ expect(propertyFactory.create("subject", "testSubject")).andReturn(propertySubject);
Property propertyId = createMock(Property.class);
- expect(propertyFactory.create("changeId", "testId"))
- .andReturn(propertyId);
+ expect(propertyFactory.create("changeId", "testId")).andReturn(propertyId);
Property propertyNumber = createMock(Property.class);
- expect(propertyFactory.create("changeNumber", "4711"))
- .andReturn(propertyNumber);
+ expect(propertyFactory.create("changeNumber", "4711")).andReturn(propertyNumber);
Property propertyUrl = createMock(Property.class);
expect(propertyFactory.create("changeUrl", "http://www.example.org/test"))
.andReturn(propertyUrl);
Property propertyId2 = createMock(Property.class);
- expect(propertyFactory.create("change-id", "testId"))
- .andReturn(propertyId2);
+ expect(propertyFactory.create("change-id", "testId")).andReturn(propertyId2);
Property propertyNumber2 = createMock(Property.class);
- expect(propertyFactory.create("change-number", "4711"))
- .andReturn(propertyNumber2);
+ expect(propertyFactory.create("change-number", "4711")).andReturn(propertyNumber2);
Property propertyUrl2 = createMock(Property.class);
expect(propertyFactory.create("change-url", "http://www.example.org/test"))
.andReturn(propertyUrl2);
Property propertyStatus = createMock(Property.class);
- expect(propertyFactory.create("status", "ABANDONED"))
- .andReturn(propertyStatus);
+ expect(propertyFactory.create("status", "ABANDONED")).andReturn(propertyStatus);
- Property propertyEmail= createMock(Property.class);
- expect(propertyFactory.create("ownerEmail", "testEmail"))
- .andReturn(propertyEmail);
+ Property propertyEmail = createMock(Property.class);
+ expect(propertyFactory.create("ownerEmail", "testEmail")).andReturn(propertyEmail);
Property propertyName = createMock(Property.class);
- expect(propertyFactory.create("ownerName", "testName"))
- .andReturn(propertyName);
+ expect(propertyFactory.create("ownerName", "testName")).andReturn(propertyName);
Property propertyUsername = createMock(Property.class);
- expect(propertyFactory.create("ownerUsername", "testUsername"))
- .andReturn(propertyUsername);
+ expect(propertyFactory.create("ownerUsername", "testUsername")).andReturn(propertyUsername);
Property propertyCommitMessage = createMock(Property.class);
expect(propertyFactory.create("commitMessage", "Commit Message"))
.andReturn(propertyCommitMessage);
- Property propertyEmail2= createMock(Property.class);
- expect(propertyFactory.create("owner-email", "testEmail"))
- .andReturn(propertyEmail2);
+ Property propertyEmail2 = createMock(Property.class);
+ expect(propertyFactory.create("owner-email", "testEmail")).andReturn(propertyEmail2);
Property propertyName2 = createMock(Property.class);
- expect(propertyFactory.create("owner-name", "testName"))
- .andReturn(propertyName2);
+ expect(propertyFactory.create("owner-name", "testName")).andReturn(propertyName2);
Property propertyUsername2 = createMock(Property.class);
- expect(propertyFactory.create("owner-username", "testUsername"))
- .andReturn(propertyUsername2);
+ expect(propertyFactory.create("owner-username", "testUsername")).andReturn(propertyUsername2);
Property propertyCommitMessage2 = createMock(Property.class);
expect(propertyFactory.create("commit-message", "Commit Message"))
@@ -333,11 +292,9 @@
expect(facade.createLinkForWebui("http://www.example.org/test", "http://www.example.org/test"))
.andReturn("http://www.example.org/test");
-
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(changeAttribute);
@@ -388,49 +345,38 @@
patchSetAttribute.author = author;
Property propertyRevision = createMock(Property.class);
- expect(propertyFactory.create("revision",
- "1234567891123456789212345678931234567894"))
+ expect(propertyFactory.create("revision", "1234567891123456789212345678931234567894"))
.andReturn(propertyRevision);
Property propertyNumber = createMock(Property.class);
- expect(propertyFactory.create("patchSetNumber", "42"))
- .andReturn(propertyNumber);
+ expect(propertyFactory.create("patchSetNumber", "42")).andReturn(propertyNumber);
Property propertyNumber2 = createMock(Property.class);
- expect(propertyFactory.create("patch-set-number", "42"))
- .andReturn(propertyNumber2);
+ expect(propertyFactory.create("patch-set-number", "42")).andReturn(propertyNumber2);
Property propertyRef = createMock(Property.class);
- expect(propertyFactory.create("ref", "testRef"))
- .andReturn(propertyRef);
+ expect(propertyFactory.create("ref", "testRef")).andReturn(propertyRef);
Property propertyCreatedOn = createMock(Property.class);
- expect(propertyFactory.create("createdOn", "1234567890"))
- .andReturn(propertyCreatedOn);
+ expect(propertyFactory.create("createdOn", "1234567890")).andReturn(propertyCreatedOn);
Property propertyCreatedOn2 = createMock(Property.class);
- expect(propertyFactory.create("created-on", "1234567890"))
- .andReturn(propertyCreatedOn2);
+ expect(propertyFactory.create("created-on", "1234567890")).andReturn(propertyCreatedOn2);
Property propertyParents = createMock(Property.class);
- expect(propertyFactory.create("parents", "[parent1, parent2]"))
- .andReturn(propertyParents);
+ expect(propertyFactory.create("parents", "[parent1, parent2]")).andReturn(propertyParents);
Property propertyDeletions = createMock(Property.class);
- expect(propertyFactory.create("deletions", "7"))
- .andReturn(propertyDeletions);
+ expect(propertyFactory.create("deletions", "7")).andReturn(propertyDeletions);
Property propertyInsertions = createMock(Property.class);
- expect(propertyFactory.create("insertions", "12"))
- .andReturn(propertyInsertions);
+ expect(propertyFactory.create("insertions", "12")).andReturn(propertyInsertions);
Property propertyUploaderEmail = createMock(Property.class);
- expect(propertyFactory.create("uploaderEmail", "testEmail1"))
- .andReturn(propertyUploaderEmail);
+ expect(propertyFactory.create("uploaderEmail", "testEmail1")).andReturn(propertyUploaderEmail);
Property propertyUploaderName = createMock(Property.class);
- expect(propertyFactory.create("uploaderName", "testName1"))
- .andReturn(propertyUploaderName);
+ expect(propertyFactory.create("uploaderName", "testName1")).andReturn(propertyUploaderName);
Property propertyUploaderUsername = createMock(Property.class);
expect(propertyFactory.create("uploaderUsername", "testUsername1"))
@@ -441,42 +387,35 @@
.andReturn(propertyUploaderEmail2);
Property propertyUploaderName2 = createMock(Property.class);
- expect(propertyFactory.create("uploader-name", "testName1"))
- .andReturn(propertyUploaderName2);
+ expect(propertyFactory.create("uploader-name", "testName1")).andReturn(propertyUploaderName2);
Property propertyUploaderUsername2 = createMock(Property.class);
expect(propertyFactory.create("uploader-username", "testUsername1"))
.andReturn(propertyUploaderUsername2);
Property propertyAuthorEmail = createMock(Property.class);
- expect(propertyFactory.create("authorEmail", "testEmail2"))
- .andReturn(propertyAuthorEmail);
+ expect(propertyFactory.create("authorEmail", "testEmail2")).andReturn(propertyAuthorEmail);
Property propertyAuthorName = createMock(Property.class);
- expect(propertyFactory.create("authorName", "testName2"))
- .andReturn(propertyAuthorName);
+ expect(propertyFactory.create("authorName", "testName2")).andReturn(propertyAuthorName);
Property propertyAuthorUsername = createMock(Property.class);
expect(propertyFactory.create("authorUsername", "testUsername2"))
.andReturn(propertyAuthorUsername);
Property propertyAuthorEmail2 = createMock(Property.class);
- expect(propertyFactory.create("author-email", "testEmail2"))
- .andReturn(propertyAuthorEmail2);
+ expect(propertyFactory.create("author-email", "testEmail2")).andReturn(propertyAuthorEmail2);
Property propertyAuthorName2 = createMock(Property.class);
- expect(propertyFactory.create("author-name", "testName2"))
- .andReturn(propertyAuthorName2);
+ expect(propertyFactory.create("author-name", "testName2")).andReturn(propertyAuthorName2);
Property propertyAuthorUsername2 = createMock(Property.class);
expect(propertyFactory.create("author-username", "testUsername2"))
.andReturn(propertyAuthorUsername2);
-
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(patchSetAttribute);
@@ -513,32 +452,26 @@
refUpdateAttribute.refName = "testRef";
Property propertyRevision = createMock(Property.class);
- expect(propertyFactory.create("revision",
- "1234567891123456789212345678931234567894"))
+ expect(propertyFactory.create("revision", "1234567891123456789212345678931234567894"))
.andReturn(propertyRevision);
Property propertyRevisionOld = createMock(Property.class);
- expect(propertyFactory.create("revisionOld",
- "9876543211987654321298765432139876543214"))
+ expect(propertyFactory.create("revisionOld", "9876543211987654321298765432139876543214"))
.andReturn(propertyRevisionOld);
Property propertyRevisionOld2 = createMock(Property.class);
- expect(propertyFactory.create("revision-old",
- "9876543211987654321298765432139876543214"))
+ expect(propertyFactory.create("revision-old", "9876543211987654321298765432139876543214"))
.andReturn(propertyRevisionOld2);
Property propertyProject = createMock(Property.class);
- expect(propertyFactory.create("project", "testProject"))
- .andReturn(propertyProject);
+ expect(propertyFactory.create("project", "testProject")).andReturn(propertyProject);
Property propertyRef = createMock(Property.class);
- expect(propertyFactory.create("ref", "testRef"))
- .andReturn(propertyRef);
+ expect(propertyFactory.create("ref", "testRef")).andReturn(propertyRef);
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(refUpdateAttribute);
@@ -557,18 +490,14 @@
approvalAttribute.value = "TestValue";
Property propertyApproval = createMock(Property.class);
- expect(propertyFactory.create("approvalTestType", "TestValue"))
- .andReturn(propertyApproval);
+ expect(propertyFactory.create("approvalTestType", "TestValue")).andReturn(propertyApproval);
Property propertyApproval2 = createMock(Property.class);
- expect(propertyFactory.create("approval-TestType", "TestValue"))
- .andReturn(propertyApproval2);
-
+ expect(propertyFactory.create("approval-TestType", "TestValue")).andReturn(propertyApproval2);
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(approvalAttribute);
@@ -584,18 +513,14 @@
approvalAttribute.value = "TestValue";
Property propertyApproval = createMock(Property.class);
- expect(propertyFactory.create("approvalTestType", "TestValue"))
- .andReturn(propertyApproval);
+ expect(propertyFactory.create("approvalTestType", "TestValue")).andReturn(propertyApproval);
Property propertyApproval2 = createMock(Property.class);
- expect(propertyFactory.create("approval-Test-Type", "TestValue"))
- .andReturn(propertyApproval2);
-
+ expect(propertyFactory.create("approval-Test-Type", "TestValue")).andReturn(propertyApproval2);
replayMocks();
- PropertyAttributeExtractor extractor =
- injector.getInstance(PropertyAttributeExtractor.class);
+ PropertyAttributeExtractor extractor = injector.getInstance(PropertyAttributeExtractor.class);
Set<Property> actual = extractor.extractFrom(approvalAttribute);
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractorTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractorTest.java
index eeffe13..981eeae 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractorTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/util/PropertyExtractorTest.java
@@ -36,10 +36,8 @@
import com.google.gerrit.server.events.RefUpdatedEvent;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
import java.util.HashMap;
import java.util.Set;
@@ -51,26 +49,26 @@
private PropertyAttributeExtractor propertyAttributeExtractor;
public void testDummyChangeEvent() {
- PropertyExtractor propertyExtractor = injector.getInstance(
- PropertyExtractor.class);
+ PropertyExtractor propertyExtractor = injector.getInstance(PropertyExtractor.class);
Property property1 = createMock(Property.class);
- expect(propertyFactory.create("event", "com.googlesource.gerrit.plugins." +
- "its.base.util.PropertyExtractorTest$DummyEvent"))
+ expect(
+ propertyFactory.create(
+ "event",
+ "com.googlesource.gerrit.plugins."
+ + "its.base.util.PropertyExtractorTest$DummyEvent"))
.andReturn(property1);
replayMocks();
- Set<Set<Property>> actual = propertyExtractor.extractFrom(
- new DummyEvent());
+ Set<Set<Property>> actual = propertyExtractor.extractFrom(new DummyEvent());
Set<Set<Property>> expected = Sets.newHashSet();
assertEquals("Properties do not match", expected, actual);
}
public void testChangeAbandonedEvent() {
- ChangeAbandonedEvent event =
- new ChangeAbandonedEvent(testChange("testProject", "testBranch"));
+ ChangeAbandonedEvent event = new ChangeAbandonedEvent(testChange("testProject", "testBranch"));
ChangeAttribute changeAttribute = createMock(ChangeAttribute.class);
event.change = Suppliers.ofInstance(changeAttribute);
@@ -79,10 +77,10 @@
.andReturn(Sets.newHashSet(propertyChange));
AccountAttribute accountAttribute = createMock(AccountAttribute.class);
- event.abandoner= Suppliers.ofInstance(accountAttribute);
+ event.abandoner = Suppliers.ofInstance(accountAttribute);
Property propertySubmitter = createMock(Property.class);
- expect(propertyAttributeExtractor.extractFrom(accountAttribute,
- "abandoner")).andReturn(Sets.newHashSet(propertySubmitter));
+ expect(propertyAttributeExtractor.extractFrom(accountAttribute, "abandoner"))
+ .andReturn(Sets.newHashSet(propertySubmitter));
PatchSetAttribute patchSetAttribute = createMock(PatchSetAttribute.class);
event.patchSet = Suppliers.ofInstance(patchSetAttribute);
@@ -92,8 +90,7 @@
event.reason = "testReason";
Property propertyReason = createMock(Property.class);
- expect(propertyFactory.create("reason", "testReason"))
- .andReturn(propertyReason);
+ expect(propertyFactory.create("reason", "testReason")).andReturn(propertyReason);
changeAttribute.project = "testProject";
changeAttribute.number = 176;
@@ -106,13 +103,11 @@
common.add(propertyPatchSet);
common.add(propertyReason);
- eventHelper(event, "ChangeAbandonedEvent", "change-abandoned", common,
- true);
+ eventHelper(event, "ChangeAbandonedEvent", "change-abandoned", common, true);
}
public void testChangeMergedEvent() {
- ChangeMergedEvent event =
- new ChangeMergedEvent(testChange("testProject", "testBranch"));
+ ChangeMergedEvent event = new ChangeMergedEvent(testChange("testProject", "testBranch"));
ChangeAttribute changeAttribute = createMock(ChangeAttribute.class);
event.change = Suppliers.ofInstance(changeAttribute);
@@ -123,8 +118,8 @@
AccountAttribute accountAttribute = createMock(AccountAttribute.class);
event.submitter = Suppliers.ofInstance(accountAttribute);
Property propertySubmitter = createMock(Property.class);
- expect(propertyAttributeExtractor.extractFrom(accountAttribute,
- "submitter")).andReturn(Sets.newHashSet(propertySubmitter));
+ expect(propertyAttributeExtractor.extractFrom(accountAttribute, "submitter"))
+ .andReturn(Sets.newHashSet(propertySubmitter));
PatchSetAttribute patchSetAttribute = createMock(PatchSetAttribute.class);
event.patchSet = Suppliers.ofInstance(patchSetAttribute);
@@ -146,8 +141,7 @@
}
public void testChangeRestoredEvent() {
- ChangeRestoredEvent event =
- new ChangeRestoredEvent(testChange("testProject", "testBranch"));
+ ChangeRestoredEvent event = new ChangeRestoredEvent(testChange("testProject", "testBranch"));
ChangeAttribute changeAttribute = createMock(ChangeAttribute.class);
event.change = Suppliers.ofInstance(changeAttribute);
@@ -158,8 +152,8 @@
AccountAttribute accountAttribute = createMock(AccountAttribute.class);
event.restorer = Suppliers.ofInstance(accountAttribute);
Property propertySubmitter = createMock(Property.class);
- expect(propertyAttributeExtractor.extractFrom(accountAttribute,
- "restorer")).andReturn(Sets.newHashSet(propertySubmitter));
+ expect(propertyAttributeExtractor.extractFrom(accountAttribute, "restorer"))
+ .andReturn(Sets.newHashSet(propertySubmitter));
PatchSetAttribute patchSetAttribute = createMock(PatchSetAttribute.class);
event.patchSet = Suppliers.ofInstance(patchSetAttribute);
@@ -169,8 +163,7 @@
event.reason = "testReason";
Property propertyReason = createMock(Property.class);
- expect(propertyFactory.create("reason", "testReason"))
- .andReturn(propertyReason);
+ expect(propertyFactory.create("reason", "testReason")).andReturn(propertyReason);
changeAttribute.project = "testProject";
changeAttribute.number = 176;
@@ -187,8 +180,7 @@
}
public void testCommentAddedEventWOApprovals() {
- CommentAddedEvent event =
- new CommentAddedEvent(testChange("testProject", "testBranch"));
+ CommentAddedEvent event = new CommentAddedEvent(testChange("testProject", "testBranch"));
ChangeAttribute changeAttribute = createMock(ChangeAttribute.class);
event.change = Suppliers.ofInstance(changeAttribute);
@@ -199,8 +191,8 @@
AccountAttribute accountAttribute = createMock(AccountAttribute.class);
event.author = Suppliers.ofInstance(accountAttribute);
Property propertySubmitter = createMock(Property.class);
- expect(propertyAttributeExtractor.extractFrom(accountAttribute,
- "commenter")).andReturn(Sets.newHashSet(propertySubmitter));
+ expect(propertyAttributeExtractor.extractFrom(accountAttribute, "commenter"))
+ .andReturn(Sets.newHashSet(propertySubmitter));
PatchSetAttribute patchSetAttribute = createMock(PatchSetAttribute.class);
event.patchSet = Suppliers.ofInstance(patchSetAttribute);
@@ -210,8 +202,7 @@
event.comment = "testComment";
Property propertyComment = createMock(Property.class);
- expect(propertyFactory.create("comment", "testComment"))
- .andReturn(propertyComment);
+ expect(propertyFactory.create("comment", "testComment")).andReturn(propertyComment);
changeAttribute.project = "testProject";
changeAttribute.number = 176;
@@ -228,8 +219,7 @@
}
public void testCommentAddedEventWApprovals() {
- CommentAddedEvent event =
- new CommentAddedEvent(testChange("testProject", "testBranch"));
+ CommentAddedEvent event = new CommentAddedEvent(testChange("testProject", "testBranch"));
ChangeAttribute changeAttribute = createMock(ChangeAttribute.class);
event.change = Suppliers.ofInstance(changeAttribute);
@@ -240,8 +230,8 @@
AccountAttribute accountAttribute = createMock(AccountAttribute.class);
event.author = Suppliers.ofInstance(accountAttribute);
Property propertySubmitter = createMock(Property.class);
- expect(propertyAttributeExtractor.extractFrom(accountAttribute,
- "commenter")).andReturn(Sets.newHashSet(propertySubmitter));
+ expect(propertyAttributeExtractor.extractFrom(accountAttribute, "commenter"))
+ .andReturn(Sets.newHashSet(propertySubmitter));
PatchSetAttribute patchSetAttribute = createMock(PatchSetAttribute.class);
event.patchSet = Suppliers.ofInstance(patchSetAttribute);
@@ -257,14 +247,12 @@
Property propertyApproval2 = createMock(Property.class);
expect(propertyAttributeExtractor.extractFrom(approvalAttribute2))
.andReturn(Sets.newHashSet(propertyApproval2));
- ApprovalAttribute approvalAttributes[] = { approvalAttribute1,
- approvalAttribute2 };
+ ApprovalAttribute[] approvalAttributes = {approvalAttribute1, approvalAttribute2};
event.approvals = Suppliers.ofInstance(approvalAttributes);
event.comment = "testComment";
Property propertyComment = createMock(Property.class);
- expect(propertyFactory.create("comment", "testComment"))
- .andReturn(propertyComment);
+ expect(propertyFactory.create("comment", "testComment")).andReturn(propertyComment);
changeAttribute.project = "testProject";
changeAttribute.number = 176;
@@ -283,8 +271,7 @@
}
public void testPatchSetCreatedEvent() {
- PatchSetCreatedEvent event =
- new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
+ PatchSetCreatedEvent event = new PatchSetCreatedEvent(testChange("testProject", "testBranch"));
ChangeAttribute changeAttribute = createMock(ChangeAttribute.class);
event.change = Suppliers.ofInstance(changeAttribute);
@@ -295,8 +282,8 @@
AccountAttribute accountAttribute = createMock(AccountAttribute.class);
event.uploader = Suppliers.ofInstance(accountAttribute);
Property propertySubmitter = createMock(Property.class);
- expect(propertyAttributeExtractor.extractFrom(accountAttribute,
- "uploader")).andReturn(Sets.newHashSet(propertySubmitter));
+ expect(propertyAttributeExtractor.extractFrom(accountAttribute, "uploader"))
+ .andReturn(Sets.newHashSet(propertySubmitter));
PatchSetAttribute patchSetAttribute = createMock(PatchSetAttribute.class);
event.patchSet = Suppliers.ofInstance(patchSetAttribute);
@@ -314,8 +301,7 @@
common.add(propertySubmitter);
common.add(propertyPatchSet);
- eventHelper(event, "PatchSetCreatedEvent", "patchset-created", common,
- true);
+ eventHelper(event, "PatchSetCreatedEvent", "patchset-created", common, true);
}
public void testRefUpdatedEvent() {
@@ -324,11 +310,10 @@
AccountAttribute accountAttribute = createMock(AccountAttribute.class);
event.submitter = Suppliers.ofInstance(accountAttribute);
Property propertySubmitter = createMock(Property.class);
- expect(propertyAttributeExtractor.extractFrom(accountAttribute,
- "submitter")).andReturn(Sets.newHashSet(propertySubmitter));
+ expect(propertyAttributeExtractor.extractFrom(accountAttribute, "submitter"))
+ .andReturn(Sets.newHashSet(propertySubmitter));
- RefUpdateAttribute refUpdateAttribute =
- createMock(RefUpdateAttribute.class);
+ RefUpdateAttribute refUpdateAttribute = createMock(RefUpdateAttribute.class);
event.refUpdate = Suppliers.ofInstance(refUpdateAttribute);
Property propertyRefUpdated = createMock(Property.class);
expect(propertyAttributeExtractor.extractFrom(refUpdateAttribute))
@@ -344,53 +329,46 @@
eventHelper(event, "RefUpdatedEvent", "ref-updated", common, false);
}
- private void eventHelper(Event event, String className, String type,
- Set<Property> common, boolean withRevision) {
- PropertyExtractor propertyExtractor = injector.getInstance(
- PropertyExtractor.class);
+ private void eventHelper(
+ Event event, String className, String type, Set<Property> common, boolean withRevision) {
+ PropertyExtractor propertyExtractor = injector.getInstance(PropertyExtractor.class);
Property propertyItsName = createMock(Property.class);
- expect(propertyFactory.create("its-name", "ItsTestName"))
- .andReturn(propertyItsName).anyTimes();
+ expect(propertyFactory.create("its-name", "ItsTestName")).andReturn(propertyItsName).anyTimes();
Property propertyEvent = createMock(Property.class);
- expect(propertyFactory.create("event", "com.google.gerrit.server.events." +
- className)).andReturn(propertyEvent);
+ expect(propertyFactory.create("event", "com.google.gerrit.server.events." + className))
+ .andReturn(propertyEvent);
Property propertyEventType = createMock(Property.class);
- expect(propertyFactory.create("event-type", type))
- .andReturn(propertyEventType);
+ expect(propertyFactory.create("event-type", type)).andReturn(propertyEventType);
Property propertyAssociationFooter = createMock(Property.class);
- expect(propertyFactory.create("association", "footer"))
- .andReturn(propertyAssociationFooter);
+ expect(propertyFactory.create("association", "footer")).andReturn(propertyAssociationFooter);
Property propertyAssociationAnywhere = createMock(Property.class);
expect(propertyFactory.create("association", "anywhere"))
- .andReturn(propertyAssociationAnywhere).times(2);
+ .andReturn(propertyAssociationAnywhere)
+ .times(2);
Property propertyAssociationBody = createMock(Property.class);
- expect(propertyFactory.create("association", "body"))
- .andReturn(propertyAssociationBody);
+ expect(propertyFactory.create("association", "body")).andReturn(propertyAssociationBody);
Property propertyIssue42 = createMock(Property.class);
- expect(propertyFactory.create("issue", "42"))
- .andReturn(propertyIssue42);
+ expect(propertyFactory.create("issue", "42")).andReturn(propertyIssue42);
Property propertyIssue4711 = createMock(Property.class);
- expect(propertyFactory.create("issue", "4711"))
- .andReturn(propertyIssue4711);
+ expect(propertyFactory.create("issue", "4711")).andReturn(propertyIssue4711);
- HashMap<String,Set<String>> issueMap = Maps.newHashMap();
+ HashMap<String, Set<String>> issueMap = Maps.newHashMap();
issueMap.put("4711", Sets.newHashSet("body", "anywhere"));
issueMap.put("42", Sets.newHashSet("footer", "anywhere"));
if (withRevision) {
PatchSet.Id patchSetId = new PatchSet.Id(new Change.Id(176), 3);
- expect(issueExtractor.getIssueIds("testProject", "testRevision",
- patchSetId)).andReturn(issueMap);
+ expect(issueExtractor.getIssueIds("testProject", "testRevision", patchSetId))
+ .andReturn(issueMap);
} else {
- expect(issueExtractor.getIssueIds("testProject", "testRevision"))
- .andReturn(issueMap);
+ expect(issueExtractor.getIssueIds("testProject", "testRevision")).andReturn(issueMap);
}
replayMocks();
@@ -429,15 +407,13 @@
private class TestModule extends FactoryModule {
@Override
protected void configure() {
- bind(String.class).annotatedWith(PluginName.class)
- .toInstance("ItsTestName");
+ bind(String.class).annotatedWith(PluginName.class).toInstance("ItsTestName");
issueExtractor = createMock(IssueExtractor.class);
bind(IssueExtractor.class).toInstance(issueExtractor);
propertyAttributeExtractor = createMock(PropertyAttributeExtractor.class);
- bind(PropertyAttributeExtractor.class).toInstance(
- propertyAttributeExtractor);
+ bind(PropertyAttributeExtractor.class).toInstance(propertyAttributeExtractor);
propertyFactory = createMock(Property.Factory.class);
bind(Property.Factory.class).toInstance(propertyFactory);
@@ -450,4 +426,4 @@
super(null);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateCommentTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateCommentTest.java
index ea38875..5573878 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateCommentTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/validation/ItsValidateCommentTest.java
@@ -24,22 +24,19 @@
import com.google.gerrit.server.git.validators.CommitValidationMessage;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsConfig;
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.util.IssueExtractor;
-
+import java.io.IOException;
+import java.util.List;
+import java.util.regex.Pattern;
import org.eclipse.jgit.revwalk.RevCommit;
import org.eclipse.jgit.transport.ReceiveCommand;
import org.junit.runner.RunWith;
import org.powermock.core.classloader.annotations.PrepareForTest;
import org.powermock.modules.junit4.PowerMockRunner;
-import java.io.IOException;
-import java.util.List;
-import java.util.regex.Pattern;
-
@RunWith(PowerMockRunner.class)
@PrepareForTest({RevCommit.class})
public class ItsValidateCommentTest extends LoggingMockingTestCase {
@@ -55,11 +52,11 @@
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.OPTIONAL).atLeastOnce();
+ .andReturn(ItsAssociationPolicy.OPTIONAL)
+ .atLeastOnce();
replayMocks();
@@ -73,41 +70,39 @@
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.SUGGESTED).atLeastOnce();
+ .andReturn(ItsAssociationPolicy.SUGGESTED)
+ .atLeastOnce();
expect(commit.getFullMessage()).andReturn("TestMessage").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("TestMessage")).andReturn(
- new String[] {}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("TestMessage")).andReturn(new String[] {}).atLeastOnce();
replayMocks();
ret = ivc.onCommitReceived(event);
- assertEquals("Size of returned CommitValidationMessages does not match",
- 1, ret.size());
- assertTrue("First CommitValidationMessages does not contain 'Missing " +
- "issue'",ret.get(0).getMessage().contains("Missing issue"));
+ assertEquals("Size of returned CommitValidationMessages does not match", 1, ret.size());
+ assertTrue(
+ "First CommitValidationMessages does not contain 'Missing " + "issue'",
+ ret.get(0).getMessage().contains("Missing issue"));
}
public void testMandatoryNonMatching() {
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.MANDATORY).atLeastOnce();
+ .andReturn(ItsAssociationPolicy.MANDATORY)
+ .atLeastOnce();
expect(commit.getFullMessage()).andReturn("TestMessage").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("TestMessage")).andReturn(
- new String[] {}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("TestMessage")).andReturn(new String[] {}).atLeastOnce();
replayMocks();
@@ -115,27 +110,25 @@
ivc.onCommitReceived(event);
fail("onCommitReceived did not throw any exception");
} catch (CommitValidationException e) {
- assertTrue("Message of thrown CommitValidationException does not "
- + "contain 'Missing issue'",
+ assertTrue(
+ "Message of thrown CommitValidationException does not " + "contain 'Missing issue'",
e.getMessage().contains("Missing issue"));
}
}
- public void testSuggestedMatchingSingleExisting()
- throws CommitValidationException, IOException {
+ public void testSuggestedMatchingSingleExisting() throws CommitValidationException, IOException {
List<CommitValidationMessage> ret;
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.SUGGESTED).atLeastOnce();
+ .andReturn(ItsAssociationPolicy.SUGGESTED)
+ .atLeastOnce();
expect(commit.getFullMessage()).andReturn("bug#4711").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711")).andReturn(
- new String[] {"4711"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711")).andReturn(new String[] {"4711"}).atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(true).atLeastOnce();
replayMocks();
@@ -145,22 +138,20 @@
assertEmptyList(ret);
}
- public void testMandatoryMatchingSingleExisting()
- throws CommitValidationException, IOException {
+ public void testMandatoryMatchingSingleExisting() throws CommitValidationException, IOException {
List<CommitValidationMessage> ret;
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.MANDATORY).atLeastOnce();
+ .andReturn(ItsAssociationPolicy.MANDATORY)
+ .atLeastOnce();
expect(commit.getFullMessage()).andReturn("bug#4711").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711")).andReturn(
- new String[] {"4711"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711")).andReturn(new String[] {"4711"}).atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(true).atLeastOnce();
replayMocks();
@@ -176,45 +167,43 @@
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.SUGGESTED).atLeastOnce();
+ .andReturn(ItsAssociationPolicy.SUGGESTED)
+ .atLeastOnce();
expect(commit.getFullMessage()).andReturn("bug#4711").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711")).andReturn(
- new String[] {"4711"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711")).andReturn(new String[] {"4711"}).atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(false).atLeastOnce();
replayMocks();
ret = ivc.onCommitReceived(event);
- assertEquals("Size of returned CommitValidationMessages does not match",
- 1, ret.size());
- assertTrue("First CommitValidationMessages does not contain " +
- "'Non-existing'",ret.get(0).getMessage().contains("Non-existing"));
- assertTrue("First CommitValidationMessages does not contain '4711'",
+ assertEquals("Size of returned CommitValidationMessages does not match", 1, ret.size());
+ assertTrue(
+ "First CommitValidationMessages does not contain " + "'Non-existing'",
+ ret.get(0).getMessage().contains("Non-existing"));
+ assertTrue(
+ "First CommitValidationMessages does not contain '4711'",
ret.get(0).getMessage().contains("4711"));
}
- public void testMandatoryMatchingSingleNonExisting()
- throws IOException {
+ public void testMandatoryMatchingSingleNonExisting() throws IOException {
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.MANDATORY).atLeastOnce();
+ .andReturn(ItsAssociationPolicy.MANDATORY)
+ .atLeastOnce();
expect(commit.getFullMessage()).andReturn("bug#4711").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711")).andReturn(
- new String[] {"4711"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711")).andReturn(new String[] {"4711"}).atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(false).atLeastOnce();
replayMocks();
@@ -223,28 +212,28 @@
ivc.onCommitReceived(event);
fail("onCommitReceived did not throw any exception");
} catch (CommitValidationException e) {
- assertTrue("Message of thrown CommitValidationException does not "
- + "contain 'Non-existing'", e.getMessage().contains("Non-existing"));
+ assertTrue(
+ "Message of thrown CommitValidationException does not " + "contain 'Non-existing'",
+ e.getMessage().contains("Non-existing"));
}
}
- public void testSuggestedMatchingMultiple()
- throws CommitValidationException, IOException {
+ public void testSuggestedMatchingMultiple() throws CommitValidationException, IOException {
List<CommitValidationMessage> ret;
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.SUGGESTED).atLeastOnce();
- expect(commit.getFullMessage()).andReturn("bug#4711, bug#42")
+ .andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
+ expect(commit.getFullMessage()).andReturn("bug#4711, bug#42").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711, bug#42")).andReturn(
- new String[] {"4711", "42"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711, bug#42"))
+ .andReturn(new String[] {"4711", "42"})
+ .atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(true).atLeastOnce();
expect(itsFacade.exists("42")).andReturn(true).atLeastOnce();
@@ -255,23 +244,22 @@
assertEmptyList(ret);
}
- public void testMandatoryMatchingMultiple()
- throws CommitValidationException, IOException {
+ public void testMandatoryMatchingMultiple() throws CommitValidationException, IOException {
List<CommitValidationMessage> ret;
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.MANDATORY).atLeastOnce();
- expect(commit.getFullMessage()).andReturn("bug#4711, bug#42")
+ .andReturn(ItsAssociationPolicy.MANDATORY)
.atLeastOnce();
+ expect(commit.getFullMessage()).andReturn("bug#4711, bug#42").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711, bug#42")).andReturn(
- new String[] {"4711", "42"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711, bug#42"))
+ .andReturn(new String[] {"4711", "42"})
+ .atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(true).atLeastOnce();
expect(itsFacade.exists("42")).andReturn(true).atLeastOnce();
@@ -288,17 +276,17 @@
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.SUGGESTED).atLeastOnce();
- expect(commit.getFullMessage()).andReturn("bug#4711, bug#42")
+ .andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
+ expect(commit.getFullMessage()).andReturn("bug#4711, bug#42").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711, bug#42")).andReturn(
- new String[] {"4711", "42"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711, bug#42"))
+ .andReturn(new String[] {"4711", "42"})
+ .atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(false).atLeastOnce();
expect(itsFacade.exists("42")).andReturn(true).atLeastOnce();
@@ -306,32 +294,33 @@
ret = ivc.onCommitReceived(event);
- assertEquals("Size of returned CommitValidationMessages does not match",
- 1, ret.size());
- assertTrue("First CommitValidationMessages does not contain " +
- "'Non-existing'",ret.get(0).getMessage().contains("Non-existing"));
- assertTrue("First CommitValidationMessages does not contain '4711'",
+ assertEquals("Size of returned CommitValidationMessages does not match", 1, ret.size());
+ assertTrue(
+ "First CommitValidationMessages does not contain " + "'Non-existing'",
+ ret.get(0).getMessage().contains("Non-existing"));
+ assertTrue(
+ "First CommitValidationMessages does not contain '4711'",
ret.get(0).getMessage().contains("4711"));
- assertFalse("First CommitValidationMessages contains '42', although " +
- "that bug exists", ret.get(0).getMessage().contains("42"));
+ assertFalse(
+ "First CommitValidationMessages contains '42', although " + "that bug exists",
+ ret.get(0).getMessage().contains("42"));
}
- public void testMandatoryMatchingMultipleOneNonExsting()
- throws IOException {
+ public void testMandatoryMatchingMultipleOneNonExsting() throws IOException {
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.MANDATORY).atLeastOnce();
- expect(commit.getFullMessage()).andReturn("bug#4711, bug#42")
+ .andReturn(ItsAssociationPolicy.MANDATORY)
.atLeastOnce();
+ expect(commit.getFullMessage()).andReturn("bug#4711, bug#42").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711, bug#42")).andReturn(
- new String[] {"4711", "42"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711, bug#42"))
+ .andReturn(new String[] {"4711", "42"})
+ .atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(false).atLeastOnce();
expect(itsFacade.exists("42")).andReturn(true).atLeastOnce();
@@ -341,8 +330,9 @@
ivc.onCommitReceived(event);
fail("onCommitReceived did not throw any exception");
} catch (CommitValidationException e) {
- assertTrue("Message of thrown CommitValidationException does not "
- + "contain 'Non-existing'", e.getMessage().contains("Non-existing"));
+ assertTrue(
+ "Message of thrown CommitValidationException does not " + "contain 'Non-existing'",
+ e.getMessage().contains("Non-existing"));
}
}
@@ -352,17 +342,17 @@
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.SUGGESTED).atLeastOnce();
- expect(commit.getFullMessage()).andReturn("bug#4711, bug#42")
+ .andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
+ expect(commit.getFullMessage()).andReturn("bug#4711, bug#42").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711, bug#42")).andReturn(
- new String[] {"4711", "42"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711, bug#42"))
+ .andReturn(new String[] {"4711", "42"})
+ .atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(false).atLeastOnce();
expect(itsFacade.exists("42")).andReturn(false).atLeastOnce();
@@ -370,32 +360,33 @@
ret = ivc.onCommitReceived(event);
- assertEquals("Size of returned CommitValidationMessages does not match",
- 1, ret.size());
- assertTrue("First CommitValidationMessages does not contain " +
- "'Non-existing'",ret.get(0).getMessage().contains("Non-existing"));
- assertTrue("First CommitValidationMessages does not contain '4711'",
+ assertEquals("Size of returned CommitValidationMessages does not match", 1, ret.size());
+ assertTrue(
+ "First CommitValidationMessages does not contain " + "'Non-existing'",
+ ret.get(0).getMessage().contains("Non-existing"));
+ assertTrue(
+ "First CommitValidationMessages does not contain '4711'",
ret.get(0).getMessage().contains("4711"));
- assertTrue("First CommitValidationMessages does not contain '42'",
+ assertTrue(
+ "First CommitValidationMessages does not contain '42'",
ret.get(0).getMessage().contains("42"));
}
- public void testMandatoryMatchingMultipleSomeNonExsting()
- throws IOException {
+ public void testMandatoryMatchingMultipleSomeNonExsting() throws IOException {
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.MANDATORY).atLeastOnce();
- expect(commit.getFullMessage()).andReturn("bug#4711, bug#42")
+ .andReturn(ItsAssociationPolicy.MANDATORY)
.atLeastOnce();
+ expect(commit.getFullMessage()).andReturn("bug#4711, bug#42").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
- expect(issueExtractor.getIssueIds("bug#4711, bug#42")).andReturn(
- new String[] {"4711", "42"}).atLeastOnce();
+ expect(issueExtractor.getIssueIds("bug#4711, bug#42"))
+ .andReturn(new String[] {"4711", "42"})
+ .atLeastOnce();
expect(itsFacade.exists("4711")).andReturn(false).atLeastOnce();
expect(itsFacade.exists("42")).andReturn(false).atLeastOnce();
@@ -405,8 +396,9 @@
ivc.onCommitReceived(event);
fail("onCommitReceived did not throw any exception");
} catch (CommitValidationException e) {
- assertTrue("Message of thrown CommitValidationException does not "
- + "contain 'Non-existing'", e.getMessage().contains("Non-existing"));
+ assertTrue(
+ "Message of thrown CommitValidationException does not " + "contain 'Non-existing'",
+ e.getMessage().contains("Non-existing"));
}
}
@@ -416,38 +408,42 @@
ItsValidateComment ivc = injector.getInstance(ItsValidateComment.class);
ReceiveCommand command = createMock(ReceiveCommand.class);
RevCommit commit = createMock(RevCommit.class);
- CommitReceivedEvent event = newCommitReceivedEvent(command, project, null,
- commit, null);
+ CommitReceivedEvent event = newCommitReceivedEvent(command, project, null, commit, null);
expect(itsConfig.getItsAssociationPolicy())
- .andReturn(ItsAssociationPolicy.SUGGESTED).atLeastOnce();
- expect(commit.getFullMessage()).andReturn("bug#4711, bug#42")
+ .andReturn(ItsAssociationPolicy.SUGGESTED)
.atLeastOnce();
+ expect(commit.getFullMessage()).andReturn("bug#4711, bug#42").atLeastOnce();
expect(commit.getId()).andReturn(commit).anyTimes();
expect(commit.getName()).andReturn("TestCommit").anyTimes();
expect(issueExtractor.getIssueIds("bug#4711, bug#42"))
- .andReturn(new String[] {"4711", "42"}).atLeastOnce();
- expect(itsFacade.exists("4711")).andThrow(new IOException("InjectedEx1"))
+ .andReturn(new String[] {"4711", "42"})
.atLeastOnce();
+ expect(itsFacade.exists("4711")).andThrow(new IOException("InjectedEx1")).atLeastOnce();
expect(itsFacade.exists("42")).andReturn(false).atLeastOnce();
replayMocks();
ret = ivc.onCommitReceived(event);
- assertEquals("Size of returned CommitValidationMessages does not match",
- 2, ret.size());
- assertTrue("First CommitValidationMessages does not contain " +
- "'Failed'",ret.get(0).getMessage().contains("Failed"));
- assertTrue("First CommitValidationMessages does not contain '4711'",
+ assertEquals("Size of returned CommitValidationMessages does not match", 2, ret.size());
+ assertTrue(
+ "First CommitValidationMessages does not contain " + "'Failed'",
+ ret.get(0).getMessage().contains("Failed"));
+ assertTrue(
+ "First CommitValidationMessages does not contain '4711'",
ret.get(0).getMessage().contains("4711"));
- assertFalse("First CommitValidationMessages contains '42', although " +
- "that bug exists", ret.get(0).getMessage().contains("42"));
- assertTrue("Second CommitValidationMessages does not contain " +
- "'Non-existing'",ret.get(1).getMessage().contains("Non-existing"));
- assertTrue("Second CommitValidationMessages does not contain '4711'",
+ assertFalse(
+ "First CommitValidationMessages contains '42', although " + "that bug exists",
+ ret.get(0).getMessage().contains("42"));
+ assertTrue(
+ "Second CommitValidationMessages does not contain " + "'Non-existing'",
+ ret.get(1).getMessage().contains("Non-existing"));
+ assertTrue(
+ "Second CommitValidationMessages does not contain '4711'",
ret.get(1).getMessage().contains("4711"));
- assertTrue("Second CommitValidationMessages does not contain '42'",
+ assertTrue(
+ "Second CommitValidationMessages does not contain '42'",
ret.get(1).getMessage().contains("42"));
assertLogMessageContains("4711");
@@ -466,11 +462,9 @@
}
private void setupCommonMocks() {
- expect(itsConfig.getIssuePattern())
- .andReturn(Pattern.compile("bug#(\\d+)")).anyTimes();
+ expect(itsConfig.getIssuePattern()).andReturn(Pattern.compile("bug#(\\d+)")).anyTimes();
Project.NameKey projectNK = new Project.NameKey("myProject");
- expect(itsConfig.isEnabled(projectNK, null)).andReturn(true)
- .anyTimes();
+ expect(itsConfig.isEnabled(projectNK, null)).andReturn(true).anyTimes();
}
@Override
@@ -502,8 +496,7 @@
private class TestModule extends FactoryModule {
@Override
protected void configure() {
- bind(String.class).annotatedWith(PluginName.class)
- .toInstance("ItsTestName");
+ bind(String.class).annotatedWith(PluginName.class).toInstance("ItsTestName");
issueExtractor = createMock(IssueExtractor.class);
bind(IssueExtractor.class).toInstance(issueExtractor);
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionControllerTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionControllerTest.java
index e62184a..d2926f9 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionControllerTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionControllerTest.java
@@ -23,11 +23,9 @@
import com.google.gerrit.server.events.Event;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsConfig;
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.util.PropertyExtractor;
-
import java.util.Collection;
import java.util.Collections;
import java.util.Set;
@@ -46,8 +44,7 @@
ChangeEvent event = createMock(ChangeEvent.class);
Set<Set<Property>> propertySets = Collections.emptySet();
- expect(propertyExtractor.extractFrom(event)).andReturn(propertySets)
- .anyTimes();
+ expect(propertyExtractor.extractFrom(event)).andReturn(propertySets).anyTimes();
replayMocks();
@@ -63,8 +60,7 @@
Set<Property> propertySet = Collections.emptySet();
propertySets.add(propertySet);
- expect(propertyExtractor.extractFrom(event)).andReturn(propertySets)
- .anyTimes();
+ expect(propertyExtractor.extractFrom(event)).andReturn(propertySets).anyTimes();
Collection<ActionRequest> actions = Collections.emptySet();
expect(ruleBase.actionRequestsFor(propertySet)).andReturn(actions).once();
@@ -83,8 +79,7 @@
Set<Property> propertySet = Collections.emptySet();
propertySets.add(propertySet);
- expect(propertyExtractor.extractFrom(event)).andReturn(propertySets)
- .anyTimes();
+ expect(propertyExtractor.extractFrom(event)).andReturn(propertySets).anyTimes();
Collection<ActionRequest> actions = Lists.newArrayListWithCapacity(1);
ActionRequest action1 = createMock(ActionRequest.class);
@@ -111,15 +106,12 @@
Set<Set<Property>> propertySets = Sets.newHashSet();
propertySets.add(propertySet);
- expect(propertyExtractor.extractFrom(event)).andReturn(propertySets)
- .anyTimes();
+ expect(propertyExtractor.extractFrom(event)).andReturn(propertySets).anyTimes();
- Collection<ActionRequest> actionRequests =
- Lists.newArrayListWithCapacity(1);
+ Collection<ActionRequest> actionRequests = Lists.newArrayListWithCapacity(1);
ActionRequest actionRequest1 = createMock(ActionRequest.class);
actionRequests.add(actionRequest1);
- expect(ruleBase.actionRequestsFor(propertySet)).andReturn(actionRequests)
- .once();
+ expect(ruleBase.actionRequestsFor(propertySet)).andReturn(actionRequests).once();
actionExecutor.execute("testIssue", actionRequests, propertySet);
@@ -152,25 +144,20 @@
propertySets.add(propertySet1);
propertySets.add(propertySet2);
- expect(propertyExtractor.extractFrom(event)).andReturn(propertySets)
- .anyTimes();
+ expect(propertyExtractor.extractFrom(event)).andReturn(propertySets).anyTimes();
- Collection<ActionRequest> actionRequests1 =
- Lists.newArrayListWithCapacity(1);
+ Collection<ActionRequest> actionRequests1 = Lists.newArrayListWithCapacity(1);
ActionRequest actionRequest1 = createMock(ActionRequest.class);
actionRequests1.add(actionRequest1);
- Collection<ActionRequest> actionRequests2 =
- Lists.newArrayListWithCapacity(2);
+ Collection<ActionRequest> actionRequests2 = Lists.newArrayListWithCapacity(2);
ActionRequest actionRequest2 = createMock(ActionRequest.class);
actionRequests2.add(actionRequest2);
ActionRequest actionRequest3 = createMock(ActionRequest.class);
actionRequests2.add(actionRequest3);
- expect(ruleBase.actionRequestsFor(propertySet1)).andReturn(actionRequests1)
- .once();
- expect(ruleBase.actionRequestsFor(propertySet2)).andReturn(actionRequests2)
- .once();
+ expect(ruleBase.actionRequestsFor(propertySet1)).andReturn(actionRequests1).once();
+ expect(ruleBase.actionRequestsFor(propertySet2)).andReturn(actionRequests2).once();
actionExecutor.execute("testIssue", actionRequests1, propertySet1);
actionExecutor.execute("testIssue", actionRequests2, propertySet2);
@@ -180,13 +167,13 @@
actionController.onEvent(event);
}
+
private ActionController createActionController() {
return injector.getInstance(ActionController.class);
}
private void setupCommonMocks() {
- expect(itsConfig.isEnabled(anyObject(Event.class))).andReturn(true)
- .anyTimes();
+ expect(itsConfig.isEnabled(anyObject(Event.class))).andReturn(true).anyTimes();
}
@Override
@@ -213,4 +200,4 @@
bind(ItsConfig.class).toInstance(itsConfig);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionExecutorTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionExecutorTest.java
index 8cab7a4..4dbc779 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionExecutorTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionExecutorTest.java
@@ -20,14 +20,12 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.workflow.action.AddComment;
import com.googlesource.gerrit.plugins.its.base.workflow.action.AddSoyComment;
import com.googlesource.gerrit.plugins.its.base.workflow.action.AddStandardComment;
import com.googlesource.gerrit.plugins.its.base.workflow.action.LogEvent;
-
import java.io.IOException;
import java.util.Collections;
import java.util.Set;
@@ -91,8 +89,7 @@
replayMocks();
ActionExecutor actionExecutor = createActionExecutor();
- actionExecutor.execute("4711", Sets.newHashSet(
- actionRequest1, actionRequest2), properties);
+ actionExecutor.execute("4711", Sets.newHashSet(actionRequest1, actionRequest2), properties);
}
public void testExecuteIterableExceptions() throws IOException {
@@ -119,8 +116,8 @@
replayMocks();
ActionExecutor actionExecutor = createActionExecutor();
- actionExecutor.execute("4711", Sets.newHashSet(
- actionRequest1, actionRequest2, actionRequest3), properties);
+ actionExecutor.execute(
+ "4711", Sets.newHashSet(actionRequest1, actionRequest2, actionRequest3), properties);
assertLogThrowableMessageContains("injected exception 1");
assertLogThrowableMessageContains("injected exception 3");
@@ -149,8 +146,7 @@
Set<Property> properties = Collections.emptySet();
- AddSoyComment addSoyComment =
- createMock(AddSoyComment.class);
+ AddSoyComment addSoyComment = createMock(AddSoyComment.class);
expect(addSoyCommentFactory.create()).andReturn(addSoyComment);
addSoyComment.execute("4711", actionRequest, properties);
@@ -167,8 +163,7 @@
Set<Property> properties = Collections.emptySet();
- AddStandardComment addStandardComment =
- createMock(AddStandardComment.class);
+ AddStandardComment addStandardComment = createMock(AddStandardComment.class);
expect(addStandardCommentFactory.create()).andReturn(addStandardComment);
addStandardComment.execute("4711", actionRequest, properties);
@@ -216,15 +211,13 @@
bind(AddComment.Factory.class).toInstance(addCommentFactory);
addSoyCommentFactory = createMock(AddSoyComment.Factory.class);
- bind(AddSoyComment.Factory.class).toInstance(
- addSoyCommentFactory);
+ bind(AddSoyComment.Factory.class).toInstance(addSoyCommentFactory);
addStandardCommentFactory = createMock(AddStandardComment.Factory.class);
- bind(AddStandardComment.Factory.class).toInstance(
- addStandardCommentFactory);
+ bind(AddStandardComment.Factory.class).toInstance(addStandardCommentFactory);
logEventFactory = createMock(LogEvent.Factory.class);
bind(LogEvent.Factory.class).toInstance(logEventFactory);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequestTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequestTest.java
index d2461dc..e6de999 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequestTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ActionRequestTest.java
@@ -16,9 +16,7 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
-
import java.util.Arrays;
public class ActionRequestTest extends LoggingMockingTestCase {
@@ -28,146 +26,127 @@
replayMocks();
ActionRequest actionRequest = createActionRequest("action");
- assertEquals("Unparsed string does not match", "action",
- actionRequest.getUnparsed());
+ assertEquals("Unparsed string does not match", "action", actionRequest.getUnparsed());
}
public void testUnparsedNull() {
replayMocks();
ActionRequest actionRequest = createActionRequest(null);
- assertEquals("Unparsed string does not match", "",
- actionRequest.getUnparsed());
+ assertEquals("Unparsed string does not match", "", actionRequest.getUnparsed());
}
public void testUnparsedSingleParameter() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param");
- assertEquals("Unparsed string does not match", "action param",
- actionRequest.getUnparsed());
+ assertEquals("Unparsed string does not match", "action param", actionRequest.getUnparsed());
}
public void testUnparsedMultipleParameters() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param1 param2");
- assertEquals("Unparsed string does not match", "action param1 param2",
- actionRequest.getUnparsed());
+ assertEquals(
+ "Unparsed string does not match", "action param1 param2", actionRequest.getUnparsed());
}
public void testNameParameterless() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action");
- assertEquals("Unparsed string does not match", "action",
- actionRequest.getName());
+ assertEquals("Unparsed string does not match", "action", actionRequest.getName());
}
public void testNameSingleParameter() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param");
- assertEquals("Unparsed string does not match", "action",
- actionRequest.getName());
+ assertEquals("Unparsed string does not match", "action", actionRequest.getName());
}
public void testNameMultipleParameters() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param1 param2");
- assertEquals("Unparsed string does not match", "action",
- actionRequest.getName());
+ assertEquals("Unparsed string does not match", "action", actionRequest.getName());
}
public void testNameNull() {
replayMocks();
ActionRequest actionRequest = createActionRequest(null);
- assertEquals("Unparsed string does not match", "",
- actionRequest.getName());
+ assertEquals("Unparsed string does not match", "", actionRequest.getName());
}
public void testParameter1Parameterless() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action");
- assertEquals("Unparsed string does not match", "",
- actionRequest.getParameter(1));
+ assertEquals("Unparsed string does not match", "", actionRequest.getParameter(1));
}
public void testParameter1Null() {
replayMocks();
ActionRequest actionRequest = createActionRequest(null);
- assertEquals("Unparsed string does not match", "",
- actionRequest.getParameter(1));
+ assertEquals("Unparsed string does not match", "", actionRequest.getParameter(1));
}
public void testParameter1SingleParameter() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param");
- assertEquals("Unparsed string does not match", "param",
- actionRequest.getParameter(1));
+ assertEquals("Unparsed string does not match", "param", actionRequest.getParameter(1));
}
public void testParemeter1MultipleParameters() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param1 param2");
- assertEquals("Unparsed string does not match", "param1",
- actionRequest.getParameter(1));
+ assertEquals("Unparsed string does not match", "param1", actionRequest.getParameter(1));
}
public void testParameter3Parameterless() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action");
- assertEquals("Unparsed string does not match", "",
- actionRequest.getParameter(3));
+ assertEquals("Unparsed string does not match", "", actionRequest.getParameter(3));
}
public void testParameter3Null() {
replayMocks();
ActionRequest actionRequest = createActionRequest(null);
- assertEquals("Unparsed string does not match", "",
- actionRequest.getParameter(3));
+ assertEquals("Unparsed string does not match", "", actionRequest.getParameter(3));
}
public void testParameter3SingleParameter() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param");
- assertEquals("Unparsed string does not match", "",
- actionRequest.getParameter(3));
+ assertEquals("Unparsed string does not match", "", actionRequest.getParameter(3));
}
public void testParemeter3With2Parameters() {
replayMocks();
ActionRequest actionRequest = createActionRequest("action param1 param2");
- assertEquals("Unparsed string does not match", "",
- actionRequest.getParameter(3));
+ assertEquals("Unparsed string does not match", "", actionRequest.getParameter(3));
}
public void testParemeter3With3Parameters() {
replayMocks();
- ActionRequest actionRequest = createActionRequest("action param1 param2 " +
- "param3");
- assertEquals("Unparsed string does not match", "param3",
- actionRequest.getParameter(3));
+ ActionRequest actionRequest = createActionRequest("action param1 param2 " + "param3");
+ assertEquals("Unparsed string does not match", "param3", actionRequest.getParameter(3));
}
public void testParemeter3With4Parameters() {
replayMocks();
- ActionRequest actionRequest = createActionRequest("action param1 param2 " +
- "param3 param4");
- assertEquals("Unparsed string does not match", "param3",
- actionRequest.getParameter(3));
+ ActionRequest actionRequest = createActionRequest("action param1 param2 " + "param3 param4");
+ assertEquals("Unparsed string does not match", "param3", actionRequest.getParameter(3));
}
public void testParametersParameterless() {
@@ -176,7 +155,9 @@
ActionRequest actionRequest = createActionRequest("action");
String[] expected = new String[0];
- assertEquals("Parameters do not match", Arrays.asList(expected),
+ assertEquals(
+ "Parameters do not match",
+ Arrays.asList(expected),
Arrays.asList(actionRequest.getParameters()));
}
@@ -186,7 +167,9 @@
ActionRequest actionRequest = createActionRequest(null);
String[] expected = new String[0];
- assertEquals("Parameters do not match", Arrays.asList(expected),
+ assertEquals(
+ "Parameters do not match",
+ Arrays.asList(expected),
Arrays.asList(actionRequest.getParameters()));
}
@@ -195,25 +178,27 @@
ActionRequest actionRequest = createActionRequest("action param");
- String[] expected = new String[] { "param" };
- assertEquals("Parameters do not match", Arrays.asList(expected),
+ String[] expected = new String[] {"param"};
+ assertEquals(
+ "Parameters do not match",
+ Arrays.asList(expected),
Arrays.asList(actionRequest.getParameters()));
}
public void testParameters3Parameter() {
replayMocks();
- ActionRequest actionRequest = createActionRequest("action param1 param2 " +
- "param3");
+ ActionRequest actionRequest = createActionRequest("action param1 param2 " + "param3");
- String[] expected = new String[] { "param1", "param2", "param3" };
- assertEquals("Parameters do not match", Arrays.asList(expected),
+ String[] expected = new String[] {"param1", "param2", "param3"};
+ assertEquals(
+ "Parameters do not match",
+ Arrays.asList(expected),
Arrays.asList(actionRequest.getParameters()));
}
private ActionRequest createActionRequest(String specification) {
- ActionRequest.Factory factory = injector.getInstance(
- ActionRequest.Factory.class);
+ ActionRequest.Factory factory = injector.getInstance(ActionRequest.Factory.class);
return factory.create(specification);
}
@@ -229,4 +214,4 @@
factory(ActionRequest.Factory.class);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ConditionTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ConditionTest.java
index 5f58dcc..c1a6a8d 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ConditionTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/ConditionTest.java
@@ -19,13 +19,11 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
-
import java.util.Collection;
import java.util.Collections;
-public class ConditionTest extends LoggingMockingTestCase {
+public class ConditionTest extends LoggingMockingTestCase {
private Injector injector;
public void testGetKeyNull() {
@@ -351,4 +349,4 @@
factory(Condition.Factory.class);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/PropertyTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/PropertyTest.java
index dcffab4..06ac41a 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/PropertyTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/PropertyTest.java
@@ -16,10 +16,9 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
-public class PropertyTest extends LoggingMockingTestCase {
+public class PropertyTest extends LoggingMockingTestCase {
private Injector injector;
public void testGetKeyNull() {
@@ -61,36 +60,31 @@
public void testEqualsNulledKey() {
Property propertyA = new Property(null, "testValue");
Property propertyB = createProperty("testKey", "testValue");
- assertFalse("Single nulled key does match",
- propertyA.equals(propertyB));
+ assertFalse("Single nulled key does match", propertyA.equals(propertyB));
}
public void testEqualsNulledKey2() {
Property propertyA = createProperty("testKey", "testValue");
Property propertyB = new Property(null, "testValue");
- assertFalse("Single nulled key does match",
- propertyA.equals(propertyB));
+ assertFalse("Single nulled key does match", propertyA.equals(propertyB));
}
public void testEqualsNulledValue() {
Property propertyA = createProperty("testKey", "testValue");
Property propertyB = createProperty("testKey", null);
- assertFalse("Single nulled value does match",
- propertyA.equals(propertyB));
+ assertFalse("Single nulled value does match", propertyA.equals(propertyB));
}
public void testEqualsNulledValue2() {
Property propertyA = createProperty("testKey", null);
Property propertyB = createProperty("testKey", "testValue");
- assertFalse("Single nulled value does match",
- propertyA.equals(propertyB));
+ assertFalse("Single nulled value does match", propertyA.equals(propertyB));
}
public void testHashCodeEquals() {
Property propertyA = createProperty("testKey", "testValue");
Property propertyB = createProperty("testKey", "testValue");
- assertEquals("Hash codes do not match", propertyA.hashCode(),
- propertyB.hashCode());
+ assertEquals("Hash codes do not match", propertyA.hashCode(), propertyB.hashCode());
}
public void testHashCodeNullKey() {
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBaseTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBaseTest.java
index 596afc7..80e12fb 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBaseTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleBaseTest.java
@@ -21,11 +21,7 @@
import com.google.gerrit.server.config.SitePath;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
-
-import org.eclipse.jgit.util.FileUtils;
-
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileWriter;
@@ -37,6 +33,7 @@
import java.util.Collections;
import java.util.List;
import java.util.UUID;
+import org.eclipse.jgit.util.FileUtils;
public class RuleBaseTest extends LoggingMockingTestCase {
private Injector injector;
@@ -49,9 +46,9 @@
private boolean cleanupSitePath;
private enum RuleBaseKind {
- GLOBAL,
- ITS,
- FAULTY
+ GLOBAL,
+ ITS,
+ FAULTY
}
public void testWarnNonExistingRuleBase() {
@@ -71,9 +68,7 @@
}
public void testSimpleRuleBase() throws IOException {
- injectRuleBase("[rule \"rule1\"]\n" +
- "\tconditionA = value1\n" +
- "\taction = action1");
+ injectRuleBase("[rule \"rule1\"]\n" + "\tconditionA = value1\n" + "\taction = action1");
Rule rule1 = createMock(Rule.class);
expect(ruleFactory.create("rule1")).andReturn(rule1);
@@ -92,36 +87,34 @@
}
public void testBasicRuleBase() throws IOException {
- injectRuleBase("[rule \"rule1\"]\n" +
- "\tconditionA = value1,value2\n" +
- "\tconditionA = value3,value of 4\n" +
- "\tconditionB = value5\n" +
- "\taction = action1\n" +
- "\taction = action2 param\n" +
- "\n" +
- "[ruleXZ \"nonrule\"]\n" +
- "\tconditionA = value1\n" +
- "\taction = action2\n" +
- "[rule \"rule2\"]\n" +
- "\tconditionC = value6\n" +
- "\taction = action3");
+ injectRuleBase(
+ "[rule \"rule1\"]\n"
+ + "\tconditionA = value1,value2\n"
+ + "\tconditionA = value3,value of 4\n"
+ + "\tconditionB = value5\n"
+ + "\taction = action1\n"
+ + "\taction = action2 param\n"
+ + "\n"
+ + "[ruleXZ \"nonrule\"]\n"
+ + "\tconditionA = value1\n"
+ + "\taction = action2\n"
+ + "[rule \"rule2\"]\n"
+ + "\tconditionC = value6\n"
+ + "\taction = action3");
Rule rule1 = createMock(Rule.class);
expect(ruleFactory.create("rule1")).andReturn(rule1);
Condition condition1 = createMock(Condition.class);
- expect(conditionFactory.create("conditionA", "value1,value2")).
- andReturn(condition1);
+ expect(conditionFactory.create("conditionA", "value1,value2")).andReturn(condition1);
rule1.addCondition(condition1);
Condition condition2 = createMock(Condition.class);
- expect(conditionFactory.create("conditionA", "value3,value of 4")).
- andReturn(condition2);
+ expect(conditionFactory.create("conditionA", "value3,value of 4")).andReturn(condition2);
rule1.addCondition(condition2);
Condition condition3 = createMock(Condition.class);
- expect(conditionFactory.create("conditionB", "value5")).
- andReturn(condition3);
+ expect(conditionFactory.create("conditionB", "value5")).andReturn(condition3);
rule1.addCondition(condition3);
ActionRequest actionRequest1 = createMock(ActionRequest.class);
@@ -136,8 +129,7 @@
expect(ruleFactory.create("rule2")).andReturn(rule2);
Condition condition4 = createMock(Condition.class);
- expect(conditionFactory.create("conditionC", "value6")).
- andReturn(condition4);
+ expect(conditionFactory.create("conditionC", "value6")).andReturn(condition4);
rule2.addCondition(condition4);
ActionRequest actionRequest3 = createMock(ActionRequest.class);
@@ -150,8 +142,7 @@
}
public void testActionRequestsForSimple() throws IOException {
- injectRuleBase("[rule \"rule1\"]\n" +
- "\taction = action1");
+ injectRuleBase("[rule \"rule1\"]\n" + "\taction = action1");
Rule rule1 = createMock(Rule.class);
expect(ruleFactory.create("rule1")).andReturn(rule1);
@@ -178,12 +169,13 @@
}
public void testActionRequestsForExtended() throws IOException {
- injectRuleBase("[rule \"rule1\"]\n" +
- "\taction = action1\n" +
- "\taction = action2\n" +
- "\n" +
- "[rule \"rule2\"]\n" +
- "\taction = action3");
+ injectRuleBase(
+ "[rule \"rule1\"]\n"
+ + "\taction = action1\n"
+ + "\taction = action2\n"
+ + "\n"
+ + "[rule \"rule2\"]\n"
+ + "\taction = action3");
Rule rule1 = createMock(Rule.class);
expect(ruleFactory.create("rule1")).andReturn(rule1);
@@ -241,9 +233,9 @@
}
public void testSimpleFaultyNameRuleBase() throws IOException {
- injectRuleBase("[rule \"rule1\"]\n" +
- "\tconditionA = value1\n" +
- "\taction = action1", RuleBaseKind.FAULTY);
+ injectRuleBase(
+ "[rule \"rule1\"]\n" + "\tconditionA = value1\n" + "\taction = action1",
+ RuleBaseKind.FAULTY);
Rule rule1 = createMock(Rule.class);
expect(ruleFactory.create("rule1")).andReturn(rule1);
@@ -265,9 +257,8 @@
}
public void testSimpleItsRuleBase() throws IOException {
- injectRuleBase("[rule \"rule1\"]\n" +
- "\tconditionA = value1\n" +
- "\taction = action1", RuleBaseKind.ITS);
+ injectRuleBase(
+ "[rule \"rule1\"]\n" + "\tconditionA = value1\n" + "\taction = action1", RuleBaseKind.ITS);
Rule rule1 = createMock(Rule.class);
expect(ruleFactory.create("rule1")).andReturn(rule1);
@@ -286,14 +277,11 @@
}
public void testAllRuleBaseFilesAreLoaded() throws IOException {
- injectRuleBase("[rule \"rule1\"]\n" +
- "\taction = action1", RuleBaseKind.FAULTY);
+ injectRuleBase("[rule \"rule1\"]\n" + "\taction = action1", RuleBaseKind.FAULTY);
- injectRuleBase("[rule \"rule2\"]\n" +
- "\taction = action2", RuleBaseKind.GLOBAL);
+ injectRuleBase("[rule \"rule2\"]\n" + "\taction = action2", RuleBaseKind.GLOBAL);
- injectRuleBase("[rule \"rule3\"]\n" +
- "\taction = action3", RuleBaseKind.ITS);
+ injectRuleBase("[rule \"rule3\"]\n" + "\taction = action3", RuleBaseKind.ITS);
Collection<Property> properties = Collections.emptySet();
@@ -369,14 +357,16 @@
default:
fail("Unknown ruleBaseKind");
}
- File ruleBaseFile = new File(sitePath.toFile(),
- "etc" + File.separatorChar + "its" + File.separator +
- baseName + ".config");
+ File ruleBaseFile =
+ new File(
+ sitePath.toFile(),
+ "etc" + File.separatorChar + "its" + File.separator + baseName + ".config");
File ruleBaseParentFile = ruleBaseFile.getParentFile();
if (!ruleBaseParentFile.exists()) {
- assertTrue("Failed to create parent (" + ruleBaseParentFile + ") for " +
- "rule base", ruleBaseParentFile.mkdirs());
+ assertTrue(
+ "Failed to create parent (" + ruleBaseParentFile + ") for " + "rule base",
+ ruleBaseParentFile.mkdirs());
}
try (FileWriter unbufferedWriter = new FileWriter(ruleBaseFile);
BufferedWriter writer = new BufferedWriter(unbufferedWriter)) {
@@ -409,12 +399,10 @@
@Override
protected void configure() {
- bind(String.class).annotatedWith(PluginName.class)
- .toInstance("ItsTestName");
+ bind(String.class).annotatedWith(PluginName.class).toInstance("ItsTestName");
sitePath = randomTargetPath();
- assertFalse("sitePath already (" + sitePath + ") already exists",
- Files.exists(sitePath));
+ assertFalse("sitePath already (" + sitePath + ") already exists", Files.exists(sitePath));
cleanupSitePath = true;
bind(Path.class).annotatedWith(SitePath.class).toInstance(sitePath);
@@ -429,4 +417,4 @@
bind(ActionRequest.Factory.class).toInstance(actionRequestFactory);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleTest.java
index 4b31618..6d4d234 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/RuleTest.java
@@ -19,9 +19,7 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
-
import java.util.Collection;
import java.util.Collections;
import java.util.List;
@@ -133,4 +131,4 @@
factory(Rule.Factory.class);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddCommentTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddCommentTest.java
index a86798e..3039b12 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddCommentTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddCommentTest.java
@@ -18,12 +18,10 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.workflow.ActionRequest;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
import java.io.IOException;
import java.util.HashSet;
@@ -34,26 +32,24 @@
public void testEmpty() throws IOException {
ActionRequest actionRequest = createMock(ActionRequest.class);
- expect(actionRequest.getParameters()).andReturn(
- new String[] {});
+ expect(actionRequest.getParameters()).andReturn(new String[] {});
replayMocks();
AddComment addComment = createAddComment();
- addComment.execute("4711", actionRequest, new HashSet<Property>());
+ addComment.execute("4711", actionRequest, new HashSet<>());
}
public void testPlain() throws IOException {
ActionRequest actionRequest = createMock(ActionRequest.class);
- expect(actionRequest.getParameters()).andReturn(
- new String[] {"Some", "test", "comment"});
+ expect(actionRequest.getParameters()).andReturn(new String[] {"Some", "test", "comment"});
its.addComment("4711", "Some test comment");
replayMocks();
AddComment addComment = createAddComment();
- addComment.execute("4711", actionRequest, new HashSet<Property>());
+ addComment.execute("4711", actionRequest, new HashSet<>());
}
private AddComment createAddComment() {
@@ -73,4 +69,4 @@
bind(ItsFacade.class).toInstance(its);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardCommentTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardCommentTest.java
index df42702..cfcda01 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardCommentTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/AddStandardCommentTest.java
@@ -19,12 +19,10 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.its.ItsFacade;
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.workflow.ActionRequest;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
import java.io.IOException;
import java.util.Set;
@@ -66,30 +64,29 @@
properties.add(propertySubject);
Property propertyChangeNumber = createMock(Property.class);
- expect(propertyChangeNumber.getKey()).andReturn("changeNumber")
- .anyTimes();
+ expect(propertyChangeNumber.getKey()).andReturn("changeNumber").anyTimes();
expect(propertyChangeNumber.getValue()).andReturn("4711").anyTimes();
properties.add(propertyChangeNumber);
Property propertySubmitterName = createMock(Property.class);
- expect(propertySubmitterName.getKey()).andReturn("submitterName")
- .anyTimes();
+ expect(propertySubmitterName.getKey()).andReturn("submitterName").anyTimes();
expect(propertySubmitterName.getValue()).andReturn("John Doe").anyTimes();
properties.add(propertySubmitterName);
- Property propertyChangeUrl= createMock(Property.class);
+ Property propertyChangeUrl = createMock(Property.class);
expect(propertyChangeUrl.getKey()).andReturn("changeUrl").anyTimes();
- expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change")
- .anyTimes();
+ expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change").anyTimes();
properties.add(propertyChangeUrl);
- expect(its.createLinkForWebui("http://example.org/change",
- "http://example.org/change")).andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
+ expect(its.createLinkForWebui("http://example.org/change", "http://example.org/change"))
+ .andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
- its.addComment("176", "Change 4711 merged by John Doe:\n" +
- "Test-Change-Subject\n" +
- "\n" +
- "HtTp://ExAmPlE.OrG/ChAnGe");
+ its.addComment(
+ "176",
+ "Change 4711 merged by John Doe:\n"
+ + "Test-Change-Subject\n"
+ + "\n"
+ + "HtTp://ExAmPlE.OrG/ChAnGe");
replayMocks();
Action action = injector.getInstance(AddStandardComment.class);
@@ -134,33 +131,32 @@
properties.add(propertySubject);
Property propertyChangeNumber = createMock(Property.class);
- expect(propertyChangeNumber.getKey()).andReturn("changeNumber")
- .anyTimes();
+ expect(propertyChangeNumber.getKey()).andReturn("changeNumber").anyTimes();
expect(propertyChangeNumber.getValue()).andReturn("4711").anyTimes();
properties.add(propertyChangeNumber);
Property propertySubmitterName = createMock(Property.class);
- expect(propertySubmitterName.getKey()).andReturn("abandonerName")
- .anyTimes();
+ expect(propertySubmitterName.getKey()).andReturn("abandonerName").anyTimes();
expect(propertySubmitterName.getValue()).andReturn("John Doe").anyTimes();
properties.add(propertySubmitterName);
- Property propertyChangeUrl= createMock(Property.class);
+ Property propertyChangeUrl = createMock(Property.class);
expect(propertyChangeUrl.getKey()).andReturn("changeUrl").anyTimes();
- expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change")
- .anyTimes();
+ expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change").anyTimes();
properties.add(propertyChangeUrl);
- expect(its.createLinkForWebui("http://example.org/change",
- "http://example.org/change")).andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
+ expect(its.createLinkForWebui("http://example.org/change", "http://example.org/change"))
+ .andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
- its.addComment("176", "Change 4711 abandoned by John Doe:\n" +
- "Test-Change-Subject\n" +
- "\n" +
- "Reason:\n" +
- "Test-Reason\n" +
- "\n" +
- "HtTp://ExAmPlE.OrG/ChAnGe");
+ its.addComment(
+ "176",
+ "Change 4711 abandoned by John Doe:\n"
+ + "Test-Change-Subject\n"
+ + "\n"
+ + "Reason:\n"
+ + "Test-Reason\n"
+ + "\n"
+ + "HtTp://ExAmPlE.OrG/ChAnGe");
replayMocks();
Action action = injector.getInstance(AddStandardComment.class);
@@ -205,33 +201,32 @@
properties.add(propertySubject);
Property propertyChangeNumber = createMock(Property.class);
- expect(propertyChangeNumber.getKey()).andReturn("changeNumber")
- .anyTimes();
+ expect(propertyChangeNumber.getKey()).andReturn("changeNumber").anyTimes();
expect(propertyChangeNumber.getValue()).andReturn("4711").anyTimes();
properties.add(propertyChangeNumber);
Property propertySubmitterName = createMock(Property.class);
- expect(propertySubmitterName.getKey()).andReturn("restorerName")
- .anyTimes();
+ expect(propertySubmitterName.getKey()).andReturn("restorerName").anyTimes();
expect(propertySubmitterName.getValue()).andReturn("John Doe").anyTimes();
properties.add(propertySubmitterName);
- Property propertyChangeUrl= createMock(Property.class);
+ Property propertyChangeUrl = createMock(Property.class);
expect(propertyChangeUrl.getKey()).andReturn("changeUrl").anyTimes();
- expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change")
- .anyTimes();
+ expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change").anyTimes();
properties.add(propertyChangeUrl);
- expect(its.createLinkForWebui("http://example.org/change",
- "http://example.org/change")).andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
+ expect(its.createLinkForWebui("http://example.org/change", "http://example.org/change"))
+ .andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
- its.addComment("176", "Change 4711 restored by John Doe:\n" +
- "Test-Change-Subject\n" +
- "\n" +
- "Reason:\n" +
- "Test-Reason\n" +
- "\n" +
- "HtTp://ExAmPlE.OrG/ChAnGe");
+ its.addComment(
+ "176",
+ "Change 4711 restored by John Doe:\n"
+ + "Test-Change-Subject\n"
+ + "\n"
+ + "Reason:\n"
+ + "Test-Reason\n"
+ + "\n"
+ + "HtTp://ExAmPlE.OrG/ChAnGe");
replayMocks();
Action action = injector.getInstance(AddStandardComment.class);
@@ -271,31 +266,30 @@
properties.add(propertySubject);
Property propertyChangeNumber = createMock(Property.class);
- expect(propertyChangeNumber.getKey()).andReturn("changeNumber")
- .anyTimes();
+ expect(propertyChangeNumber.getKey()).andReturn("changeNumber").anyTimes();
expect(propertyChangeNumber.getValue()).andReturn("4711").anyTimes();
properties.add(propertyChangeNumber);
Property propertySubmitterName = createMock(Property.class);
- expect(propertySubmitterName.getKey()).andReturn("uploaderName")
- .anyTimes();
+ expect(propertySubmitterName.getKey()).andReturn("uploaderName").anyTimes();
expect(propertySubmitterName.getValue()).andReturn("John Doe").anyTimes();
properties.add(propertySubmitterName);
- Property propertyChangeUrl= createMock(Property.class);
+ Property propertyChangeUrl = createMock(Property.class);
expect(propertyChangeUrl.getKey()).andReturn("changeUrl").anyTimes();
- expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change")
- .anyTimes();
+ expect(propertyChangeUrl.getValue()).andReturn("http://example.org/change").anyTimes();
properties.add(propertyChangeUrl);
- expect(its.createLinkForWebui("http://example.org/change",
- "http://example.org/change")).andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
+ expect(its.createLinkForWebui("http://example.org/change", "http://example.org/change"))
+ .andReturn("HtTp://ExAmPlE.OrG/ChAnGe");
- its.addComment("176", "Change 4711 had a related patch set uploaded by " +
- "John Doe:\n" +
- "Test-Change-Subject\n" +
- "\n" +
- "HtTp://ExAmPlE.OrG/ChAnGe");
+ its.addComment(
+ "176",
+ "Change 4711 had a related patch set uploaded by "
+ + "John Doe:\n"
+ + "Test-Change-Subject\n"
+ + "\n"
+ + "HtTp://ExAmPlE.OrG/ChAnGe");
replayMocks();
Action action = injector.getInstance(AddStandardComment.class);
@@ -316,4 +310,4 @@
bind(ItsFacade.class).toInstance(its);
}
}
-}
\ No newline at end of file
+}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEventTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEventTest.java
index 3d147a9..99fa2c3 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEventTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/base/workflow/action/LogEventTest.java
@@ -19,16 +19,13 @@
import com.google.gerrit.extensions.config.FactoryModule;
import com.google.inject.Guice;
import com.google.inject.Injector;
-
import com.googlesource.gerrit.plugins.its.base.testutil.LoggingMockingTestCase;
import com.googlesource.gerrit.plugins.its.base.workflow.ActionRequest;
import com.googlesource.gerrit.plugins.its.base.workflow.Property;
-
-import org.apache.log4j.Level;
-
import java.io.IOException;
import java.util.HashSet;
import java.util.Set;
+import org.apache.log4j.Level;
public class LogEventTest extends LoggingMockingTestCase {
private Injector injector;
@@ -40,7 +37,7 @@
replayMocks();
LogEvent logEvent = createLogEvent();
- logEvent.execute("4711", actionRequest, new HashSet<Property>());
+ logEvent.execute("4711", actionRequest, new HashSet<>());
}
public void testLevelDefault() throws IOException {
@@ -117,7 +114,6 @@
ActionRequest actionRequest = createMock(ActionRequest.class);
expect(actionRequest.getParameter(1)).andReturn("info");
-
Set<Property> properties = Sets.newHashSet();
properties.add(new PropertyMock("KeyA", "ValueA", "PropertyA"));
properties.add(new PropertyMock("KeyB", "ValueB", "PropertyB"));
@@ -144,8 +140,7 @@
private class TestModule extends FactoryModule {
@Override
- protected void configure() {
- }
+ protected void configure() {}
}
private class PropertyMock extends Property {
@@ -161,4 +156,4 @@
return toString;
}
}
-}
\ No newline at end of file
+}
