QoL: Fix tox.ini syntax and other minor things

- Fix tox.ini syntax

Modern tox versions require the expected environment variables to be
listed one by one on separate lines in tox.ini

- Add `venv` to list of ignored names for git

This is a common name for a local Python virtual environment. Less
typing than `virtualenv`

- Add `tox` to requirements.txt

Installing `tox` via `pip` has the advantage of including it in the
virtual environment, thus avoiding trouble on operating systems shipping
by default with python3.6 or older. It's also nice that `pip install -r
requirements.txt` is now sufficient to set up the testing environment,
after initializing the virtual environment with a moder-enough python
version.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
(cherry picked from commit 3a0f1f0ead)
This commit is contained in:
Moritz Röhrich 2023-02-28 12:19:54 +01:00 committed by Casey Bodley
parent e5dabbe45d
commit 7fadea7416
3 changed files with 5 additions and 1 deletions

1
.gitignore vendored
View file

@ -10,5 +10,6 @@
/*.egg-info /*.egg-info
/virtualenv /virtualenv
/venv
config.yaml config.yaml

View file

@ -12,3 +12,4 @@ pytz >=2011k
httplib2 httplib2
lxml lxml
pytest pytest
tox

View file

@ -3,5 +3,7 @@ envlist = py
[testenv] [testenv]
deps = -rrequirements.txt deps = -rrequirements.txt
passenv = S3TEST_CONF S3_USE_SIGV4 passenv =
S3TEST_CONF
S3_USE_SIGV4
commands = pytest {posargs} commands = pytest {posargs}