Skip to content

Commit d7c1773

Browse files
committed
Add a few test cases to NonEmptyStringValidator
1 parent 86410c8 commit d7c1773

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

src/Domain/Question/Validation/NonEmptyStringValidator.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@
77
{
88
public function validate(string $value): ValidationResult
99
{
10+
if (trim($value) === '') {
11+
return ValidationResult::invalid('The value cannot be empty');
12+
}
13+
1014
return ValidationResult::valid();
1115
}
1216
}

tests/unit/Domain/Question/Validation/NonEmptyStringValidatorTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,5 +23,8 @@ public function theNonEmptyStringValidator(string $value, ValidationResult $expe
2323
public static function provideDataToTestTheNonEmptyStringValidator(): Generator
2424
{
2525
yield ['test', ValidationResult::valid()];
26+
yield [' test ', ValidationResult::valid()];
27+
yield ['', ValidationResult::invalid('The value cannot be empty')];
28+
yield [' ', ValidationResult::invalid('The value cannot be empty')];
2629
}
2730
}

0 commit comments

Comments
 (0)