Fix runHook twice

Author: compulim

__tests__/html2/timestamp/attachmentSendTimeout.html

@@ -65,7 +65,11 @@
           fileBlob.name = 'empty.zip';
 
           // WHEN: Sending out the file.
-          await pageObjects.runHook(({ useSendFiles }) => useSendFiles()([fileBlob]));
+          await pageObjects.runHook(({ useSendFiles }, numCalled) => {
+            const sendFiles = useSendFiles();
+
+            numCalled || sendFiles([fileBlob]);
+          });
           await pageConditions.numActivitiesShown(2);
 
           // THEN: It should show "Sending".
@@ -92,7 +96,6 @@
           updateProps(props => updateIn(props, ['styleOptions', 'sendTimeoutForAttachments'], () => 130000));
 
           // THEN: It should revive the message and show "Sending" again.
-          expect(pageElements.activities()).toHaveProperty('length', 1);
           await expect(pageElements.activityStatuses()[1].innerText).toBe('Sending');
           await host.snapshot('local');
         },

packages/test/page-object/src/globals/testHelpers/createRunHookActivityMiddleware.js

@@ -1,15 +1,19 @@
-const RunHook = ({ fn, resolve }) => {
-  resolve(fn(window.WebChat.hooks));
+const { createElement, memo, useRef } = React;
+
+const RunHook = memo(({ fn, resolve }) => {
+  const numCalledRef = useRef(0);
+
+  resolve(fn(window.WebChat.hooks, numCalledRef.current++));
 
   return false;
-};
+});
 
 function createRunHookActivityMiddleware() {
   return () =>
     next =>
     ({ activity, ...others }) => {
       if (activity.type === 'event' && activity.name === '__RUN_HOOK') {
-        return () => !activity.ref.count++ && window.React.createElement(RunHook, activity.value);
+        return () => !activity.ref.count++ && createElement(RunHook, activity.value);
       }
 
       return next({ activity, ...others });