diff options
author | Teryk Bellahsene <teryk.bellahsene@sonarsource.com> | 2016-02-02 11:41:19 +0100 |
---|---|---|
committer | Teryk Bellahsene <teryk.bellahsene@sonarsource.com> | 2016-02-02 11:41:19 +0100 |
commit | 152688e28380deb43cb5400b1ef0d54404fbc662 (patch) | |
tree | 3d3f1938f2f928a9a3d855580c89984c797c0086 /server | |
parent | d12f2e96085ec80308239f0777c0b0af80ec210e (diff) | |
download | sonarqube-152688e28380deb43cb5400b1ef0d54404fbc662.tar.gz sonarqube-152688e28380deb43cb5400b1ef0d54404fbc662.zip |
SONAR-7134 WS api/measures/* return empy measures array when there is no measure
Diffstat (limited to 'server')
2 files changed, 5 insertions, 5 deletions
diff --git a/server/sonar-server/src/main/java/org/sonar/server/measure/ws/ComponentDtoToWsComponent.java b/server/sonar-server/src/main/java/org/sonar/server/measure/ws/ComponentDtoToWsComponent.java index 4bbee6274f7..123dfdc9c97 100644 --- a/server/sonar-server/src/main/java/org/sonar/server/measure/ws/ComponentDtoToWsComponent.java +++ b/server/sonar-server/src/main/java/org/sonar/server/measure/ws/ComponentDtoToWsComponent.java @@ -43,7 +43,7 @@ class ComponentDtoToWsComponent { } for (Map.Entry<MetricDto, MeasureDto> entry : measuresByMetric.entrySet()) { - wsComponent.getMeasuresBuilder().addMeasures(measureDtoToWsMeasure(entry.getKey(), entry.getValue())); + wsComponent.addMeasures(measureDtoToWsMeasure(entry.getKey(), entry.getValue())); } return wsComponent; diff --git a/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java index b466058fdaa..2f6af91b770 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/measure/ws/ComponentTreeActionTest.java @@ -160,9 +160,9 @@ public class ComponentTreeActionTest { .setParam(PARAM_METRIC_KEYS, "ncloc,coverage") .setParam(PARAM_ADDITIONAL_FIELDS, ADDITIONAL_PERIODS)); - assertThat(response.getComponentsList().get(0).getMeasures().getMeasuresList()).extracting("metric").containsOnly("coverage"); + assertThat(response.getComponentsList().get(0).getMeasuresList()).extracting("metric").containsOnly("coverage"); // file measures - List<WsMeasures.Measure> fileMeasures = response.getComponentsList().get(1).getMeasures().getMeasuresList(); + List<WsMeasures.Measure> fileMeasures = response.getComponentsList().get(1).getMeasuresList(); assertThat(fileMeasures).extracting("metric").containsOnly("ncloc", "coverage"); assertThat(fileMeasures).extracting("value").containsOnly("5", "15.5"); assertThat(response.getPeriods().getPeriodsList()).extracting("mode").containsOnly("last_version", "last_analysis"); @@ -199,9 +199,9 @@ public class ComponentTreeActionTest { .setParam(PARAM_ADDITIONAL_FIELDS, "metrics")); // directory measures - assertThat(response.getComponentsList().get(0).getMeasures().getMeasuresList()).extracting("metric").containsOnly("coverage"); + assertThat(response.getComponentsList().get(0).getMeasuresList()).extracting("metric").containsOnly("coverage"); // file measures - List<WsMeasures.Measure> fileMeasures = response.getComponentsList().get(1).getMeasures().getMeasuresList(); + List<WsMeasures.Measure> fileMeasures = response.getComponentsList().get(1).getMeasuresList(); assertThat(fileMeasures).extracting("metric").containsOnly("ncloc", "coverage", "new_violations"); assertThat(fileMeasures).extracting("value").containsOnly("100", "15.5", ""); |