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 Python 3.5 system-test target in tox.ini #2670

Closed
duggelz opened this issue Nov 3, 2016 · 7 comments
Closed

Add Python 3.5 system-test target in tox.ini #2670

duggelz opened this issue Nov 3, 2016 · 7 comments
Assignees
Labels

Comments

@duggelz
Copy link

duggelz commented Nov 3, 2016

Currently it has 'system-test' -> python 2.7 and 'system-test3' -> python 3.4

See also #2350

@dhermes dhermes added the testing label Nov 3, 2016
@dhermes
Copy link
Contributor

dhermes commented Nov 3, 2016

@duggelz Thanks for taking an interest in our test setup. Any particular reason this matters to you? Keep in mind you can already run the system tests in Python 3.5 if you please via:

$ git clone https://github.com/GoogleCloudPlatform/google-cloud-python
$ cd google-cloud-python/
$ virtualenv --python=python3.5 venv
$ source venv/bin/activate
(venv) $ venv/bin/pip install core/
(venv) $ venv/bin/pip install \
> bigtable/ \
> storage/ \
> datastore/ \
> bigquery/ \
> pubsub/ \
> logging/ \
> dns/ \
> language/ \
> error_reporting/ \
> resource_manager/ \
> monitoring/ \
> vision/ \
> translate/ \
> speech/
(venv) $ venv/bin/python system_tests/attempt_system_tests.py
(venv) $ deactivate
$ rm -fr venv/

@duggelz
Copy link
Author

duggelz commented Nov 15, 2016

I can run the tests as you describe, but a tox target is much nicer, since I already use tox to run the tests against Python 2.7 and 3.4

@dhermes
Copy link
Contributor

dhermes commented Nov 15, 2016

Anything wrong with just making system-tests3 use py3.5 instead of py3.4?

@duggelz
Copy link
Author

duggelz commented Nov 15, 2016

Sure, I can do that. But your documentation specifically calls out 2.7, 3.4, and 3.5 as the supported versions, so it seems like you'd want to test on all those versions.

- The feature must work fully on the following CPython versions:  2.7,
  3.4, and 3.5 on both UNIX and Windows.

@tseaver
Copy link
Contributor

tseaver commented Nov 16, 2016

Sure, I can do that. But your documentation specifically calls out 2.7, 3.4, and 3.5 as the supported versions, so it seems like you'd want to test on all those versions.

Our unit test coverage is really good: it is super unlikely that running system tests on 3.4 and 3.5 will show any 3.5-only breakage not already caught by the unit tests (which we do run on both). The difference is that system tests take minutes to run, rather than seconds.

@duggelz
Copy link
Author

duggelz commented Nov 16, 2016

Done. Please take a look #2741

@duggelz
Copy link
Author

duggelz commented Feb 1, 2021

This issue is closed, right? Do I need to do something more?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants