Adding unit test for: Sonar blocker: obj could be null and is guaranteed

dkayiwa · dkayiwa · commit 5993bd468155 · 2014-01-08T12:25:21.000+03:00
to be dereferenced in org.openmrs.validator.UserValidator.validate -
TRUNK-4199
diff --git a/api/src/main/java/org/openmrs/validator/UserValidator.java b/api/src/main/java/org/openmrs/validator/UserValidator.java
@@ -63,11 +63,12 @@ public boolean supports(Class c) {
 	 * @should pass validation if all required fields have proper values
 	 * @should fail validation if email as username enabled and email invalid
 	 * @should fail validation if email as username disabled and email provided
+	 * @should not throw NPE when user is null
 	 */
 	public void validate(Object obj, Errors errors) {
 		User user = (User) obj;
 		if (user == null) {
-			errors.rejectValue("user", "error.general");
+			errors.reject("error.general");
 		} else {
 			if (user.isRetired() && StringUtils.isBlank(user.getRetireReason()))
 				errors.rejectValue("retireReason", "error.null");
diff --git a/api/src/test/java/org/openmrs/validator/UserValidatorTest.java b/api/src/test/java/org/openmrs/validator/UserValidatorTest.java
@@ -248,4 +248,16 @@ public void validate_shouldFailValidationIfEmailAsUsernameDisabledAndEmailProvid
 		
 		Assert.assertTrue(errors.hasFieldErrors("username"));
 	}
+	
+	/**
+	 * @see UserValidator#validate(Object,Errors)
+	 */
+	@Test
+	@Verifies(value = "not throw NPE when user is null", method = "validate(Object,Errors)")
+	public void validate_shouldNotThrowNPEWhenUserIsNull() throws Exception {
+		UserValidator userValidator = new UserValidator();
+		Errors errors = new BindException(new User(), "user");
+		userValidator.validate(null, errors);
+		Assert.assertTrue(true);
+	}
 }
