-
Notifications
You must be signed in to change notification settings - Fork 159
tests: move system tests into a package #51
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
Conversation
4e4e810
to
bc1df28
Compare
There was a problem hiding this comment.
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)
"logo": {"path": DATA_DIRNAME + "/CloudPlatform_128px_Retina.png"}, | ||
"big": {"path": DATA_DIRNAME + "/five-point-one-mb-file.zip"}, | ||
"simple": {"path": DATA_DIRNAME + "/simple.txt"}, |
There was a problem hiding this comment.
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)
Prep for factoring out e.g. signing tests into separate modules. Port of googleapis/google-cloud-python#9777 into new reposiory.
bc1df28
to
33c2407
Compare
Ugh, new systest flake reported in #55. |
Prep for factoring out e.g. signing tests into separate modules. Port of googleapis/google-cloud-python#9777 into new reposiory.
Prep for factoring out e.g. signing tests into separate modules. Port of googleapis/google-cloud-python#9777 into new reposiory.
Prep for factoring out e.g. signing tests into separate modules.
Port of googleapis/google-cloud-python#9777
into new reposiory.