Skip to content
This repository was archived by the owner on Aug 21, 2024. It is now read-only.

[IR-2164] Unit Test - Physics#10282

Merged
HexaField merged 96 commits intodevfrom
IR-2164-unit-test-physics
Jun 28, 2024
Merged

[IR-2164] Unit Test - Physics#10282
HexaField merged 96 commits intodevfrom
IR-2164-unit-test-physics

Conversation

@heysokam
Copy link
Contributor

@heysokam heysokam commented May 30, 2024

Summary

Subtasks Checklist

  • All Physics.ts Rapier->ECS APIs
  • All Physics.ts External APIs
  • RigidbodyComponent
  • ColliderComponent
  • CollisionComponent
  • TriggerComponent
  • TriggerSystem
  • PhysicsSystem

Breaking Changes

References

QA Steps

@heysokam heysokam self-assigned this May 30, 2024
@heysokam
Copy link
Contributor Author

@HexaField @DanielBelmes

  1. Are these tests going in the right direction?
  2. I have tests for like 35-40% of a single file, and its already at 700sloc. Should I subdivide this task into multiple PRs? It feels massive for a single one 🤔

heysokam added 23 commits May 31, 2024 04:21
`RigidBodyComponent.onSet` allowed type modification/coercion when passing an array of `length === 3` that has incorrectly typed values.
This fix should stop `@ts-ignore` from accidentally modifying values that shouldn't be modified.
@heysokam heysokam marked this pull request as ready for review June 27, 2024 03:35
@HexaField HexaField self-requested a review June 28, 2024 04:58
@HexaField HexaField added this pull request to the merge queue Jun 28, 2024
Merged via the queue into dev with commit f38852e Jun 28, 2024
@HexaField HexaField deleted the IR-2164-unit-test-physics branch June 28, 2024 07:06
MbfloydIR added a commit that referenced this pull request Jun 28, 2024
* dev:
  Reflection probes (#10458)
  standardize IR-Engine Deploy across dev,int,stg; IR-2698 (#10474)
  added scene helpers tools (#10477)
  fix loading screens (#10476)
  [IR-2164] Unit Test - Physics (#10282)
  studio: fix viewport container bg and margin (#10473)
  optimize spline helper (#10295)
  Point & Click Placement Mode (#10042)
  add postprocessing editor back (#10472)
  IR-1579-Resource-tracker (#10423)
  Ir 2790 mouseover tooltip to show fullname (#10456)

# Conflicts:
#	packages/editor/src/services/ComponentEditors.tsx
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants