fix(components/modals): SkyModalTestingController.closeTopModal pas…

…ses `reason` and `result` to the modal instance (#2565)

libs/components/modals/testing/src/modal/controller/modal-testing.controller.spec.ts

@@ -82,6 +82,10 @@ class TestComponent implements OnDestroy {
     this.#instances.forEach((i) => i.close());
   }
 
+  public getInstanceAt(index: number): SkyModalInstance | undefined {
+    return this.#instances.at(index);
+  }
+
   public openModal(): void {
     const instance = this.#modalSvc.open(ModalTestComponent, {
       providers: [
@@ -143,6 +147,24 @@ describe('modal-testing.controller', () => {
     modalController.expectNone();
   });
 
+  it('should close a modal with args', () => {
+    const { fixture, modalController } = setupTest();
+
+    fixture.componentInstance.openModal();
+    fixture.detectChanges();
+
+    const closeSpy = spyOn(
+      fixture.componentInstance.getInstanceAt(0)!,
+      'close',
+    ).and.callThrough();
+
+    modalController.closeTopModal({ reason: 'save', data: { foo: 'bar' } });
+
+    fixture.detectChanges();
+
+    expect(closeSpy).toHaveBeenCalledWith({ foo: 'bar' }, 'save');
+  });
+
   it('should throw if topmost modal does not match criteria', () => {
     const { fixture, modalController } = setupTest();
 

libs/components/modals/testing/src/modal/controller/modal-testing.service.ts

@@ -40,7 +40,7 @@ export class SkyModalTestingService
       );
     }
 
-    modal.instance.close(args);
+    modal.instance.close(args?.data, args?.reason);
   }
 
   public expectCount(value: number): void {