Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đã đóng Phù hợp với người mới
  • Có gì không ổn?
#826 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
86/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
php
Lĩnh vực
testing-qa

Hướng nghiên cứu

Bắt đầu với tests/Rule/Date/DateTimeTest.php, tests/Rule/Date/TimeTest.php và NumberTest, xem xét testGetName(), testSkipOnError() và testWhen(). Sử dụng các test Date và Number hiện có làm mẫu, sau đó chạy các test rule bị ảnh hưởng để xác nhận rằng DateTime, Time và Integer đều có các kiểm tra contract này.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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
Ngôn ngữ chính
PHP
Star
167
Fork
47
Merge trung bình
1 ngày 16 giờ
Pull request đã merge (30 ngày)
4

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của yiisoft/validator

Tất cả issue của yiisoft/validator

Issue tương tự

Thêm issue về PHP

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.