mirror of
https://github.com/Respect/Validation.git
synced 2026-03-15 14:55:44 +01:00
d8e31dbc3a
The main focus of this change is to make those optional dependencies more testable. Unfortunately, some phpstan-ignores had to be included, since ::set is not a PsrContainer method. We're only using it on tests though, so it's fine. It targets our php-di container for testing purposes only. The real implementation only relies on ::get. This change also has the side effect of improving the performance of those validators by not instantiating their databases each time a iso validator is built, achieving massive improvements in those scenarios. A small benchmark with no assertions was added to track that improvement.
87 lines
3 KiB
PHP
87 lines
3 KiB
PHP
<?php
|
|
|
|
/*
|
|
* SPDX-License-Identifier: MIT
|
|
* SPDX-FileCopyrightText: (c) Respect Project Contributors
|
|
* SPDX-FileContributor: Alexandre Gomes Gaigalas <alganet@gmail.com>
|
|
* SPDX-FileContributor: Felipe Martins <me@fefas.net>
|
|
* SPDX-FileContributor: Gabriel Caruso <carusogabriel34@gmail.com>
|
|
* SPDX-FileContributor: Henrique Moody <henriquemoody@gmail.com>
|
|
* SPDX-FileContributor: William Espindola <oi@williamespindola.com.br>
|
|
*/
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace Respect\Validation\Validators;
|
|
|
|
use DI;
|
|
use PHPUnit\Framework\Attributes\CoversClass;
|
|
use PHPUnit\Framework\Attributes\Group;
|
|
use PHPUnit\Framework\Attributes\Test;
|
|
use Respect\Validation\ContainerRegistry;
|
|
use Respect\Validation\Exceptions\InvalidValidatorException;
|
|
use Respect\Validation\Exceptions\MissingComposerDependencyException;
|
|
use Respect\Validation\Test\RuleTestCase;
|
|
|
|
#[Group('validator')]
|
|
#[CoversClass(CountryCode::class)]
|
|
final class CountryCodeTest extends RuleTestCase
|
|
{
|
|
#[Test]
|
|
public function itShouldThrowsExceptionWhenInvalidFormat(): void
|
|
{
|
|
$this->expectException(InvalidValidatorException::class);
|
|
$this->expectExceptionMessage(
|
|
'"whatever" is not a valid set for ISO 3166-1 (Available: "alpha-2", "alpha-3", and "numeric")',
|
|
);
|
|
|
|
// @phpstan-ignore-next-line
|
|
new CountryCode('whatever');
|
|
}
|
|
|
|
#[Test]
|
|
public function shouldThrowWhenMissingComponent(): void
|
|
{
|
|
$mainContainer = ContainerRegistry::getContainer();
|
|
ContainerRegistry::setContainer((new DI\ContainerBuilder())->useAutowiring(false)->build());
|
|
try {
|
|
new CountryCode('alpha-3');
|
|
$this->fail('Expected MissingComposerDependencyException was not thrown.');
|
|
} catch (MissingComposerDependencyException $e) {
|
|
$this->assertStringContainsString(
|
|
'CountryCode rule requires PHP ISO Codes',
|
|
$e->getMessage(),
|
|
);
|
|
} finally {
|
|
ContainerRegistry::setContainer($mainContainer);
|
|
}
|
|
}
|
|
|
|
/** @return iterable<array{CountryCode, mixed}> */
|
|
public static function providerForValidInput(): iterable
|
|
{
|
|
return [
|
|
[new CountryCode('alpha-2'), 'BR'],
|
|
[new CountryCode('alpha-3'), 'BRA'],
|
|
[new CountryCode('numeric'), '076'],
|
|
[new CountryCode('alpha-2'), 'DE'],
|
|
[new CountryCode('alpha-3'), 'DEU'],
|
|
[new CountryCode('numeric'), '276'],
|
|
[new CountryCode('alpha-2'), 'US'],
|
|
[new CountryCode('alpha-3'), 'USA'],
|
|
[new CountryCode('numeric'), '840'],
|
|
];
|
|
}
|
|
|
|
/** @return iterable<array{CountryCode, mixed}> */
|
|
public static function providerForInvalidInput(): iterable
|
|
{
|
|
return [
|
|
[new CountryCode(), []],
|
|
[new CountryCode(), 'ca'],
|
|
[new CountryCode('alpha-2'), 'USA'],
|
|
[new CountryCode('alpha-3'), 'US'],
|
|
[new CountryCode('numeric'), '000'],
|
|
];
|
|
}
|
|
}
|