commit 372d1ed715b9f6aee0a2fcf059b25f6e6eb94f0f
author Edwin Kempin <ekempin@google.com> Wed Mar 25 17:00:32 2020 +0100
committer Edwin Kempin <ekempin@google.com> Wed Mar 25 17:00:32 2020 +0100
tree 7ca972ff9716269117a1e51a72cf8e8f0e4ccf04
parent 1055514463f1bcdade31379f0c5aff584bac97f9

GetArchive: Test request with unknown format

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

javatests/com/google/gerrit/acceptance/rest/change/GetArchiveIT.java

diff --git a/javatests/com/google/gerrit/acceptance/rest/change/GetArchiveIT.java b/javatests/com/google/gerrit/acceptance/rest/change/GetArchiveIT.java
index de949ea..7abd75a 100644
--- a/javatests/com/google/gerrit/acceptance/rest/change/GetArchiveIT.java
+++ b/javatests/com/google/gerrit/acceptance/rest/change/GetArchiveIT.java
@@ -18,6 +18,7 @@
 import static com.google.gerrit.testing.GerritJUnit.assertThrows;
 
 import com.google.gerrit.acceptance.AbstractDaemonTest;
+import com.google.gerrit.acceptance.RestResponse;
 import com.google.gerrit.extensions.client.ArchiveFormat;
 import com.google.gerrit.extensions.restapi.BadRequestException;
 import com.google.gerrit.extensions.restapi.MethodNotAllowedException;
@@ -42,6 +43,17 @@
   }
 
   @Test
+  public void unknownFormat() throws Exception {
+    // Test this by a REST call, since the Java API doesn't allow to specify an unknown format.
+    RestResponse res =
+        adminRestSession.get(
+            String.format(
+                "/changes/%s/revisions/current/archive?format=%s", changeId, "unknownFormat"));
+    res.assertBadRequest();
+    assertThat(res.getEntityContent()).isEqualTo("unknown archive format");
+  }
+
+  @Test
   public void zipFormatIsDisabled() throws Exception {
     MethodNotAllowedException ex =
         assertThrows(