Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Copy-paste gaps in rule tests: `DateTime`/`Time` tests check `Date`, and `Integer` has no contract tests

Abierto Apto para principiantes
  • ¿Algo no está bien?
#826 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
86/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
php
Área
testing-qa

Línea de trabajo

Comienza con tests/Rule/Date/DateTimeTest.php, tests/Rule/Date/TimeTest.php y NumberTest, revisando testGetName(), testSkipOnError() y testWhen(). Usa los tests existentes de Date y Number como patrones y luego ejecuta los tests de las reglas afectadas para confirmar que DateTime, Time e Integer tienen cada uno estas comprobaciones del contrato.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

What steps will reproduce the problem?

1. DateTimeTest and TimeTest test the Date rule instead of their own rule.

Three methods in tests/Rule/Date/DateTimeTest.php and tests/Rule/Date/TimeTest.php create new Date() instead of new DateTime() / new Time():

public function testGetName(): void
{
    $rule = new Date();                      // should be new DateTime() / new Time()
    $this->assertSame('date', $rule->getName());
}

public function testSkipOnError(): void
{
    $this->testSkipOnErrorInternal(new Date(), new Date(skipOnError: true));
}

public function testWhen(): void
{
    $this->testWhenInternal(
        new Date(),
        new Date(when: static fn(mixed $value): bool => $value !== null),
    );
}

So getName(), skipOnError and when are never tested for DateTime and Time. DateTest already covers Date, so these checks only run twice more on the same rule.

2. The Integer rule has no tests for getName(), skipOnError and when.

Integer has no test class of its own. NumberTest checks how Integer validates values (dataValidationPassed()/dataValidationFailed()), but testGetName(), testSkipOnError() and testWhen() there only use new Number(). AbstractNumber::getName() returns static::class, so Integer returns its own name, and no test checks that.

What is the expected result?

Every rule has these contract checks run on an instance of that rule.

What do you get instead?

DateTime, Time and Integer are not covered. A regression in their getName(), skipOnError() or when() would go unnoticed. For example: an override in the class, a missing trait after a refactoring, or a wrong default in the constructor.

Suggested fix

  • In DateTimeTest/TimeTest, replace Date with DateTime/Time in the three methods above. getName() returns 'date' for all of them (from BaseDate), so the expected value stays the same.

  • Add testIntegerGetName(), testIntegerSkipOnError() and testIntegerWhen() to NumberTest, or split them into a separate IntegerTest:

    public function testIntegerSkipOnError(): void
{
    $this->testSkipOnErrorInternal(new Integer(), new Integer(skipOnError: true));
}

  • Optionally, prevent this kind of copy-paste mistake: add one data-provider test that runs the skipOnError/when checks over a list of all rules, instead of a separate method in each test class. Then adding a new rule means adding one line to the provider. The #[DataProvider] returns rule × option pairs (skipOnError, when); each pair gives a factory, the option name, its default, a new value and the getter name.

Additional info
Q A
Version 2.x-dev (9be0c31)
PHP version any
Operating system any
Lenguaje dominante
PHP
Estrellas
167
Forks
47
Merge medio
1 d 19 h
PR fusionados (30 d)
3

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de yiisoft/validator

Todos los issues de yiisoft/validator

Issues similares

Más issues de PHP

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.