minor #32249 [Validator] remove invalid test cases (xabbuh)
This PR was merged into the 3.4 branch.
Discussion
----------
[Validator] remove invalid test cases
| Q | A
| ------------- | ---
| Branch? | 3.4
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Spotted while working on #32179 for the Validator component. Using property paths for comparison when validating arrays simply does not work.
Commits
-------
d1261e78a4
remove invalid test cases
This commit is contained in:
commit
9bc8b39709
@ -148,20 +148,6 @@ abstract class AbstractComparisonValidatorTestCase extends ConstraintValidatorTe
|
|||||||
$this->assertNoViolation();
|
$this->assertNoViolation();
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* @dataProvider provideValidComparisonsToPropertyPath
|
|
||||||
*/
|
|
||||||
public function testValidComparisonToPropertyPathOnArray($comparedValue)
|
|
||||||
{
|
|
||||||
$constraint = $this->createConstraint(['propertyPath' => '[root][value]']);
|
|
||||||
|
|
||||||
$this->setObject(['root' => ['value' => 5]]);
|
|
||||||
|
|
||||||
$this->validator->validate($comparedValue, $constraint);
|
|
||||||
|
|
||||||
$this->assertNoViolation();
|
|
||||||
}
|
|
||||||
|
|
||||||
public function testNoViolationOnNullObjectWithPropertyPath()
|
public function testNoViolationOnNullObjectWithPropertyPath()
|
||||||
{
|
{
|
||||||
$constraint = $this->createConstraint(['propertyPath' => 'propertyPath']);
|
$constraint = $this->createConstraint(['propertyPath' => 'propertyPath']);
|
||||||
|
Reference in New Issue
Block a user