[FrameworkBundle] Inject the clock into comparison and range validators - #66683

#63360 made the comparison validators and RangeValidator accept a clock, and bound ClockInterface on the validator prototype in validator.php. That prototype is abstract and tagged container.excluded, so it only exists for the translation extractor. AddConstraintValidatorsPass skips excluded services, the factory never finds these validators in its locator, and it builds them with new $name(). The binding is never used, so MockClock and ClockSensitiveTrait::mockTime() have no effect on bounds like '+48 hours'.

This registers RangeValidator and the eight comparison validators as concrete services that get clock, the same way validator.email or validator.when are registered. The property accessor argument stays null, as it was with new $name(). The dead binding is removed. DivisibleByValidator also extends AbstractComparisonValidator, but relative dates make no sense there, so I left it out.

The new test fails without the fix (1 violation instead of 0) and passes with it. ./phpunit src/Symfony/Bundle/FrameworkBundle passes (1486 tests, 3 skipped).

Merge-up to 8.2: the config moved to src/Symfony/Component/Validator/Resources/config/validator.php, and git applies the change there. Only its imports conflict: keep ConstantFunctionProvider and drop ClockInterface. FrameworkExtensionTestCase conflicts too: keep the 8.2 side and move the test to ValidationBundleTest, where it needs no version check (with the MockClock, GreaterThan, LessThan and Range imports):

public function testComparisonAndRangeValidatorsUseTheClock()
{
    $container = $this->load([], merge: false);
    $container->register('clock', MockClock::class)->setArguments(['2020-01-01 12:00:00']);
    $container->setAlias('validator.alias', 'validator')->setPublic(true);
    $container->compile();

    $validator = $container->get('validator.alias');
    $value = new \DateTimeImmutable('2020-02-01');

    $this->assertCount(0, $validator->validate($value, new Range(min: '+48 hours', max: '+1 year')));
    $this->assertCount(0, $validator->validate($value, new GreaterThan('+48 hours')));
    $this->assertCount(0, $validator->validate($value, new LessThan('+1 year')));
}

Checked on 8.2: the test fails without the change and passes with it.