tox.ini - Issue 29968569: Issue 4014 - Publish python-abp on PyPI

Side by Side Diff: tox.ini

Issue 29968569: Issue 4014 - Publish python-abp on PyPI (Closed) Base URL: https://hg.adblockplus.org/python-abp/

Patch Set: Created Dec. 27, 2018, 11:31 p.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
1 [tox]	1 [tox]

2 envlist = py27,py35,py36,pypy,coverage,flake8	2 envlist = py27,py35,py36,pypy,coverage2,coverage3,flake8

3	3

4 [flake8]	4 [flake8]

5 ignore = D107, N801, W503	5 ignore = D107, N801, W503

6	6

7 [testenv]	7 [testenv]

8 deps =	8 deps =

9 pytest	9 pytest

10 mock	10 mock

11 commands =	11 commands =

12 py.test tests	12 py.test tests

13	13

14 [testenv:coverage]	14 # We have to install our package in development mode so that pytest-cov can

	15 # find the .coverage and .coveragerc files.

	16 usedevelop = true

	17

	18 [testenv:coverage2]

	19 basepython = python2

	20 setenv = PY_VER = 2

15 deps =	21 deps =

16 pytest-cov	22 pytest-cov

17 mock	23 mock

18 commands =	24 commands =

19 py.test --cov=abp --cov-report=term-missing tests	25 py.test --cov=abp --cov-report=term-missing tests

20	26

21 # We have to install our package in development mode	27 [testenv:coverage3]
	Sebastian Noack 2018/12/28 00:22:51 What's the point of using a separate environment f What's the point of using a separate environment for coverage, if you want to run coverage for both Python 2 and 3 anyway? rhowell 2018/12/28 21:50:08 By default, tox only checks coverage for the versi Show quoted text On 2018/12/28 00:22:51, Sebastian Noack wrote: > What's the point of using a separate environment for coverage, if you want to > run coverage for both Python 2 and 3 anyway? By default, tox only checks coverage for the version of Python that is the current default on the local system. Seems like it would be best to check both Python 2 & 3. But, I ran into an error trying to change the basepython variable in one environment: py._vendored_packages.iniconfig.ParseError: tox.ini:26: duplicate name 'basepython' This was a fairly quick & dirty work-around. I did some research, but wasn't able to find a better way yet, though this article was very informative: https://anonbadger.wordpress.com/2017/11/24/better-unittest-coverage-stats-ex... I'm open for discussion or suggestions on this, though. Sebastian Noack 2018/12/28 22:39:02 If you just run coverage reports for every environ Show quoted text On 2018/12/28 21:50:08, rhowell wrote: > On 2018/12/28 00:22:51, Sebastian Noack wrote: > > What's the point of using a separate environment for coverage, if you want to > > run coverage for both Python 2 and 3 anyway? > > By default, tox only checks coverage for the version of Python that is the > current default on the local system. Seems like it would be best to check both > Python 2 & 3. But, I ran into an error trying to change the basepython variable > in one environment: > > py._vendored_packages.iniconfig.ParseError: tox.ini:26: duplicate name > 'basepython' > > This was a fairly quick & dirty work-around. I did some research, but wasn't > able to find a better way yet, though this article was very informative: > https://anonbadger.wordpress.com/2017/11/24/better-unittest-coverage-stats-ex... > > I'm open for discussion or suggestions on this, though. If you just run coverage reports for every environment you run tests on, you don't have to explicitly set the basepython setting. However, the tricky part is to get the Python major version into the .coveragerc. This should do the trick: [testenv] deps = pytest-cov mock commands = python -c 'import sys; open(".coveragerc", "w").write("[report]\nexclude_lines = pragma: no py{} cover".format(sys.version_info[0]))' py.test --cov=abp --cov-report=term-missing tests rhowell 2018/12/29 01:30:51 Done. Show quoted text On 2018/12/28 22:39:02, Sebastian Noack wrote: > On 2018/12/28 21:50:08, rhowell wrote: > > On 2018/12/28 00:22:51, Sebastian Noack wrote: > > > What's the point of using a separate environment for coverage, if you want > to > > > run coverage for both Python 2 and 3 anyway? > > > > By default, tox only checks coverage for the version of Python that is the > > current default on the local system. Seems like it would be best to check both > > Python 2 & 3. But, I ran into an error trying to change the basepython > variable > > in one environment: > > > > py._vendored_packages.iniconfig.ParseError: tox.ini:26: duplicate name > > 'basepython' > > > > This was a fairly quick & dirty work-around. I did some research, but wasn't > > able to find a better way yet, though this article was very informative: > > > https://anonbadger.wordpress.com/2017/11/24/better-unittest-coverage-stats-ex... > > > > I'm open for discussion or suggestions on this, though. > > If you just run coverage reports for every environment you run tests on, you > don't have to explicitly set the basepython setting. However, the tricky part is > to get the Python major version into the .coveragerc. This should do the trick: > > [testenv] > deps = > pytest-cov > mock > commands = > python -c 'import sys; open(".coveragerc", > "w").write("[report]\nexclude_lines = pragma: no py{} > cover".format(sys.version_info[0]))' > py.test --cov=abp --cov-report=term-missing tests Done.
22 # so that pytest-cov can find the .coverage file.	28 basepython = python3

23 usedevelop = true	29 setenv = PY_VER = 3

	30 deps =

	31 pytest-cov

	32 mock

	33 commands =

	34 py.test --cov=abp --cov-report=term-missing tests

24	35

25 [testenv:flake8]	36 [testenv:flake8]

26 basepython = python3	37 basepython = python3

27 skip_install = true	38 skip_install = true

28 deps =	39 deps =

29 flake8	40 flake8

30 flake8-docstrings	41 flake8-docstrings

31 flake8-commas	42 flake8-commas

32 pep8-naming	43 pep8-naming

33 hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake 8-eyeo	44 hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake 8-eyeo

34 commands =	45 commands =

35 flake8 abp	46 flake8 abp

36 flake8 --extend-ignore=D1 tests setup.py	47 flake8 --extend-ignore=D1 tests setup.py

OLD	NEW

« setup.py ('K') | « tests/test_fs_source.py ('k') | no next file » | no next file with comments »