]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-13398 deprecate issue count fields in response for api/project_pull_requests...
authorJacek <jacek.poreda@sonarsource.com>
Tue, 9 Jun 2020 13:16:56 +0000 (15:16 +0200)
committersonartech <sonartech@sonarsource.com>
Fri, 26 Jun 2020 20:04:57 +0000 (20:04 +0000)
server/sonar-webserver-webapi/src/main/java/org/sonar/server/branch/pr/ws/ListAction.java
server/sonar-webserver-webapi/src/test/java/org/sonar/server/branch/pr/ws/ListActionTest.java

index 8acbd6e396fb1936293ef7ee988cc435f4a19c55..c9d484de6945d8b5283e01a50a4e6b11cb0376fb 100644 (file)
@@ -26,6 +26,7 @@ import java.util.Optional;
 import java.util.function.Function;
 import javax.annotation.Nullable;
 import org.apache.commons.lang.StringUtils;
+import org.sonar.api.server.ws.Change;
 import org.sonar.api.server.ws.Request;
 import org.sonar.api.server.ws.Response;
 import org.sonar.api.server.ws.WebService;
@@ -84,8 +85,9 @@ public class ListAction implements PullRequestWsAction {
         "<li>'Execute Analysis' rights on the specified project</li>" +
         "</ul>")
       .setResponseExample(getClass().getResource("list-example.json"))
-      .setHandler(this);
-
+      .setHandler(this)
+      .setChangelog(
+        new Change("8.4", "Response fields: 'bugs', 'vulnerabilities', 'codeSmells' are deprecated."));
     addProjectParam(action);
   }
 
index 0f81c264826b423baa3fe2dab152b240f3e2e820..4d123a46de49d26dcbe62c0d535b0a57d3c1967d 100644 (file)
@@ -24,6 +24,7 @@ import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.ExpectedException;
 import org.sonar.api.resources.ResourceTypes;
+import org.sonar.api.server.ws.Change;
 import org.sonar.api.server.ws.WebService;
 import org.sonar.api.utils.DateUtils;
 import org.sonar.api.utils.System2;
@@ -108,6 +109,10 @@ public class ListActionTest {
     assertThat(definition.isInternal()).isFalse();
     assertThat(definition.params()).extracting(WebService.Param::key).containsExactlyInAnyOrder("project");
     assertThat(definition.since()).isEqualTo("7.1");
+
+    assertThat(definition.changelog())
+      .extracting(Change::getVersion, Change::getDescription)
+      .contains(tuple("8.4", "Response fields: 'bugs', 'vulnerabilities', 'codeSmells' are deprecated."));
   }
 
   @Test