Skip to content

#352 | test(code-builder): add unit tests for collision detection - #765

Closed
xtroon wants to merge 1 commit into
sugarlabs:developfrom
xtroon:#352-unit-tests-collision
Closed

#352 | test(code-builder): add unit tests for collision detection#765
xtroon wants to merge 1 commit into
sugarlabs:developfrom
xtroon:#352-unit-tests-collision

Conversation

@xtroon

@xtroon xtroon commented Aug 7, 2026

Copy link
Copy Markdown

Description

Adds unit tests for the collision detection logic in the code-builder module to cover:

  • Core circle and rectangle overlap utilities (utils/index.ts)
  • Brute force collision space implementation (Brute.ts)
  • Quadtree-based collision space implementation (QuadTree.ts)

Related Issue

Closes #352

Verification

  • Ran npx vitest run modules/code-builder and verified all 42 tests pass.
  • Verified linter (npm run lint) passes.

@xtroon

xtroon commented Aug 7, 2026

Copy link
Copy Markdown
Author

hi @meganindya, i have created a pull request to resolve issue #352. it adds the required white box unit tests for the collision detection module (utils, Brute, and QuadTree), please review it when you have time.

@parthdagia05

Copy link
Copy Markdown
Member

refer to the comment #793

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add unit tests for collision detection unit

2 participants