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

Unified Diff: tox.ini

Issue 29342955: Issue 3985 - Unified test runner for sitescripts (Closed)
Patch Set: Remove filterhits, remove dependencies for notifications tests, make everything run in one environm… Created May 24, 2016, 6:09 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « README.md ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tox.ini
===================================================================
new file mode 100644
--- /dev/null
+++ b/tox.ini
@@ -0,0 +1,37 @@
+[tox]
+envlist = py27
+skipsdist = true
+
+[pytest]
+# The names of the test files here don't start with 'test_' so we need
+# to instruct pytest to load tests from all files.
+python_files = *.py
+
+[run]
+# Exclude tests from coverage.
+omit = */test/*
+
+[testenv]
+basepython=python2
+deps =
+ pytest
+ pytest-cov
+ mock
+ jinja2
+setenv =
+ SITESCRIPTS_CONFIG = ./.sitescripts.test
+whitelist_externals =
+ bash
+ rm
+commands =
+ bash -c "sed -e 's/sitescripts\.[a-zA-Z_]*\.web.*/# &/' .sitescripts.example >.sitescripts.test"
Sebastian Noack 2016/05/25 07:51:36 Perhaps we should only exclude the web handlers th
Vasily Kuznetsov 2016/05/25 15:17:40 Done.
+ python ensure_dependencies.py
+ py.test \
Sebastian Noack 2016/05/25 07:51:36 I think we should also run flake8, probably with a
Vasily Kuznetsov 2016/05/25 15:17:40 Done. Not sure if this is very meaningful at this
+ --cov-config tox.ini --cov-report term \
+ --cov sitescripts.hg \
+ --cov sitescripts.notifications \
+ --cov sitescripts.stats \
+ sitescripts/hg/test \
+ sitescripts/notifications/test \
+ sitescripts/stats/test
+ rm .sitescripts.test
« no previous file with comments | « README.md ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld