Merge pull request #2638 from daspecster/unify-speech-async-operation-results

Clean up async docs and unify for future GAPIC async.

Author: daspecster

packages/google-cloud-speech/google/cloud/speech/operation.py

@@ -123,8 +123,9 @@ def _update(self, response):
         raw_results = response.get('response', {}).get('results', None)
         results = []
         if raw_results:
-            for result in raw_results[0]['alternatives']:
-                results.append(Transcript.from_api_repr(result))
+            for result in raw_results:
+                for alternative in result['alternatives']:
+                    results.append(Transcript.from_api_repr(alternative))
         if metadata:
             self._metadata = Metadata.from_api_repr(metadata)
 

packages/google-cloud-speech/unit_tests/test_operation.py

@@ -36,7 +36,7 @@ def test_ctor_defaults(self):
 
     def test_from_api_repr(self):
         from unit_tests._fixtures import OPERATION_COMPLETE_RESPONSE
-        from google.cloud.speech.operation import Transcript
+        from google.cloud.speech.transcript import Transcript
         from google.cloud.speech.metadata import Metadata
         RESPONSE = OPERATION_COMPLETE_RESPONSE
 
@@ -46,10 +46,12 @@ def test_from_api_repr(self):
         self.assertEqual('123456789', operation.name)
         self.assertTrue(operation.complete)
 
+        self.assertEqual(len(operation.results), 1)
         self.assertIsInstance(operation.results[0], Transcript)
         self.assertEqual(operation.results[0].transcript,
                          'how old is the Brooklyn Bridge')
-        self.assertEqual(operation.results[0].confidence, 0.98267895)
+        self.assertEqual(operation.results[0].confidence,
+                         0.98267895)
         self.assertTrue(operation.complete)
         self.assertIsInstance(operation.metadata, Metadata)
         self.assertEqual(operation.metadata.progress_percent, 100)