WS client should consider codes other than 200 for success

1 files changed, 22 insertions, 0 deletions

diff --git a/sonar-ws-client/src/test/java/org/sonar/wsclient/internal/HttpRequestFactoryTest.java b/sonar-ws-client/src/test/java/org/sonar/wsclient/internal/HttpRequestFactoryTest.java
index ff870ec5938..c0f8c726cc3 100644
--- a/sonar-ws-client/src/test/java/org/sonar/wsclient/internal/HttpRequestFactoryTest.java
+++ b/sonar-ws-client/src/test/java/org/sonar/wsclient/internal/HttpRequestFactoryTest.java
@@ -76,6 +76,28 @@ public class HttpRequestFactoryTest {
   }
 
   @Test
+  public void post_successful_if_created() {
+    httpServer.stubStatusCode(201).stubResponseBody("{}");
+
+    HttpRequestFactory factory = new HttpRequestFactory(httpServer.url());
+    String json = factory.post("/api/issues/change", Collections.<String, Object>emptyMap());
+
+    assertThat(json).isEqualTo("{}");
+    assertThat(httpServer.requestedPath()).isEqualTo("/api/issues/change");
+  }
+
+  @Test
+  public void post_successful_if_no_content() {
+    httpServer.stubStatusCode(204).stubResponseBody("");
+
+    HttpRequestFactory factory = new HttpRequestFactory(httpServer.url());
+    String json = factory.post("/api/issues/change", Collections.<String, Object>emptyMap());
+
+    assertThat(json).isEqualTo("");
+    assertThat(httpServer.requestedPath()).isEqualTo("/api/issues/change");
+  }
+
+  @Test
   public void test_authentication() {
     httpServer.stubStatusCode(200).stubResponseBody("{}");