Add coverage for all sniffs identified by testUntested()
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Bắt đầu trong LoadsysStandardTest::testUntested(), bỏ comment markTestIncomplete() và chạy vendor/bin/phpunit để xác định tên của các sniff còn lại. Đọc phần Testing của README, sau đó thêm các tệp mẫu vào tests/files/must/ và tests/files/must_not/ cho các sniff còn lại; công việc được hoàn tất khi toàn bộ danh sách ruleset đã được bao phủ và bộ kiểm thử chạy thành công.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The test suite now compares the full list of sniffs employed by a coding standard against those in the test sample file //~ annotations and throws an assertion failure if the sample files fail to "cover" all tests.
This acts as a crude kind of "coverage" metric of the sniffs defined in the ruleset, but right now we're only covering about 31 out of 120+ sniffs.
To resolve this, uncomment the markTestIncomplete() in LoadsysStandardTest::testUntested(). Then repeatedly run vendor/bin/phpunit and add new sample files for each remaining sniff name listed. Ideally, there should be two files for each sniff-- one confirming that incorrect style is caught by our ruleset, and another that correct behavior is not caught. The new files should be arranged in the tests/files/must/ and tests/files/must_not/ folders. Read the ## Testing section of the README for more naming details and goals.
- Ngôn ngữ chính
- PHP
- Star
- 0
- Fork
- 0
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của loadsys/loadsys_codesniffer
-
enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
loadsys/loadsys_codesniffer#51 · 1 bình luận ·
-
question
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
-
enhancement help-wanted
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
.editorconfig to prevent stripping trailing space, adding final newline in snifftests/files/. Đang mởenhancement help-wanted
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 45/100
-
enhancement help-wanted
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
Tất cả issue của loadsys/loadsys_codesniffer
Issue tương tự
-
tooling
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
UX
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
bug customer-reported
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
MagnaCapax/PMSS#949 ·
-
responsive-media kses allow-list strips clipPathUnits, collapsing objectBoundingBox clip paths Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Automattic/static-site-importer#1829 ·