Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: tox.ini

Issue 29501558: Issue 5383 - Add tests for the Chrome and Firefox packagers (Closed)
Patch Set: Addressing pep8 error from last review Created Aug. 11, 2017, 12:12 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
« tests/test_packagerGecko.py ('K') | « tests/tools.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 [tox] 1 [tox]
2 envlist = py27 2 envlist = py27
3 skipsdist = true 3 skipsdist = true
4 4
5 [flake8] 5 [flake8]
6 putty-ignore = 6 putty-ignore =
7 build.py : A102,A104,A107,A201,A206,A302,E501,E711,F401,N802,N803,N806 7 build.py : A102,A104,A107,A201,A206,A302,E501,E711,F401,N802,N803,N806
8 chainedconfigparser.py : A107,E501 8 chainedconfigparser.py : A107,E501
9 ensure_dependencies.py : A102,A107,A108,A302,E129,E501,E713,E721,F821 9 ensure_dependencies.py : A102,A107,A108,A302,E129,E501,E713,E721,F821
10 localeTools.py : A103,A104,A107,A108,A206,A301,A302,E129,E501,E711,E713,F401 ,F812,N802,N803,N806 10 localeTools.py : A103,A104,A107,A108,A206,A301,A302,E129,E501,E711,E713,F401 ,F812,N802,N803,N806
11 packager.py : A102,A107,A206,A302,E501,E711,N802,N803,N806 11 packager.py : A102,A107,A206,A302,E501,E711,N802,N803,N806
12 packagerChrome.py : A101,A104,A105,A107,A108,A111,A112,A302,E129,E501,E711,E 713,E731,F401,F841,N802,N803,N806 12 packagerChrome.py : A101,A104,A105,A107,A108,A111,A112,A302,E129,E501,E711,E 713,E731,F401,F841,N802,N803,N806
13 packagerGecko.py : A101,A104,A105,A107,A108,A201,A302,E501,E711,E713,F401,N8 02,N803,N806 13 packagerGecko.py : A101,A104,A105,A107,A108,A201,A302,E501,E711,E713,F401,N8 02,N803,N806
14 packagerSafari.py : A101,A107,A301,A302,E501,N802,N803,N806 14 packagerSafari.py : A101,A107,A301,A302,E501,N802,N803,N806
15 publicSuffixListUpdater.py : A108,E501,F821,N802,N803 15 publicSuffixListUpdater.py : A108,E501,F821,N802,N803
16 releaseAutomation.py : A102,A107,A108,A302,E501,F401,N803,N806 16 releaseAutomation.py : A102,A107,A108,A302,E501,F401,N803,N806
17 17
18 [testenv] 18 [testenv]
19 setenv = 19 setenv =
20 PYTHONPATH = {toxinidir}/.. 20 PYTHONPATH = {toxinidir}/..
21 deps = 21 deps =
22 pycrypto
23 pillow
22 pytest 24 pytest
23 jinja2 25 jinja2
24 # flake8-putty is not yet compatible with flake8 3.0 26 # flake8-putty is not yet compatible with flake8 3.0
25 flake8==2.* 27 flake8==2.*
26 flake8-putty 28 flake8-putty
27 pep8-naming 29 pep8-naming
28 hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake 8-eyeo 30 hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake 8-eyeo
29 commands = 31 commands =
30 py.test tests 32 py.test tests --capture=sys
Vasily Kuznetsov 2017/08/11 16:46:02 While we're at it, let's add coverage measurement
tlucas 2017/08/14 14:23:18 Done. But py.test is valid too - why did you sugge
31 flake8 33 flake8
OLDNEW
« tests/test_packagerGecko.py ('K') | « tests/tools.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld