Delta Between Two Patch Sets: tox.ini
Issue 29501558:
Issue 5383 - Add tests for the Chrome and Firefox packagers (Closed)
Left Patch Set: Addressing Dave's comments
Created Oct. 17, 2017, 12:41 p.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.
« 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') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| LEFT | RIGHT |
|---|---|
| 1 [tox] | 1 [tox] |
| 2 envlist = py27 | 2 envlist = py27 |
| 3 skipsdist = true | 3 skipsdist = true |
| 4 | 4 |
| 5 [flake8] | 5 [flake8] |
| 6 ignore = D1 | 6 ignore = D1 |
| 7 per-file-ignores = | 7 per-file-ignores = |
| 8 build.py : A104,A107,A201,A206,A302,E501,E711,N802,N803,N806 | 8 build.py : A104,A107,A201,A206,A302,E501,E711,N802,N803,N806 |
| 9 chainedconfigparser.py : A107,E501,D205,D208,D400 | 9 chainedconfigparser.py : A107,E501,D205,D208,D400 |
| 10 ensure_dependencies.py : A102,A107,A108,D400,A302,E129,E501,E713,E721,F821 | 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 localeTools.py : A104,A107,A301,A302,E501,E713,F401,N802,N803,N806 |
| 12 packager.py : A102,A107,A206,A302,E501,E711,N802,N803,N806 | 12 packager.py : A102,A107,A206,A302,E501,E711,N802,N803,N806 |
| 13 packagerChrome.py : A101,A104,A107,A111,A112,A302,E501,E711,F841,N802,N803,N 806 | 13 packagerChrome.py : A101,A104,A107,A111,A112,A302,E501,E711,F841,N802,N803,N 806 |
| 14 publicSuffixListUpdater.py : A108,D200,D202,D205,D400,D401,D403,E501,F821,N8 02,N803 | 14 publicSuffixListUpdater.py : A108,D200,D202,D205,D400,D401,D403,E501,F821,N8 02,N803 |
| 15 releaseAutomation.py : A102,A107,A108,A302,D202,D400,D401,E501,F401,N803,N80 6 | 15 releaseAutomation.py : A102,A107,A108,A302,D202,D400,D401,E501,F401,N803,N80 6 |
| 16 | 16 |
| 17 [coverage:run] | 17 [coverage:run] |
| 18 omit = tests/* | 18 omit = tests/* |
| 19 | 19 |
| 20 [coverage:report] | 20 [coverage:report] |
| 21 show_missing = True | 21 show_missing = True |
| 22 | 22 |
| 23 [testenv] | 23 [testenv] |
| 24 recreate = true | 24 whitelist_externals = npm |
|
Sebastian Noack
2017/10/17 22:19:10
Any reason we force the venv to be recreated on ev
tlucas
2017/10/18 11:23:12
This was an unintended leftover - removed it.
| |
| 25 setenv = | 25 setenv = |
| 26 PYTHONPATH = {toxinidir}/.. | 26 PYTHONPATH = {toxinidir}/.. |
| 27 deps = | 27 deps = |
| 28 pycrypto | 28 pycrypto |
| 29 pytest | 29 pytest |
| 30 pytest-cov | 30 pytest-cov |
| 31 jinja2 | 31 jinja2 |
| 32 flake8 | 32 flake8 |
| 33 flake8-per-file-ignores | 33 flake8-per-file-ignores |
| 34 flake8-docstrings | 34 flake8-docstrings |
| 35 pep8-naming | 35 pep8-naming |
| 36 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 |
| 37 commands = | 37 commands = |
| 38 python ensure_dependencies.py --nodejs-only | 38 npm install --no-package-lock --only=production |
| 39 pytest tests --capture=sys --cov=buildtools | 39 pytest tests --capture=sys --cov=buildtools |
| 40 flake8 | 40 flake8 |
| LEFT | RIGHT |
« tests/test_packagerWebExt.py ('k')
|
no next file »
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
