Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented May 20, 2021

Toward #38.

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

@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/python-storage API. label May 20, 2021
@tseaver tseaver changed the title 38 refactor client list resource refactor: add / use 'Client._list_resource' method May 20, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label May 20, 2021
@tseaver tseaver force-pushed the 38-refactor-client-_list_resource branch from 0af1e83 to 5024d1f Compare June 8, 2021 16:57
@tseaver tseaver marked this pull request as ready for review June 8, 2021 16:57
@tseaver tseaver requested a review from a team June 8, 2021 16:57
@tseaver tseaver requested a review from a team as a code owner June 8, 2021 16:57
@tseaver tseaver 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 tseaver force-pushed the 38-refactor-client-_list_resource branch from 5024d1f to 68b762e Compare June 8, 2021 17:47
Copy link

@chrisrossi chrisrossi left a comment

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 master Jun 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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/python-storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants