diff options
| author | Julien Lancelot <julien.lancelot@gmail.com> | 2014-02-25 11:24:45 +0100 |
|---|---|---|
| committer | Julien Lancelot <julien.lancelot@gmail.com> | 2014-02-25 19:02:54 +0100 |
| commit | f359a4283c663ebff7416bb003895ebde323e8d0 (patch) | |
| tree | bfcd66f17dbfb17cf5de057908c1ec833b178268 /sonar-batch/src/test/java | |
| parent | b102e79cb3bf76e6d5eab14fd767d967c2fb84e6 (diff) | |
| download | sonarqube-f359a4283c663ebff7416bb003895ebde323e8d0.tar.gz sonarqube-f359a4283c663ebff7416bb003895ebde323e8d0.zip | |
SONAR-4996 Save debt on issues and measures in minutes (instead of seconds)
Diffstat (limited to 'sonar-batch/src/test/java')
| -rw-r--r-- | sonar-batch/src/test/java/org/sonar/batch/debt/RuleDebtCalculatorTest.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sonar-batch/src/test/java/org/sonar/batch/debt/RuleDebtCalculatorTest.java b/sonar-batch/src/test/java/org/sonar/batch/debt/RuleDebtCalculatorTest.java index f045ae3d3db..96ecfd2edf0 100644 --- a/sonar-batch/src/test/java/org/sonar/batch/debt/RuleDebtCalculatorTest.java +++ b/sonar-batch/src/test/java/org/sonar/batch/debt/RuleDebtCalculatorTest.java @@ -66,7 +66,7 @@ public class RuleDebtCalculatorTest { .setOffsetUnit(WorkDuration.UNIT.MINUTES); when(model.requirementsByRule(ruleKey)).thenReturn(requirement); - assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(15 * 60); + assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(15L); } @Test @@ -82,7 +82,7 @@ public class RuleDebtCalculatorTest { .setOffsetUnit(WorkDuration.UNIT.MINUTES); when(model.requirementsByRule(ruleKey)).thenReturn(requirement); - assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(((10 * 2) + 5) * 60); + assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(((10 * 2) + 5)); } @Test @@ -96,7 +96,7 @@ public class RuleDebtCalculatorTest { .setFactorUnit(WorkDuration.UNIT.HOURS); when(model.requirementsByRule(ruleKey)).thenReturn(requirement); - assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo((10 * 2) * 60 * 60); + assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo((10 * 2) * 60); } @Test @@ -111,7 +111,7 @@ public class RuleDebtCalculatorTest { when(model.requirementsByRule(ruleKey)).thenReturn(requirement); - assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(5 * HOURS_IN_DAY * 60 * 60); + assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(5 * HOURS_IN_DAY * 60); } @Test @@ -135,7 +135,7 @@ public class RuleDebtCalculatorTest { when(model.requirementsByRule(ruleKey)).thenReturn(requirement); try { - assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(15 * 60); + assertThat(calculator.calculateTechnicalDebt(issue.ruleKey(), issue.effortToFix())).isEqualTo(15); fail(); } catch (Exception e) { assertThat(e).isInstanceOf(IllegalArgumentException.class) |