summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSébastien Lesaint <sebastien.lesaint@sonarsource.com>2015-07-23 09:33:53 +0200
committerSébastien Lesaint <sebastien.lesaint@sonarsource.com>2015-07-23 10:30:10 +0200
commit51f8fa984e4a2c11ededba97e6b2c0a2fe29bb02 (patch)
treefce5813a9b4f40b094e0643c0ee15ae2f32f918f
parent7920e9f6efe19d0147e3360d927e3ffb5c6f3679 (diff)
downloadsonarqube-51f8fa984e4a2c11ededba97e6b2c0a2fe29bb02.tar.gz
sonarqube-51f8fa984e4a2c11ededba97e6b2c0a2fe29bb02.zip
fix some coverage flaws
-rw-r--r--server/sonar-server/src/main/java/org/sonar/server/computation/event/Event.java2
-rw-r--r--server/sonar-server/src/main/java/org/sonar/server/computation/qualityprofile/QualityProfile.java5
-rw-r--r--server/sonar-server/src/test/java/org/sonar/server/computation/event/EventTest.java6
-rw-r--r--server/sonar-server/src/test/java/org/sonar/server/computation/qualityprofile/QualityProfileTest.java11
4 files changed, 17 insertions, 7 deletions
diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/event/Event.java b/server/sonar-server/src/main/java/org/sonar/server/computation/event/Event.java
index 260be3c0474..e2a977e7d26 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/computation/event/Event.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/computation/event/Event.java
@@ -68,7 +68,7 @@ public class Event {
}
@Override
- public boolean equals(Object o) {
+ public boolean equals(@Nullable Object o) {
if (this == o) {
return true;
}
diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/qualityprofile/QualityProfile.java b/server/sonar-server/src/main/java/org/sonar/server/computation/qualityprofile/QualityProfile.java
index accfd11c1a0..bad693b7006 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/computation/qualityprofile/QualityProfile.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/computation/qualityprofile/QualityProfile.java
@@ -21,6 +21,7 @@ package org.sonar.server.computation.qualityprofile;
import com.google.common.base.Objects;
import java.util.Date;
+import javax.annotation.Nullable;
import javax.annotation.concurrent.Immutable;
import static java.util.Objects.requireNonNull;
@@ -60,7 +61,7 @@ public class QualityProfile {
}
@Override
- public boolean equals(Object o) {
+ public boolean equals(@Nullable Object o) {
if (this == o) {
return true;
}
@@ -83,7 +84,7 @@ public class QualityProfile {
.add("key", qpKey)
.add("name", qpName)
.add("language", languageKey)
- .add("rulesUpdatedAt", rulesUpdatedAt)
+ .add("rulesUpdatedAt", rulesUpdatedAt.getTime())
.toString();
}
}
diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/event/EventTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/event/EventTest.java
index b3a25ffa67d..92e21eaebf4 100644
--- a/server/sonar-server/src/test/java/org/sonar/server/computation/event/EventTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/computation/event/EventTest.java
@@ -59,7 +59,9 @@ public class EventTest {
@Test
public void same_name_and_category_make_equal_events() {
- assertThat(Event.createAlert(SOME_NAME, null, null)).isEqualTo(Event.createAlert(SOME_NAME, null, null));
-
+ Event source = Event.createAlert(SOME_NAME, null, null);
+ assertThat(source).isEqualTo(Event.createAlert(SOME_NAME, null, null));
+ assertThat(source).isEqualTo(source);
+ assertThat(source).isNotEqualTo(null);
}
}
diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/qualityprofile/QualityProfileTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/qualityprofile/QualityProfileTest.java
index d4ebfaaa834..9db6e0f1b47 100644
--- a/server/sonar-server/src/test/java/org/sonar/server/computation/qualityprofile/QualityProfileTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/computation/qualityprofile/QualityProfileTest.java
@@ -21,6 +21,7 @@ package org.sonar.server.computation.qualityprofile;
import java.util.Date;
import org.junit.Test;
+import org.sonar.api.utils.DateUtils;
import static org.assertj.core.api.Assertions.assertThat;
@@ -29,7 +30,7 @@ public class QualityProfileTest {
private static final String SOME_QP_KEY = "qpKey";
private static final String SOME_QP_NAME = "qpName";
private static final String SOME_LANGUAGE_KEY = "languageKey";
- private static final Date SOME_DATE = new Date();
+ private static final Date SOME_DATE = DateUtils.parseDateTimeQuietly("2010-05-18T15:50:45+0100");
private static final QualityProfile QUALITY_PROFILE = new QualityProfile(SOME_QP_KEY, SOME_QP_NAME, SOME_LANGUAGE_KEY, SOME_DATE);
@Test(expected = NullPointerException.class)
@@ -69,4 +70,10 @@ public class QualityProfileTest {
public void verify_equals() {
assertThat(QUALITY_PROFILE).isEqualTo(new QualityProfile(SOME_QP_KEY, SOME_QP_NAME, SOME_LANGUAGE_KEY, SOME_DATE));
}
-} \ No newline at end of file
+
+ @Test
+ public void verify_toString() {
+ assertThat(QUALITY_PROFILE.toString()).isEqualTo("QualityProfile{key=qpKey, name=qpName, language=languageKey, rulesUpdatedAt=1274194245000}");
+
+ }
+}