Add unit tests for the Stack rules engine

Đang mở
  • Có gì không ổn?
#29 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
72/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ệ
typescript
Lĩnh vực
game-dev, testing

Hướng nghiên cứu

Start with games/stack/logic.ts:104-208 and games/stack/scenes/GameScene.ts:423; the issue lists the rules and edge cases to cover in a new games/stack/logic.test.ts. Check the project's test setup because the issue says no test suite exists, then add tests for the listed behaviors and fix the gravitySeconds boundary defect. Done when the cases pass in CI.

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

Mô tả

help wanted severity: medium testing

Summary

games/stack/logic.ts is Phaser-free and says in its header that it is easy to test, but has no tests. It is the most branch-heavy pure module in the repo, and it contains one boundary defect that a table test would catch.

Where

  • games/stack/logic.ts:104-208
  • games/stack/logic.ts:207 (gravitySeconds), games/stack/scenes/GameScene.ts:423 (level is uncapped)

Details

gravitySeconds raises a base that goes negative past level 115 to an integer power, so odd levels above 258 return large positive values: gravitySeconds(259) is about 4.68 where 1/60 is expected. It needs roughly 2,580 lines to reach, so it is a low-severity edge.

Suggested fix

Add games/stack/logic.test.ts:

  • emptyBoard() is 22 rows by 10 columns of null, rows are distinct arrays
  • spawn('I') is {rot:0,x:3,y:0}, spawn('O') has x=4; none collide on an empty board
  • Every ROTATIONS[t] has 4 states of 4 cells
  • collides is true for x<0, x>=10, y>=22 and occupied cells; false for cells at y<0
  • tryMove returns null when blocked and does not mutate its input
  • tryRotate on O returns the same object; T rotates both ways; a wall kick from x=-1 lands in bounds; returns null when all five kicks are blocked; I uses its own kick table
  • dropDistance is 20 for a fresh I, O or T and 0 for a resting piece
  • lock reports lock-out for a piece entirely in the hidden rows
  • fullRows and clearRows handle none, one and two rows and keep 22 rows
  • Bag with a seeded RNG yields a permutation of all seven types per run of 7; peek does not consume and works across a refill
  • gravitySeconds is non-increasing and never below 1/60 for levels 1 to 500 (fails today; clamp the base at 0 or cap the level)

Acceptance criteria

  • The cases above pass in CI

Getting started

  • Relevant files: games/stack/logic.ts, games/stack/scenes/GameScene.ts
  • Run locally: npm install, then npm run dev
  • Checks: npm run typecheck and npm run build (there is no test suite yet)
  • Related docs: README.md

Filed by repo-audit/full-audit at b001f5d. Severity: medium. Effort: M. audit-key: testing|games/stack/logic.ts|tryRotate

Ngôn ngữ chính
TypeScript
Star
3
Fork
3
Không có pull request nào được merge trong 30 ngày

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

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

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 TMHSDigital/GH-Arcade

Tất cả issue của TMHSDigital/GH-Arcade

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.