Skip to content

Commit

Permalink
test: Add/fix tests for binding cross-validation (#18770)
Browse files Browse the repository at this point in the history 
* test: Add/fix tests for binding cross-validation

* Fix unintentional formatting changes
  • Loading branch information
@tepi @vaadin-bot
tepi authored and vaadin-bot committed Feb 21, 2024
1 parent f198f7a commit a7aceeb
Showing 1 changed file with 100 additions and 16 deletions.
116 changes: 100 additions & 16 deletions flow-data/src/test/java/com/vaadin/flow/data/binder/BinderTest.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1411,7 +1411,7 @@ public void info_validator_not_considered_error() {
}

@Test
public void two_asRequired_fields_without_initial_values() {
public void two_asRequired_fields_without_initial_values_setBean() {
binder.forField(nameField).asRequired("Empty name").bind(p -> "",
(p, s) -> {
});
Expand All @@ -1429,16 +1429,17 @@ public void two_asRequired_fields_without_initial_values() {
assertThat("Name with a value should not be an error",
nameField.getErrorMessage(), isEmptyString());

assertNotNull(
"Age field should now be in error, since setBean is used.",
ageField.getErrorMessage());
assertTrue(
"Age field should not be in error, since it was not modified.",
StringUtils.isEmpty(ageField.getErrorMessage()));

nameField.setValue("");
assertNotNull("Empty name should now be in error.",
nameField.getErrorMessage());

assertNotNull("Age field should still be in error.",
ageField.getErrorMessage());
assertFalse("Empty name should now be in error.",
StringUtils.isEmpty(nameField.getErrorMessage()));
assertTrue(
"Age field should still not be in error, since it was not modified.",
StringUtils.isEmpty(ageField.getErrorMessage()));
}

@Test
Expand All @@ -1460,16 +1461,99 @@ public void two_asRequired_fields_without_initial_values_readBean() {
assertThat("Name with a value should not be an error",
nameField.getErrorMessage(), isEmptyString());

assertThat(
"Age field should not be in error, since it has not been modified.",
ageField.getErrorMessage(), isEmptyString());
assertTrue(
"Age field should not be in error, since it was not modified.",
StringUtils.isEmpty(ageField.getErrorMessage()));

nameField.setValue("");
assertNotNull("Empty name should now be in error.",
nameField.getErrorMessage());
assertFalse("Empty name should now be in error.",
StringUtils.isEmpty(nameField.getErrorMessage()));

assertTrue(
"Age field should still not be in error, since it was not modified.",
StringUtils.isEmpty(ageField.getErrorMessage()));
}

@Test
public void validated_and_asRequired_fields_without_initial_values_setBean() {
binder.forField(nameField).asRequired("Empty name")
.bind(Person::getFirstName, Person::setFirstName);
TestTextField lastNameField = new TestTextField();
binder.forField(lastNameField)
.withValidator((v, c) -> StringUtils.isEmpty(v)
? ValidationResult.error("Empty last name")
: ValidationResult.ok())
.bind(Person::getLastName, Person::setLastName);

binder.setBean(item);
assertFalse("Initially there should be no errors",
nameField.isInvalid());
assertFalse("Initially there should be no errors",
lastNameField.isInvalid());

nameField.setValue("Foo");
assertFalse("Name with a value should not be an error",
nameField.isInvalid());
assertFalse(
"Last name field should not be in error, since it was not modified.",
lastNameField.isInvalid());

nameField.setValue("");

assertTrue("Empty name should now be in error.", nameField.isInvalid());
assertFalse(
"Last name field should not be in error, since it was not modified.",
lastNameField.isInvalid());

nameField.setValue("Bar");
lastNameField.setValue("Bar");
lastNameField.setValue("");

assertThat("Age field should still be ok.", ageField.getErrorMessage(),
isEmptyString());
assertFalse("Name with a value should not be an error",
nameField.isInvalid());
assertTrue("Empty last name field should now be in error.",
lastNameField.isInvalid());
}

@Test
public void validated_and_asRequired_fields_without_initial_values_readBean() {
binder.forField(nameField).asRequired("Empty name")
.bind(Person::getFirstName, Person::setFirstName);
TestTextField lastNameField = new TestTextField();
binder.forField(lastNameField)
.withValidator((v, c) -> StringUtils.isEmpty(v)
? ValidationResult.error("Empty last name")
: ValidationResult.ok())
.bind(Person::getLastName, Person::setLastName);

binder.readBean(item);
assertFalse("Initially there should be no errors",
nameField.isInvalid());
assertFalse("Initially there should be no errors",
lastNameField.isInvalid());

nameField.setValue("Foo");
assertFalse("Name with a value should not be an error",
nameField.isInvalid());
assertFalse(
"Last name field should not be in error, since it was not modified.",
lastNameField.isInvalid());

nameField.setValue("");

assertTrue("Empty name should now be in error.", nameField.isInvalid());
assertFalse(
"Last name field should not be in error, since it was not modified.",
lastNameField.isInvalid());

nameField.setValue("Bar");
lastNameField.setValue("Bar");
lastNameField.setValue("");

assertFalse("Name with a value should not be an error",
nameField.isInvalid());
assertTrue("Empty last name field should now be in error.",
lastNameField.isInvalid());
}

@Test
Expand Down Expand Up @@ -2202,7 +2286,7 @@ public String convertToPresentation(SubPropClass value,
ValueContext context) {
return value != null ? value.toString() : null;
}
};
}

/**
* A converter that adds/removes the euro sign and formats currencies with
Expand Down

0 comments on commit a7aceeb

Please # to comment.