commit 70cdaa8137c3a8b335883729a908e02163c9feef
author Patrick Hiesel <hiesel@google.com> Fri Oct 25 15:18:57 2019 +0200
committer Patrick Hiesel <hiesel@google.com> Fri Oct 25 15:18:57 2019 +0200
tree b0a7ad80d91dd77147c55f89f2527405e53d9dce
parent 3935b8ab77f7215391066b2c3e4adda06b32fe53

Expect API field to be null

Gerrit does not populate lists in the API when they have no entries.
This commit makes automerger cope with one instance of that.

Change-Id: Ia8f434870ae29897f91513a9ad14b508464d98fc

src/main/java/com/googlesource/gerrit/plugins/automerger/DownstreamCreator.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/automerger/DownstreamCreator.java b/src/main/java/com/googlesource/gerrit/plugins/automerger/DownstreamCreator.java
index 84ae7c3..b61675c 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/automerger/DownstreamCreator.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/automerger/DownstreamCreator.java
@@ -214,7 +214,7 @@
             ChangeInfo downstreamChange =
                 gApi.changes().id(changeNumber).get(EnumSet.of(ListChangesOption.CURRENT_REVISION));
             for (Map.Entry<String, LabelInfo> labelEntry : labels.entrySet()) {
-              if (labelEntry.getValue().all.size() > 0) {
+              if (labelEntry.getValue().all != null && labelEntry.getValue().all.size() > 0) {
                 OptionalInt maxVote =
                     labelEntry.getValue().all.stream()
                         .filter(o -> o.value != null)