commit 0a41400e2be7c88779ddbe59f51ca8590c4e6e10
author David Pursehouse <dpursehouse@collab.net> Wed Jun 20 14:08:47 2018 +0900
committer David Pursehouse <dpursehouse@collab.net> Thu Jun 21 08:52:53 2018 +0900
tree 0a6ee145639173fd2d0a1e4782f94c3af47ba1de
parent bd91e340d8149700dd4e6c762f314b77cb25ba02

BackupRef: Add missing cases in switch on RefUpdate.Result

Handle all the other cases as an unexpected result.

Adust the log message to include the value.

Change-Id: I7e2e534c2f26f518f1675a6b9820483b46912e86

src/main/java/com/googlesource/gerrit/plugins/refprotection/BackupRef.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/refprotection/BackupRef.java b/src/main/java/com/googlesource/gerrit/plugins/refprotection/BackupRef.java
index d88df07..f3777f3 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/refprotection/BackupRef.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/refprotection/BackupRef.java
@@ -141,9 +141,14 @@
             case REJECTED:
               log.error("Tag already exists while creating backup tag");
               break;
-
+            case FAST_FORWARD:
+            case IO_FAILURE:
+            case NOT_ATTEMPTED:
+            case NO_CHANGE:
+            case REJECTED_CURRENT_BRANCH:
+            case RENAMED:
             default:
-              log.error("Unknown error while creating backup tag");
+              log.error("Unexpected result while creating backup tag: {}", result);
           }
         } else {
           BranchInput input = new BranchInput();