Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

tests: move system tests into a package#51

Merged
tseaver merged 3 commits into
mastergoogleapis/python-storage:masterfrom
tests-port-gcp-9777googleapis/python-storage:tests-port-gcp-9777Copy head branch name to clipboard
Feb 12, 2020
Merged

tests: move system tests into a package#51
tseaver merged 3 commits into
mastergoogleapis/python-storage:masterfrom
tests-port-gcp-9777googleapis/python-storage:tests-port-gcp-9777Copy head branch name to clipboard

Conversation

@tseaver

@tseaver tseaver commented Feb 11, 2020

Copy link
Copy Markdown
Contributor

Prep for factoring out e.g. signing tests into separate modules.

Port of googleapis/google-cloud-python#9777
into new reposiory.

@tseaver tseaver added api: storage Issues related to the googleapis/python-storage API. type: process A process-related concern. May include testing, release, or the like. labels Feb 11, 2020
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 11, 2020
@tseaver tseaver force-pushed the tests-port-gcp-9777 branch from 4e4e810 to bc1df28 Compare February 11, 2020 22:20
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020

@plamut plamut left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, mentioned a probable existing incompatibility with Windows systems.

The CI failure seems like flakiness, will re-run.
(edit: or maybe not, the same error also occurred on a re-run)

Comment on lines +466 to +459
"logo": {"path": DATA_DIRNAME + "/CloudPlatform_128px_Retina.png"},
"big": {"path": DATA_DIRNAME + "/five-point-one-mb-file.zip"},
"simple": {"path": DATA_DIRNAME + "/simple.txt"},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Won't that be an error on Windows that use a different path separator?

(the change itself is good, referring to the existing state here)

@plamut plamut added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@release-please release-please Bot closed this Feb 12, 2020
@tseaver tseaver reopened this Feb 12, 2020
Prep for factoring out e.g. signing tests into separate modules.

Port of googleapis/google-cloud-python#9777
into new reposiory.
@tseaver tseaver force-pushed the tests-port-gcp-9777 branch from bc1df28 to 33c2407 Compare February 12, 2020 17:48
@tseaver

tseaver commented Feb 12, 2020

Copy link
Copy Markdown
Contributor Author

Ugh, new systest flake reported in #55.

@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@tseaver tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 12, 2020
@tseaver tseaver requested a review from plamut February 12, 2020 21:21
@tseaver tseaver merged commit b9c0bca into master Feb 12, 2020
@tseaver tseaver deleted the tests-port-gcp-9777 branch February 12, 2020 23:50
tseaver added a commit that referenced this pull request Nov 10, 2020
tseaver added a commit that referenced this pull request Nov 10, 2020
gcf-merge-on-green Bot pushed a commit that referenced this pull request Nov 10, 2020
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Prep for factoring out e.g. signing tests into separate modules.

Port of googleapis/google-cloud-python#9777
into new reposiory.
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Prep for factoring out e.g. signing tests into separate modules.

Port of googleapis/google-cloud-python#9777
into new reposiory.
cojenco pushed a commit to cojenco/python-storage that referenced this pull request Oct 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/python-storage API. cla: yes This human has signed the Contributor License Agreement. type: process A process-related concern. May include testing, release, or the like.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.