[Tech Debt] Add unit tests for start screen API
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 68/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- typescript
- Lĩnh vực
- testing
Hướng nghiên cứu
Start by reading PR #618 and the start-screen code in packages/lua-runtime/src/CanvasController.ts, packages/canvas-runtime/src/worker/LuaCanvasRuntime.ts, and packages/canvas-runtime/src/process/LuaCanvasProcess.ts. Create or update the named start-screen test files, covering callbacks, waiting state, overlay rendering, dismissal interactions, and callback errors. Run the relevant test suite and confirm all listed behaviors are covered.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Description
The start screen feature (PR #618) lacks automated unit tests. The following functionality needs test coverage:
canvas.set_start_screen(callback)- custom callback invocationcanvas.is_waiting_for_interaction()- return value correctness- Start screen overlay rendering (default overlay)
- Interaction detection and start screen dismissal (click/keypress)
- Error handling when custom callback throws
Location
Test Files to Create/Update
packages/lua-runtime/tests/CanvasController.startScreen.test.tspackages/canvas-runtime/tests/worker/LuaCanvasRuntime.startScreen.test.ts
Found In
PR #618 - feat: Add "Click to Start" overlay for exported games with audio (#617)
Priority
P2-Medium - Tests improve confidence in browser audio policy handling
- Ngôn ngữ chính
- TypeScript
- Star
- 1
- Fork
- 0
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
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 jcollard/LuaInTheWeb
-
tech-debt
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
jcollard/LuaInTheWeb#752 ·
-
tech-debt
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 68/100
jcollard/LuaInTheWeb#690 ·
-
tech-debt
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
jcollard/LuaInTheWeb#590 ·
-
tech-debt
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
jcollard/LuaInTheWeb#518 ·
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
jcollard/LuaInTheWeb#235 ·
Tất cả issue của jcollard/LuaInTheWeb
Issue tương tự
-
bug needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
apex-dev-tools/apex-log-parser#199 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area/frontend/library-panels automated-triage type/bug type/ux
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
PostHog/posthog.com#21607 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
ajv-validator/ajv#2689 ·
-
[Bug]: IME confirmation Enter can submit the SSH port-forward dialogCó thể đã có người làm @seijikohara đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày