Complete Debt medium test

diff --git a/sonar-server/src/test/java/org/sonar/server/debt/DebtMediumTest.java b/sonar-server/src/test/java/org/sonar/server/debt/DebtMediumTest.java
index d0b5397060bdd09161f42e15045593a7b1356749..810a7a111db663c2bb532648acedbe8e716bb621 100644 (file)
--- a/sonar-server/src/test/java/org/sonar/server/debt/DebtMediumTest.java
+++ b/sonar-server/src/test/java/org/sonar/server/debt/DebtMediumTest.java
@@ -24,6 +24,7 @@ import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 import org.sonar.api.server.debt.DebtCharacteristic;
+import org.sonar.api.server.debt.internal.DefaultDebtCharacteristic;
 import org.sonar.core.permission.GlobalPermissions;
 import org.sonar.server.tester.ServerTester;
 import org.sonar.server.user.MockUserSession;
@@ -60,8 +61,33 @@ public class DebtMediumTest {
     MockUserSession.set().setGlobalPermissions(GlobalPermissions.SYSTEM_ADMIN);
 
     DebtModelService debtModelService = serverTester.get(DebtModelService.class);
+    int nb = debtModelService.characteristics().size();
+
     DebtCharacteristic result = debtModelService.create("New characteristic", null);
 
+    assertThat(result.name()).isEqualTo("New characteristic");
+    assertThat(result.key()).isEqualTo("NEW_CHARACTERISTIC");
+    assertThat(result.isSub()).isFalse();
+    assertThat(result.order()).isEqualTo(nb + 1);
+
+    assertThat(debtModelService.characteristicByKey(result.key())).isNotNull();
+  }
+
+  @Test
+  public void create_sub_characteristic() throws Exception {
+    MockUserSession.set().setGlobalPermissions(GlobalPermissions.SYSTEM_ADMIN);
+
+    DebtModelService debtModelService = serverTester.get(DebtModelService.class);
+
+    DefaultDebtCharacteristic parent = (DefaultDebtCharacteristic) debtModelService.characteristicByKey("REUSABILITY");
+
+    DebtCharacteristic result = debtModelService.create("New characteristic", parent.id());
+
+    assertThat(result.name()).isEqualTo("New characteristic");
+    assertThat(result.key()).isEqualTo("NEW_CHARACTERISTIC");
+    assertThat(result.isSub()).isTrue();
+    assertThat(result.order()).isNull();
+
     assertThat(debtModelService.characteristicByKey(result.key())).isNotNull();
   }