fix(types): improve types for voiceSearch (#3778)

* fix(types): make isSpeechFinal non-optional boolean * fix(types): make transcript non-optional string * chore(voiceSearch): assign an empty string if no transcript
algolia · May 15, 2019 · ed2d61a · ed2d61a
1 parent 36e3a3d
commit ed2d61a
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 16 deletions.
diff --git a/src/components/VoiceSearch/__tests__/VoiceSearch-test.tsx b/src/components/VoiceSearch/__tests__/VoiceSearch-test.tsx
@@ -13,6 +13,8 @@ const defaultProps: VoiceSearchProps = {
   toggleListening: () => {},
   voiceListeningState: {
     status: 'initial',
+    isSpeechFinal: false,
+    transcript: '',
   },
   templates: {
     buttonText: 'button',

diff --git a/src/lib/voiceSearchHelper/__tests__/index-test.ts b/src/lib/voiceSearchHelper/__tests__/index-test.ts
@@ -32,9 +32,9 @@ describe('VoiceSearchHelper', () => {
     const voiceSearchHelper = getVoiceSearchHelper();
     expect(voiceSearchHelper.getState()).toEqual({
       errorCode: undefined,
-      isSpeechFinal: undefined,
+      isSpeechFinal: false,
       status: 'initial',
-      transcript: undefined,
+      transcript: '',
     });
   });
 

diff --git a/src/lib/voiceSearchHelper/index.ts b/src/lib/voiceSearchHelper/index.ts
@@ -13,8 +13,8 @@ export type VoiceSearchHelperParams = {
 
 export type VoiceListeningState = {
   status: string;
-  transcript?: string;
-  isSpeechFinal?: boolean;
+  transcript: string;
+  isSpeechFinal: boolean;
   errorCode?: string;
 };
 
@@ -37,8 +37,8 @@ export default function voiceSearchHelper({
     (window as any).SpeechRecognition;
   const getDefaultState = (status: string): VoiceListeningState => ({
     status,
-    transcript: undefined,
-    isSpeechFinal: undefined,
+    transcript: '',
+    isSpeechFinal: false,
     errorCode: undefined,
   });
   let state: VoiceListeningState = getDefaultState(STATUS_INITIAL);
@@ -89,9 +89,10 @@ export default function voiceSearchHelper({
       setState({
         status: STATUS_RECOGNIZING,
         transcript:
-          event.results[0] &&
-          event.results[0][0] &&
-          event.results[0][0].transcript,
+          (event.results[0] &&
+            event.results[0][0] &&
+            event.results[0][0].transcript) ||
+          '',
         isSpeechFinal: event.results[0] && event.results[0].isFinal,
       });
       if (searchAsYouSpeak && state.transcript) {

diff --git a/src/widgets/voice-search/__tests__/__snapshots__/voice-search-test.ts.snap b/src/widgets/voice-search/__tests__/__snapshots__/voice-search-test.ts.snap
@@ -21,9 +21,9 @@ exports[`voiceSearch() Rendering renders during init() 1`] = `
   voiceListeningState={
     Object {
       "errorCode": undefined,
-      "isSpeechFinal": undefined,
+      "isSpeechFinal": false,
       "status": "initial",
-      "transcript": undefined,
+      "transcript": "",
     }
   }
 />
@@ -50,9 +50,9 @@ exports[`voiceSearch() Rendering renders during render() 1`] = `
   voiceListeningState={
     Object {
       "errorCode": undefined,
-      "isSpeechFinal": undefined,
+      "isSpeechFinal": false,
       "status": "initial",
-      "transcript": undefined,
+      "transcript": "",
     }
   }
 />
@@ -79,9 +79,9 @@ exports[`voiceSearch() Rendering renders during render() 2`] = `
   voiceListeningState={
     Object {
       "errorCode": undefined,
-      "isSpeechFinal": undefined,
+      "isSpeechFinal": false,
       "status": "initial",
-      "transcript": undefined,
+      "transcript": "",
     }
   }
 />

diff --git a/stories/voice-search.stories.ts b/stories/voice-search.stories.ts
@@ -171,7 +171,7 @@ storiesOf('VoiceSearch', module)
             status({ isListening, transcript }) {
               return `
                 <div class="layer listening-${isListening}">
-                  <span>${transcript ? transcript : ''}</span>
+                  <span>${transcript}</span>
                 </div>
               `;
             },