feat(react-popup-manager) - adding 'response' to return value of `ope…

…n` popup. is resolved after modal is closed.

#pr-cr@itaya,nitzansi,alexko

src/PopupsWrapper.tsx

@@ -70,8 +70,8 @@ export class PopupsWrapper extends React.Component<PopupsWrapperProps> {
     this.props.popupManager.subscribeOnPopupsChange(() => this.forceUpdate());
   }
 
-  private onClose(guid: string, result: any) {
-    this.props.popupManager.close(guid, result);
+  private onClose(guid: string, onAfterClose?: Function) {
+    this.props.popupManager.close(guid, onAfterClose);
   }
 
   public render() {

src/__internal__/PopupItem.ts

@@ -33,20 +33,10 @@ export class PopupItem {
       this._resolve();
     }
 
-    let result: any;
-    let error: any;
     try {
-      result = await onAfterClose();
+      this._resolve(await onAfterClose());
     } catch (ex) {
-      error = ex;
-    }
-
-    if (error) {
-      this._reject(error);
-    } else if (result) {
-      this._resolve(result);
-    } else {
-      this._resolve();
+      this._reject(ex);
     }
   }
 

src/tests/specs/testPopups.resolve.spec.tsx → src/tests/specs/testPopups.response.spec.tsx

@@ -3,7 +3,7 @@ import {generateDataHook, TestPopupUsesIsOpen} from "../TestPopupUsesIsOpen/Test
 import {PopupManager} from "../../popupManager";
 import {TestPopupsDriver} from "../TestPopups.driver";
 
-describe('TestPopupUsesIsOpen', () => {
+describe('testPopups - "response" ', () => {
         let driver: TestPopupsDriver;
         const buttonOpenDataHook = 'button-open';