Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

refactor: add / use 'Client._list_resource' method - #446

Merged
tseaver merged 5 commits into
masterfrom
38-refactor-client-_list_resource
Jun 8, 2021
Merged

refactor: add / use 'Client._list_resource' method#446
tseaver merged 5 commits into
masterfrom
38-refactor-client-_list_resource

Conversation

@tseaver

@tseavertseaver commented May 20, 2021

Copy link
Copy Markdown
Contributor

Toward #38.

Note: this PR is based on 2acfb06 of PR #443. I will rebase to master when that PR lands. Done.

@product-auto-labelproduct-auto-labelBot added the api: storage Issues related to the googleapis/python-storage API. label May 20, 2021
@tseavertseaver changed the title 38 refactor client list resourcerefactor: add / use 'Client._list_resource' methodMay 20, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label May 20, 2021
@tseaver
tseaverforce-pushed the 38-refactor-client-_list_resource branch from 0af1e83 to 5024d1fCompareJune 8, 2021 16:57
@tseaver
tseaver marked this pull request as ready for review June 8, 2021 16:57
@tseaver
tseaver requested review from a teamJune 8, 2021 16:57
@tseavertseaver mentioned this pull request Jun 8, 2021
tseaver added 5 commits June 8, 2021 13:47
Also, adjust tests for 'Bucket.list_blobs' not to depend on anything but
calling 'Client.list_blobs'.
Toward #38
Also, provide explicit coverage for 'client._item_to_bucket' and
'client._item_to_hmack_key_metadata' helpers.
Toward #38
Also, provide explicit coverage for 'bucket._item_to_notification'.
Toward #38
@tseaver
tseaverforce-pushed the 38-refactor-client-_list_resource branch from 5024d1f to 68b762eCompareJune 8, 2021 17:47

@chrisrossichrisrossi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tseaver
tseaver merged commit 6bd8a20 into masterJun 8, 2021
@tseaver
tseaver deleted the 38-refactor-client-_list_resource branch June 8, 2021 18:25
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
* Adjust tests for 'Bucket.list_blobs' not to depend on anything but calling 'Client.list_blobs'.
* Provide explicit coverage for 'client._item_to_bucket' and 'client._item_to_hmack_key_metadata' helpers.
* Provide explicit coverage for 'bucket._item_to_notification'.
Toward googleapis#38
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
* Adjust tests for 'Bucket.list_blobs' not to depend on anything but calling 'Client.list_blobs'.
* Provide explicit coverage for 'client._item_to_bucket' and 'client._item_to_hmack_key_metadata' helpers.
* Provide explicit coverage for 'bucket._item_to_notification'.
Toward googleapis#38
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storageIssues related to the googleapis/python-storage API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@tseaver@chrisrossi