Modified required text validator

konifar · konifar · commit 661d23c1d5e1 · 2018-03-27T10:31:23.000+09:00
diff --git a/library/src/main/java/co/kyash/vtl/validators/RequiredValidator.kt b/library/src/main/java/co/kyash/vtl/validators/RequiredValidator.kt
@@ -36,7 +36,7 @@ class RequiredValidator(
      * @return result
      */
     override fun validate(text: String?): Boolean {
-        return !TextUtils.isEmpty(text)
+        return !TextUtils.isEmpty(text?.trim())
     }
 
     /**
diff --git a/library/src/test/java/co/kyash/vtl/validators/RequiredValidatorTest.kt b/library/src/test/java/co/kyash/vtl/validators/RequiredValidatorTest.kt
@@ -27,6 +27,8 @@ class RequiredValidatorTest(
             return listOf(
                     arrayOf(null, false, ERROR_MESSAGE),
                     arrayOf("", false, ERROR_MESSAGE),
+                    arrayOf(" ", false, ERROR_MESSAGE),
+                    arrayOf("　", false, ERROR_MESSAGE),
                     arrayOf("konifar", true, null)
             )
         }

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ class RequiredValidator(`
`36`	`36`	`* @return result`
`37`	`37`	`*/`
`38`	`38`	`override fun validate(text: String?): Boolean {`
`39`		`- return !TextUtils.isEmpty(text)`
	`39`	`+ return !TextUtils.isEmpty(text?.trim())`
`40`	`40`	`}`
`41`	`41`
`42`	`42`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,8 @@ class RequiredValidatorTest(`
`27`	`27`	`return listOf(`
`28`	`28`	`arrayOf(null, false, ERROR_MESSAGE),`
`29`	`29`	`arrayOf("", false, ERROR_MESSAGE),`
	`30`	`+ arrayOf(" ", false, ERROR_MESSAGE),`
	`31`	`+ arrayOf("　", false, ERROR_MESSAGE),`
`30`	`32`	`arrayOf("konifar", true, null)`
`31`	`33`	`)`
`32`	`34`	`}`