SONAR-3900 dependsUponMetrics should also return fallbackMetric

2 files changed, 7 insertions, 1 deletions

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/measures/AverageFormula.java b/sonar-plugin-api/src/main/java/org/sonar/api/measures/AverageFormula.java
index 59869be6eac..3e8ff97a3a6 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/measures/AverageFormula.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/measures/AverageFormula.java
@@ -71,7 +71,7 @@ public class AverageFormula implements Formula {
    * {@inheritDoc}
    */
   public List<Metric> dependsUponMetrics() {
-    return Arrays.asList(mainMetric, byMetric);
+    return Arrays.asList(mainMetric, fallbackMetric, byMetric);
   }
 
   /**
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/measures/AverageFormulaTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/measures/AverageFormulaTest.java
index 8cabe3cf741..6ecc868d33e 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/measures/AverageFormulaTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/measures/AverageFormulaTest.java
@@ -52,6 +52,12 @@ public class AverageFormulaTest {
   }
 
   @Test
+  public void test_depends_upon_falback_metric() throws Exception {
+    AverageFormula formula = AverageFormula.create(CoreMetrics.COMPLEXITY_IN_FUNCTIONS, CoreMetrics.FUNCTIONS).setFallbackForMainMetric(CoreMetrics.COMPLEXITY);
+    assertThat(formula.dependsUponMetrics(), hasItems(CoreMetrics.COMPLEXITY_IN_FUNCTIONS, CoreMetrics.COMPLEXITY, CoreMetrics.FUNCTIONS));
+  }
+
+  @Test
   public void test_average_calculation() {
     List<FormulaData> childrenData = newArrayList();
     FormulaData data1 = mock(FormulaData.class);