Decrease two methods complexity

2 files changed, 18 insertions, 13 deletions

diff --git a/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/timemachine/Periods.java b/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/timemachine/Periods.java
index c8424b62163..74d47c624e6 100644
--- a/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/timemachine/Periods.java
+++ b/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/timemachine/Periods.java
@@ -43,31 +43,30 @@ public class Periods implements BatchExtension {
     String param = snapshot.getPeriodModeParameter(periodIndex);
     Date date = snapshot.getPeriodDate(periodIndex);
 
+    String label = "";
     if (mode.equals(CoreProperties.TIMEMACHINE_MODE_DAYS)) {
-      return message("over_x_days", param);
+      label = message("over_x_days", param);
     } else if (mode.equals(CoreProperties.TIMEMACHINE_MODE_VERSION)) {
+      label =  message("since_version", param);
       if (date != null) {
-        return message("since_version_detailed", param, convertDate(date));
-      } else {
-        return message("since_version", param);
+        label =  message("since_version_detailed", param, convertDate(date));
       }
     } else if (mode.equals(CoreProperties.TIMEMACHINE_MODE_PREVIOUS_ANALYSIS)) {
+      label =  message("since_previous_analysis");
       if (date != null) {
-        return message("since_previous_analysis_detailed", convertDate(date));
-      } else {
-        return message("since_previous_analysis");
+        label =  message("since_previous_analysis_detailed", convertDate(date));
       }
     } else if (mode.equals(CoreProperties.TIMEMACHINE_MODE_PREVIOUS_VERSION)) {
+      label =  message("since_previous_version");
       if (param != null) {
-        return message("since_previous_version_detailed", param);
-      } else {
-        return message("since_previous_version");
+        label =  message("since_previous_version_detailed", param);
       }
     } else if (mode.equals(CoreProperties.TIMEMACHINE_MODE_DATE)) {
-      return message("since_x", convertDate(date));
+      label =  message("since_x", convertDate(date));
     } else {
       throw new IllegalStateException("This mode is not supported : " + mode);
     }
+    return label;
   }
 
   private String message(String key, Object... parameters) {
diff --git a/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/timemachine/PeriodsTest.java b/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/timemachine/PeriodsTest.java
index da6aa2c623d..b8a3f04905a 100644
--- a/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/timemachine/PeriodsTest.java
+++ b/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/timemachine/PeriodsTest.java
@@ -78,7 +78,7 @@ public class PeriodsTest {
   }
 
   @Test
-     public void shouldReturnLabelInModePreviousAnalysis() {
+     public void shouldReturnLabelInModePreviousAnalysisWithDateNotNull() {
     when(snapshot.getPeriodMode(periodIndex)).thenReturn(CoreProperties.TIMEMACHINE_MODE_VERSION);
     when(snapshot.getPeriodDate(periodIndex)).thenReturn(new Date());
     when(snapshot.getPeriodModeParameter(periodIndex)).thenReturn(param);
@@ -86,7 +86,10 @@ public class PeriodsTest {
     periods.getLabel(periodIndex);
 
     verify(i18n).message(Mockito.any(Locale.class), Mockito.eq("since_version_detailed"), Mockito.isNull(String.class), Mockito.eq(param), Mockito.anyString());
+  }
 
+  @Test
+  public void shouldReturnLabelInModePreviousAnalysisWithNullDate() {
     when(snapshot.getPeriodMode(periodIndex)).thenReturn(CoreProperties.TIMEMACHINE_MODE_VERSION);
     when(snapshot.getPeriodDate(periodIndex)).thenReturn(null);
     when(snapshot.getPeriodModeParameter(periodIndex)).thenReturn(param);
@@ -97,14 +100,17 @@ public class PeriodsTest {
   }
 
   @Test
-  public void shouldReturnLabelInModePreviousVersion() {
+  public void shouldReturnLabelInModePreviousVersionWithParamNotNull() {
     when(snapshot.getPeriodMode(periodIndex)).thenReturn(CoreProperties.TIMEMACHINE_MODE_PREVIOUS_VERSION);
     when(snapshot.getPeriodModeParameter(periodIndex)).thenReturn(param);
 
     periods.getLabel(periodIndex);
 
     verify(i18n).message(Mockito.any(Locale.class), Mockito.eq("since_previous_version_detailed"), Mockito.isNull(String.class), Mockito.eq(param));
+  }
 
+  @Test
+  public void shouldReturnLabelInModePreviousVersionWithParamNull() {
     when(snapshot.getPeriodMode(periodIndex)).thenReturn(CoreProperties.TIMEMACHINE_MODE_PREVIOUS_VERSION);
     when(snapshot.getPeriodModeParameter(periodIndex)).thenReturn(null);