add tests case to improve coverage.

src/test/java/jp/co/worksap/oss/findbugs/jpa/GetterWithLongLengthAndLob.java

@@ -0,0 +1,14 @@
+package jp.co.worksap.oss.findbugs.jpa;
+
+import javax.persistence.Column;
+import javax.persistence.Lob;
+
+public class GetterWithLongLengthAndLob {
+    private String name;
+
+    @Lob
+    @Column(length = 10000)
+    public String getName() {
+        return name;
+    }
+}

src/test/java/jp/co/worksap/oss/findbugs/jpa/GetterWithTooLongLength.java

@@ -0,0 +1,12 @@
+package jp.co.worksap.oss.findbugs.jpa;
+
+import javax.persistence.Column;
+
+public class GetterWithTooLongLength {
+    private String name;
+
+    @Column(length = 10000)
+    public String getName() {
+        return name;
+    }
+}

src/test/java/jp/co/worksap/oss/findbugs/jpa/GetterWithoutElement.java

@@ -0,0 +1,12 @@
+package jp.co.worksap.oss.findbugs.jpa;
+
+import javax.persistence.Column;
+
+public class GetterWithoutElement {
+    private String name;
+
+    @Column
+    public String getName() {
+        return name;
+    }
+}

src/test/java/jp/co/worksap/oss/findbugs/jpa/ImplicitLengthTest.java

@@ -30,12 +30,16 @@ public void testNegativeLength() throws Exception {
     public void testTooLongLength() throws Exception {
         assertBugReported(ColumnWithTooLongLength.class, detector,
                 bugReporter, ofType("ILLEGAL_LENGTH"));
+        assertBugReported(GetterWithTooLongLength.class, detector,
+                bugReporter, ofType("ILLEGAL_LENGTH"));
     }
 
     @Test
     public void testLongLengthWithLob() throws Exception {
         assertNoBugsReported(ColumnWithLongLengthAndLob.class, detector,
                 bugReporter);
+        assertNoBugsReported(GetterWithLongLengthAndLob.class, detector,
+                bugReporter);
     }
 
     @Test
@@ -48,5 +52,7 @@ public void testExplicitLength() throws Exception {
     public void testImplicitLength() throws Exception {
         assertBugReported(ColumnWithoutElement.class, detector,
                 bugReporter, ofType("IMPLICIT_LENGTH"));
+        assertBugReported(GetterWithoutElement.class, detector,
+                bugReporter, ofType("IMPLICIT_LENGTH"));
     }
 }

src/test/java/jp/co/worksap/oss/findbugs/jpa/ImplicitNullnessTest.java

@@ -29,5 +29,7 @@ public void testExplicitNullness() throws Exception {
     public void testImplicitNullness() throws Exception {
         assertBugReported(ColumnWithoutElement.class, detector,
                 bugReporter);
+        assertBugReported(GetterWithoutElement.class, detector,
+                bugReporter);
     }
 }

src/test/java/jp/co/worksap/oss/findbugs/jpa/NullableBooleanGetter.java

@@ -0,0 +1,14 @@
+package jp.co.worksap.oss.findbugs.jpa;
+
+import javax.persistence.Column;
+import javax.persistence.Entity;
+
+@Entity
+public class NullableBooleanGetter {
+    private boolean booleanValue;
+
+    @Column(nullable = true)
+    public boolean isBooleanValue() {
+        return booleanValue;
+    }
+}

src/test/java/jp/co/worksap/oss/findbugs/jpa/NullablePrimitiveDetectorTest.java

@@ -23,10 +23,11 @@ public void before() {
     @Test
     public void testNullableObject() throws Exception {
         assertNoBugsReported(UseColumnDefinition.class, detector, bugReporter);
+        assertNoBugsReported(ColumnWithoutElement.class, detector, bugReporter);
     }
 
     @Test
-    public void testNullableInt() throws Exception {
+    public void testNullablePrimitive() throws Exception {
         assertBugReported(NullableBooleanColumn.class, detector,
                 bugReporter, ofType("NULLABLE_PRIMITIVE"));
         assertBugReported(NullableByteColumn.class, detector,
@@ -41,6 +42,8 @@ public void testNullableInt() throws Exception {
                 bugReporter, ofType("NULLABLE_PRIMITIVE"));
         assertBugReported(NullableDoubleColumn.class, detector,
                 bugReporter, ofType("NULLABLE_PRIMITIVE"));
+        assertBugReported(NullableBooleanGetter.class, detector,
+                bugReporter, ofType("NULLABLE_PRIMITIVE"));
     }
 
     @Test

src/test/java/jp/co/worksap/oss/findbugs/jsr305/MutableClass.java

@@ -1,7 +1,9 @@
 package jp.co.worksap.oss.findbugs.jsr305;
 
 import javax.annotation.concurrent.Immutable;
+import javax.annotation.concurrent.NotThreadSafe;
 
+@NotThreadSafe
 @Immutable // marked as immutable, but field is not final
 public class MutableClass {
     public String value;