Skip to content

Conversation

@daspecster
Copy link
Contributor

@daspecster daspecster commented Oct 31, 2016

This adds the speech sync_recognize() JSON/REST system test.

Based from #2648

@daspecster daspecster added testing api: speech Issues related to the Speech-to-Text API. labels Oct 31, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 31, 2016
@daspecster daspecster mentioned this pull request Oct 31, 2016
14 tasks

def setUpModule():
Config.CLIENT = speech.Client()
Config.CLIENT = speech.Client

This comment was marked as spam.

max_alternatives=None):
client = Config.CLIENT
max_alternatives=None, use_gax=True):
client = Config.CLIENT(use_gax=use_gax)

This comment was marked as spam.

results = self._make_sync_request(content=file_obj.read(),
max_alternatives=2,
use_gax=False)
second_alternative = results[1]

This comment was marked as spam.

max_alternatives=2,
use_gax=False)
second_alternative = results[1]
self.assertEqual(len(results), 2)

This comment was marked as spam.

@daspecster daspecster force-pushed the add-speech-sync-rest-system-tests branch 2 times, most recently from 600828f to 0fcd66b Compare October 31, 2016 17:26
@daspecster
Copy link
Contributor Author

I'm not sure how 63cb2fa got in here? I rebased master in...

@daspecster daspecster force-pushed the add-speech-sync-rest-system-tests branch from 0fcd66b to 36843d4 Compare October 31, 2016 17:29
@daspecster
Copy link
Contributor Author

There we go!

@daspecster
Copy link
Contributor Author

Actually..this is pointless now since the switch to the env usage.

@daspecster daspecster closed this Oct 31, 2016
@daspecster daspecster deleted the add-speech-sync-rest-system-tests branch October 31, 2016 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: speech Issues related to the Speech-to-Text API. cla: yes This human has signed the Contributor License Agreement. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants