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

Delta Between Two Patch Sets: tox.ini

Issue 29501558: Issue 5383 - Add tests for the Chrome and Firefox packagers (Closed)
Left Patch Set: Refactoring tests, focussing on webext (chrome, gecko, edge) Created Sept. 6, 2017, 8:58 a.m.
Right Patch Set: Addressing Vasily's comments Created Oct. 22, 2017, 11:11 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « tests/test_packagerWebExt.py ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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 ignore = D1
7 build.py : A102,A104,A107,A201,A206,A302,E501,E711,F401,N802,N803,N806 7 per-file-ignores =
8 chainedconfigparser.py : A107,E501 8 build.py : A104,A107,A201,A206,A302,E501,E711,N802,N803,N806
9 ensure_dependencies.py : A102,A107,A108,A302,E129,E501,E713,E721,F821 9 chainedconfigparser.py : A107,E501,D205,D208,D400
10 localeTools.py : A103,A104,A107,A108,A206,A301,A302,E129,E501,E711,E713,F401 ,F812,N802,N803,N806 10 ensure_dependencies.py : A102,A107,A108,D400,A302,E129,E501,E713,E721,F821
11 localeTools.py : A104,A107,A301,A302,E501,E713,F401,N802,N803,N806
11 packager.py : A102,A107,A206,A302,E501,E711,N802,N803,N806 12 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 13 packagerChrome.py : A101,A104,A107,A111,A112,A302,E501,E711,F841,N802,N803,N 806
13 packagerGecko.py : A101,A104,A105,A107,A108,A201,A302,E501,E711,E713,F401,N8 02,N803,N806 14 publicSuffixListUpdater.py : A108,D200,D202,D205,D400,D401,D403,E501,F821,N8 02,N803
14 packagerSafari.py : A101,A107,A301,A302,E501,N802,N803,N806 15 releaseAutomation.py : A102,A107,A108,A302,D202,D400,D401,E501,F401,N803,N80 6
15 publicSuffixListUpdater.py : A108,E501,F821,N802,N803 16
16 releaseAutomation.py : A102,A107,A108,A302,E501,F401,N803,N806 17 [coverage:run]
18 omit = tests/*
19
20 [coverage:report]
21 show_missing = True
17 22
18 [testenv] 23 [testenv]
24 whitelist_externals = npm
19 setenv = 25 setenv =
20 PYTHONPATH = {toxinidir}/.. 26 PYTHONPATH = {toxinidir}/..
21 deps = 27 deps =
22 pycrypto 28 pycrypto
23 pillow
24 pytest 29 pytest
25 pytest-cov 30 pytest-cov
26 jinja2 31 jinja2
27 # flake8-putty is not yet compatible with flake8 3.0 32 flake8
28 flake8==2.* 33 flake8-per-file-ignores
29 flake8-putty 34 flake8-docstrings
30 pep8-naming 35 pep8-naming
31 hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake 8-eyeo 36 hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake 8-eyeo
32 commands = 37 commands =
33 pytest tests --capture=sys --cov-config .coveragerc --cov=buildtools --cov-r eport=term-missing 38 npm install --no-package-lock --only=production
39 pytest tests --capture=sys --cov=buildtools
34 flake8 40 flake8
LEFTRIGHT

Powered by Google App Engine
This is Rietveld