Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 'Index.list_documents' API wrapper. #1177

Merged
merged 4 commits into from
Oct 14, 2015
Merged

Add 'Index.list_documents' API wrapper. #1177

merged 4 commits into from
Oct 14, 2015

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Oct 13, 2015

No description provided.

Also, add a missing assertion for the case that no arguments are
passed to 'Client.list_indexes'.
@tseaver tseaver added the search label Oct 13, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 13, 2015
@@ -54,10 +54,10 @@ def test_list_indexes_defaults(self):
client = self._makeOne(self.PROJECT, creds)
conn = client.connection = _Connection(DATA)

zones, token = client.list_indexes()
indexes, token = client.list_indexes()

This comment was marked as spam.

This comment was marked as spam.

@dhermes
Copy link
Contributor

dhermes commented Oct 14, 2015

LGTM

tseaver added a commit that referenced this pull request Oct 14, 2015
Add 'Index.list_documents' API wrapper.
@tseaver tseaver merged commit 7196d60 into googleapis:master Oct 14, 2015
@tseaver tseaver deleted the search-list_documents branch October 14, 2015 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants