fix(ecs): fix TriggerEntity not passing params in callback call

Author: RuggeroVisintin

src/ecs/components/BoundingBoxComponent.ts

@@ -1,4 +1,4 @@
-import { SerializableCallback, Type, Vec2, WithType } from "../../core";
+import { Type, Vec2, WithType } from "../../core";
 import { PhysicsObject, Physx } from "../../physx";
 import { BaseComponent } from "./BaseComponent";
 import { TransformComponent } from "./TransformComponent";

src/ecs/entities/TriggerEntity.ts

@@ -79,6 +79,6 @@ export class TriggerEntity extends StaticObject implements TriggerEntityProps {
     }
 
     private onCollisionHandler(params: CollisionCallbackParams): void {
-        params.collider.uuid === this._targetComponent?.uuid && this.onTriggerCB?.call(params);
+        params.collider.uuid === this._targetComponent?.uuid && this.onTriggerCB?.call(this, params);
     }
 }

test/unit/ecs/entities/TriggerEntity.test.ts

@@ -72,9 +72,7 @@ describe('ecs/entities/TriggerEntity', () => {
 
     describe('.onTriggerCB', () => {
         it('Should be invoked when a collision between this entity and the target is detected', () => {
-            const callback = jest.fn();
-            triggerEntity.onTriggerCB = SerializableCallback.fromFunction(callback);
-            triggerEntity.boundingBox.onCollisionCb?.call(this, {
+            const params = {
                 collider: {
                     aabb: [1, 0, 1, 0],
                     velocity: new Vec2(),
@@ -85,9 +83,12 @@ describe('ecs/entities/TriggerEntity', () => {
                     velocity: new Vec2(),
                     uuid: triggerEntity.boundingBox.uuid
                 }
-            });
+            }
+            const callback = jest.fn();
+            triggerEntity.onTriggerCB = SerializableCallback.fromFunction(callback);
+            triggerEntity.boundingBox.onCollisionCb?.call(this, params);
 
-            expect(callback).toHaveBeenCalled();
+            expect(callback).toHaveBeenCalledWith(params);
         });
     });