commit 84c6aff5aa4f463146fd9e530404c30e28e0d68d
author Edwin Kempin <ekempin@google.com> Thu Jan 28 14:05:34 2021 +0100
committer Edwin Kempin <ekempin@google.com> Thu Jan 28 15:54:24 2021 +0100
tree ac8d6680af9883fc23649c4a9a78661956192863
parent c28be2854c8331b521e7b0484529a64f405b7b33

CodeOwnersOnPostReview: Abort when illegal state was reached

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I3ec99722fcb546ade0b6dded8d43c5898a83ea6c

java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java

diff --git a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java
index f53d831..1596e23 100644
--- a/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java
+++ b/java/com/google/gerrit/plugins/codeowners/backend/CodeOwnersOnPostReview.java
@@ -196,6 +196,7 @@
           oldApprovals,
           approvals,
           requiredApproval);
+      return Optional.empty();
     }
 
     if (ownedPaths.size() <= maxPathsInChangeMessage) {