diff options
author | Julien Lancelot <julien.lancelot@sonarsource.com> | 2014-07-03 17:29:23 +0200 |
---|---|---|
committer | Julien Lancelot <julien.lancelot@sonarsource.com> | 2014-07-03 17:51:11 +0200 |
commit | 5b9dfc041ae3355dc68dcc4cb0a97b76d33dc106 (patch) | |
tree | 99472a4ac8eac0b272c0941a9765c1ba977fd8e6 /sonar-deprecated | |
parent | 10244be8668b4adba0dbdfcb9488a235eedabb6e (diff) | |
download | sonarqube-5b9dfc041ae3355dc68dcc4cb0a97b76d33dc106.tar.gz sonarqube-5b9dfc041ae3355dc68dcc4cb0a97b76d33dc106.zip |
SONAR-5432 Remove RCI and Weighted issues metrics
Diffstat (limited to 'sonar-deprecated')
-rw-r--r-- | sonar-deprecated/src/main/java/org/sonar/api/rules/RuleUtils.java | 57 | ||||
-rw-r--r-- | sonar-deprecated/src/test/java/org/sonar/api/rules/RuleUtilsTest.java | 64 |
2 files changed, 0 insertions, 121 deletions
diff --git a/sonar-deprecated/src/main/java/org/sonar/api/rules/RuleUtils.java b/sonar-deprecated/src/main/java/org/sonar/api/rules/RuleUtils.java deleted file mode 100644 index ba3f02bacc8..00000000000 --- a/sonar-deprecated/src/main/java/org/sonar/api/rules/RuleUtils.java +++ /dev/null @@ -1,57 +0,0 @@ -/* - * SonarQube, open source software quality management tool. - * Copyright (C) 2008-2014 SonarSource - * mailto:contact AT sonarsource DOT com - * - * SonarQube is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public - * License as published by the Free Software Foundation; either - * version 3 of the License, or (at your option) any later version. - * - * SonarQube is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public License - * along with this program; if not, write to the Free Software Foundation, - * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. - */ -package org.sonar.api.rules; - -import java.util.Map; - -import org.apache.commons.configuration.Configuration; -import org.sonar.api.CoreProperties; -import org.sonar.api.utils.KeyValueFormat; - -/** - * A utility class to manipulate concepts around rules - * @deprecated in 3.7. Commons Configuration must be replaced by {@link org.sonar.api.config.Settings} - */ -@Deprecated -public final class RuleUtils { - - private RuleUtils() { - } - - /** - * Gets a Map<RulePriority, Integer> containing the weights defined in the settings - * Default value is used when the property is not set (see property key and default value in the class CoreProperties) - * - * @param configuration the Sonar configuration - * @return a map - */ - public static Map<RulePriority, Integer> getPriorityWeights(final Configuration configuration) { - String levelWeight = configuration.getString(CoreProperties.CORE_RULE_WEIGHTS_PROPERTY, CoreProperties.CORE_RULE_WEIGHTS_DEFAULT_VALUE); - - Map<RulePriority, Integer> weights = KeyValueFormat.parse(levelWeight, new KeyValueFormat.RulePriorityNumbersPairTransformer()); - - for (RulePriority priority : RulePriority.values()) { - if (!weights.containsKey(priority)) { - weights.put(priority, 1); - } - } - return weights; - } -} diff --git a/sonar-deprecated/src/test/java/org/sonar/api/rules/RuleUtilsTest.java b/sonar-deprecated/src/test/java/org/sonar/api/rules/RuleUtilsTest.java deleted file mode 100644 index 643f673f89c..00000000000 --- a/sonar-deprecated/src/test/java/org/sonar/api/rules/RuleUtilsTest.java +++ /dev/null @@ -1,64 +0,0 @@ -/* - * SonarQube, open source software quality management tool. - * Copyright (C) 2008-2014 SonarSource - * mailto:contact AT sonarsource DOT com - * - * SonarQube is free software; you can redistribute it and/or - * modify it under the terms of the GNU Lesser General Public - * License as published by the Free Software Foundation; either - * version 3 of the License, or (at your option) any later version. - * - * SonarQube is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - * Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public License - * along with this program; if not, write to the Free Software Foundation, - * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. - */ -package org.sonar.api.rules; - -import org.apache.commons.configuration.Configuration; -import org.junit.Test; -import org.mockito.Matchers; -import org.sonar.api.CoreProperties; - -import java.util.Map; - -import static org.fest.assertions.Assertions.assertThat; -import static org.mockito.Matchers.anyString; -import static org.mockito.Mockito.mock; -import static org.mockito.Mockito.when; - -public class RuleUtilsTest { - - @Test - public void getPriorityWeights() { - Configuration conf = mock(Configuration.class); - when(conf.getString(Matchers.eq(CoreProperties.CORE_RULE_WEIGHTS_PROPERTY), anyString())).thenReturn("info=0;minor=1;major=2;critical=5;blocker=10"); - - final Map<RulePriority, Integer> map = RuleUtils.getPriorityWeights(conf); - - assertThat(map.get(RulePriority.BLOCKER)).isEqualTo(10); - assertThat(map.get(RulePriority.CRITICAL)).isEqualTo(5); - assertThat(map.get(RulePriority.MAJOR)).isEqualTo(2); - assertThat(map.get(RulePriority.MINOR)).isEqualTo(1); - assertThat(map.get(RulePriority.INFO)).isEqualTo(0); - } - - @Test - public void loadMissingWeights() { - Configuration conf = mock(Configuration.class); - when(conf.getString(Matchers.eq(CoreProperties.CORE_RULE_WEIGHTS_PROPERTY), anyString())).thenReturn("foo=0;bar=1;CRITICAL=5"); - - final Map<RulePriority, Integer> map = RuleUtils.getPriorityWeights(conf); - - assertThat(map.get(RulePriority.BLOCKER)).isEqualTo(1); - assertThat(map.get(RulePriority.CRITICAL)).isEqualTo(5); - assertThat(map.get(RulePriority.MAJOR)).isEqualTo(1); - assertThat(map.get(RulePriority.MINOR)).isEqualTo(1); - assertThat(map.get(RulePriority.INFO)).isEqualTo(1); - } - -} |