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

checkPage ignores a screen's declared readiness signal

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

Maintainer thường phản hồi trong vòng 1 ngày

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
74/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ệ
playwright, typescript
Lĩnh vực
accessibility, testing

Hướng nghiên cứu

Bắt đầu với src/surfaces/playwright-helper.ts và so sánh checkPage với bước kiểm tra trạng thái sẵn sàng của CLI cho các màn hình đã cấu hình. Theo dõi cách screenId được phân giải thành một màn hình và cách reachedWhen được đánh giá. Hoàn tất khi thiếu bộ chọn trạng thái sẵn sàng sẽ tạo ra một lỗ hổng phạm vi bao phủ nêu tên bộ chọn đó và các provider không chạy; chạy các bài kiểm thử helper Playwright liên quan.

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

Mô tả

area: playwright bug

What happens

checkPage (the Playwright helper, src/surfaces/playwright-helper.ts) runs the providers right away on the page it is given. It uses its own PAGE_CHECK_CONFIG, which holds no surfaces, so a screen's declared reachedWhen selector is never checked.

Why it matters

The CLI path makes a screen a not-covered gap when its declared selector does not appear. checkPage does not. If a test calls checkPage with a configured screenId while the page shows only its empty shell, the result can be verified. That result is advisory and does not reach the merge gate, but it still reads as a pass for content that was never on the page.

Expected

When the screenId names a configured screen that declares reachedWhen, checkPage applies the same readiness rule as the CLI. If the signal does not hold, the result is a coverage gap that names the selector, and no providers run.

Status

Found in review by reading the code. This is old behavior, not added by a recent change. Related: #304 (other checkPage input checks).

Ngôn ngữ chính
TypeScript
Star
0
Fork
0
Merge trung bình
2 giờ 31 phút
Pull request đã merge (30 ngày)
9

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

  • Không có Dockerfile hay tệp Docker Compose
  • Có mẫu pull request
  • Không có hướng dẫn đóng góp

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 usabl-dev/usabl

Tất cả issue của usabl-dev/usabl

Issue tương tự

Thêm issue về TypeScript

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.