Add failing test for header parameter encoding bug (#1068)

palantir/conjure#288
palantir · Apr 24, 2019 · a0934e6 · a0934e6
1 parent 39ef706
commit a0934e6
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/...est/java/com/palantir/conjure/java/client/jaxrs/JaxRsClientJava8OptionalHandlingTest.java b/...est/java/com/palantir/conjure/java/client/jaxrs/JaxRsClientJava8OptionalHandlingTest.java
@@ -34,6 +34,7 @@
 import okhttp3.mockwebserver.MockWebServer;
 import okhttp3.mockwebserver.RecordedRequest;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 
@@ -167,6 +168,15 @@ public void testStringHeader() throws Exception {
         assertThat(takeRequest.getHeader("req"), is("str2"));
     }
 
+    @Ignore("TODO(rfink): Add support for header encoding")
+    @Test
+    public void testStringHeader_withNonAsciiCharacters() throws Exception {
+        proxy.header(Optional.of("ü"), "ø");
+        RecordedRequest takeRequest = server.takeRequest();
+        assertThat(takeRequest.getHeader("opt"), is("ü"));
+        assertThat(takeRequest.getHeader("req"), is("ø"));
+    }
+
     @Test
     public void testAbsentIntQuery() throws Exception {
         proxy.queryInt(OptionalInt.empty(), "str2");