commit 894753c7cd4fb0e41c78d223545e07a2f462331a
author Edwin Kempin <ekempin@google.com> Fri Jan 08 13:40:57 2021 +0100
committer Edwin Kempin <ekempin@google.com> Fri Jan 15 11:51:00 2021 +0100
tree e7ec3e148961f21782f72310b31c7afc44bfbec2
parent 6c5c1b849769daf3c15337da2f87f18b3e782adf

Test that the RenameEmail REST endpoint requires authentication

This slightly increases the test coverage from 98.4% to 100%.

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

javatests/com/google/gerrit/plugins/codeowners/acceptance/api/RenameEmailIT.java

diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/RenameEmailIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/RenameEmailIT.java
index 74e0d52..40d87a6 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/RenameEmailIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/api/RenameEmailIT.java
@@ -143,6 +143,15 @@
   }
 
   @Test
+  public void requiresAuthenticatedUser() throws Exception {
+    requestScopeOperations.setApiUserAnonymous();
+    AuthException authException =
+        assertThrows(
+            AuthException.class, () -> renameEmail(project, "master", new RenameEmailInput()));
+    assertThat(authException).hasMessageThat().contains("Authentication required");
+  }
+
+  @Test
   public void renameEmailRequiresDirectPushPermissionsForNonProjectOwner() throws Exception {
     String secondaryEmail = "user-foo@example.com";
     accountOperations.account(user.id()).forUpdate().addSecondaryEmail(secondaryEmail).update();