Export all hooks

microsoft · Nov 26, 2019 · 7722851 · 7722851
1 parent 3972dab
commit 7722851
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 17 deletions.
diff --git a/__tests__/hooks/useMicrophoneButton.js b/__tests__/hooks/useMicrophoneButton.js
@@ -9,20 +9,27 @@ import uiConnected from '../setup/conditions/uiConnected';
 jest.setTimeout(timeouts.test);
 
 test('microphoneButtonClick should toggle recording', async () => {
-  // TODO: [P1] Test is temporarily disabled until the hook is implemented
-  // const { driver, pageObjects } = await setupWebDriver({
-  //   props: {
-  //     webSpeechPonyfillFactory: () => window.WebSpeechMock
-  //   }
-  // });
-  // await driver.wait(uiConnected(), timeouts.directLine);
-  // await pageObjects.runHook('useMicrophoneButtonClick', [], microphoneButtonClick => microphoneButtonClick());
-  // await driver.wait(speechRecognitionStartCalled(), timeouts.ui);
-  // await expect(
-  //   pageObjects.runHook('useMicrophoneButtonDisabled', [], microphoneButtonDisabled => microphoneButtonDisabled[0])
-  // ).resolves.toBeTruthy();
-  // await pageObjects.putSpeechRecognitionResult('recognizing', 'Hello');
-  // await expect(pageObjects.isDictating()).resolves.toBeTruthy();
-  // await pageObjects.runHook('useMicrophoneButtonClick', [], microphoneButtonClick => microphoneButtonClick());
-  // await expect(pageObjects.isDictating()).resolves.toBeFalsy();
+  const { driver, pageObjects } = await setupWebDriver({
+    props: {
+      webSpeechPonyfillFactory: () => window.WebSpeechMock
+    }
+  });
+
+  await driver.wait(uiConnected(), timeouts.directLine);
+
+  await pageObjects.runHook('useMicrophoneButtonClick', [], microphoneButtonClick => microphoneButtonClick());
+
+  await driver.wait(speechRecognitionStartCalled(), timeouts.ui);
+
+  await expect(
+    pageObjects.runHook('useMicrophoneButtonDisabled', [], microphoneButtonDisabled => microphoneButtonDisabled[0])
+  ).resolves.toBeTruthy();
+
+  await pageObjects.putSpeechRecognitionResult('recognizing', 'Hello');
+
+  await expect(pageObjects.isDictating()).resolves.toBeTruthy();
+
+  await pageObjects.runHook('useMicrophoneButtonClick', [], microphoneButtonClick => microphoneButtonClick());
+
+  await expect(pageObjects.isDictating()).resolves.toBeFalsy();
 });
diff --git a/packages/component/src/SendBox/MicrophoneButton.js b/packages/component/src/SendBox/MicrophoneButton.js
@@ -132,7 +132,6 @@ const useMicrophoneButtonClick = () => {
   ]);
 };
 
-// TODO: [P1] We temporarily not exporting this hook because it requires legacy HOC code
 function useMicrophoneButtonDisabled() {
   const [dictateState] = useDictateState();
   const [disabled] = useDisabled();