PropertyExtractor: Add null check for event.submitter
If the change was open, it would cause event.submitter to be
null. Fix this by adding a null check.
Gerrit's core does the same thing.
Bug: Issue 8051
Change-Id: I0690e184ba50ea1e60176e8be5af34c43a80393e
(cherry picked from commit d4659620aa109a9d9ea6641d18e9e473117cd22d)
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 03bff09..b83f1c6 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
@@ -116,7 +116,9 @@
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"));
+ if (event.submitter != null) {
+ common.addAll(propertyAttributeExtractor.extractFrom(event.submitter.get(), "submitter"));
+ }
common.addAll(propertyAttributeExtractor.extractFrom(event.refUpdate.get()));
return issueExtractor.getIssueIds(event.getProjectNameKey().get(),
event.refUpdate.get().newRev);
