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

[IR-2164] Unit Test - Physics #10282

Merged
merged 96 commits into from
Jun 28, 2024
Merged

[IR-2164] Unit Test - Physics #10282

merged 96 commits into from
Jun 28, 2024

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
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 🤔

packages/spatial/src/physics/classes/Physics.test.ts Outdated Show resolved Hide resolved
packages/spatial/src/physics/classes/Physics.test.ts Outdated Show resolved Hide resolved
packages/spatial/src/physics/classes/Physics.test.ts Outdated Show resolved Hide resolved
packages/spatial/src/physics/classes/Physics.test.ts Outdated Show resolved Hide resolved
`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
25 checks passed
@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