diff options
-rw-r--r-- | sonar-java-api/src/main/java/org/sonar/java/api/JavaClass.java | 5 | ||||
-rw-r--r-- | sonar-java-api/src/test/java/org/sonar/java/api/JavaClassTest.java | 22 |
2 files changed, 26 insertions, 1 deletions
diff --git a/sonar-java-api/src/main/java/org/sonar/java/api/JavaClass.java b/sonar-java-api/src/main/java/org/sonar/java/api/JavaClass.java index 8db9f241414..0e2bc281479 100644 --- a/sonar-java-api/src/main/java/org/sonar/java/api/JavaClass.java +++ b/sonar-java-api/src/main/java/org/sonar/java/api/JavaClass.java @@ -105,6 +105,11 @@ public final class JavaClass extends Resource { return false; } + @Override + public String toString() { + return getName(); + } + public static JavaClass createRef(String name) { return new JavaClass(name); } diff --git a/sonar-java-api/src/test/java/org/sonar/java/api/JavaClassTest.java b/sonar-java-api/src/test/java/org/sonar/java/api/JavaClassTest.java index a2972ac2bc3..869f60b3be5 100644 --- a/sonar-java-api/src/test/java/org/sonar/java/api/JavaClassTest.java +++ b/sonar-java-api/src/test/java/org/sonar/java/api/JavaClassTest.java @@ -17,7 +17,6 @@ * License along with Sonar; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02 */ - package org.sonar.java.api; import org.junit.Test; @@ -66,4 +65,25 @@ public class JavaClassTest { javaClass = JavaClass.createRef("Bar"); assertThat(javaClass.getClassName(), is("Bar")); } + + @Test + public void shouldOverrideToString() { + JavaClass javaClass = JavaClass.createRef("org.foo.Bar"); + assertThat(javaClass.toString(), is("org.foo.Bar")); + } + + @Test + public void shouldBuild() { + JavaClass javaClass = new JavaClass.Builder().setName("org.foo", "Bar").setFromLine(30).create(); + assertThat(javaClass.getName(), is("org.foo.Bar")); + assertThat(javaClass.getFromLine(), is(30)); + assertThat(javaClass.getToLine(), is(JavaClass.UNKNOWN_LINE)); + } + + @Test + public void shouldNotBuildWithNegativeNumberOfLine() { + JavaClass javaClass = new JavaClass.Builder().setName("org.foo", "Bar").setFromLine(-30).setToLine(0).create(); + assertThat(javaClass.getFromLine(), is(JavaClass.UNKNOWN_LINE)); + assertThat(javaClass.getToLine(), is(0)); + } } |