From: Julien Lancelot <julien.lancelot@gmail.com>
Date: Wed, 16 Jan 2013 09:27:22 +0000 (+0100)
Subject: SONAR-2501 Do notAdd unit test
X-Git-Tag: 3.5~398
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1c61a5009ed00769cc9bfe7226346e6a06ca7086;p=sonarqube.git

SONAR-2501 Do notAdd unit test
---

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/tests/FileTest.java b/sonar-plugin-api/src/main/java/org/sonar/api/tests/FileTest.java
index 1d31426600c..223a7bc3807 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/tests/FileTest.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/tests/FileTest.java
@@ -50,29 +50,6 @@ public class FileTest {
     tests.add(test);
   }
 
-  @Override
-  public boolean equals(Object o) {
-    if (this == o) {
-      return true;
-    }
-    if (o == null || getClass() != o.getClass()) {
-      return false;
-    }
-
-    FileTest fileTest = (FileTest) o;
-
-    if (!key.equals(fileTest.key)) {
-      return false;
-    }
-
-    return true;
-  }
-
-  @Override
-  public int hashCode() {
-    return key.hashCode();
-  }
-
   @Override
   public String toString() {
     return new ToStringBuilder(this, ToStringStyle.SHORT_PREFIX_STYLE)
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/tests/Test.java b/sonar-plugin-api/src/main/java/org/sonar/api/tests/Test.java
index 513bc2f6578..4ccdf23ca3c 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/tests/Test.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/tests/Test.java
@@ -71,29 +71,6 @@ public class Test {
     return this;
   }
 
-  @Override
-  public boolean equals(Object o) {
-    if (this == o) {
-      return true;
-    }
-    if (o == null || getClass() != o.getClass()) {
-      return false;
-    }
-
-    Test test = (Test) o;
-
-    if (!name.equals(test.name)) {
-      return false;
-    }
-
-    return true;
-  }
-
-  @Override
-  public int hashCode() {
-    return name.hashCode();
-  }
-
   @Override
   public String toString() {
     return new ToStringBuilder(this, ToStringStyle.SHORT_PREFIX_STYLE)
diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/tests/ProjectTestsImplTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/tests/ProjectTestsImplTest.java
index dc0585c4f0c..19c6287a6ec 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/tests/ProjectTestsImplTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/tests/ProjectTestsImplTest.java
@@ -23,6 +23,7 @@ package org.sonar.api.tests;
 import org.junit.Before;
 import org.junit.Test;
 
+import static com.google.common.collect.Lists.newArrayList;
 import static org.fest.assertions.Assertions.assertThat;
 
 public class ProjectTestsImplTest {
@@ -36,7 +37,12 @@ public class ProjectTestsImplTest {
 
   @Test
   public void should_add_new_test() throws Exception {
-    org.sonar.api.tests.Test test = new org.sonar.api.tests.Test("test").setStatus("ok");
+    org.sonar.api.tests.Test test = new org.sonar.api.tests.Test("test")
+        .setStatus("ok")
+        .setMessage("message")
+        .setStackTrace("stacktrace")
+        .setDurationMilliseconds(10)
+    ;
     projectTests.addTest("key", test);
     assertThat(projectTests.getFileTests()).hasSize(1);
     assertThat(projectTests.getFileTests().get(0).getTests()).hasSize(1);
@@ -53,4 +59,12 @@ public class ProjectTestsImplTest {
     assertThat(projectTests.getFileTests()).hasSize(1);
     assertThat(projectTests.getFileTests().get(0).getTests()).hasSize(2);
   }
+
+  @Test
+  public void should_add_coverage_info() throws Exception {
+    org.sonar.api.tests.Test test = new org.sonar.api.tests.Test("test").setStatus("ok");
+    projectTests.addTest("key", test);
+
+    projectTests.cover("key", "test", "mainFile", newArrayList(1, 2));
+  }
 }