fix the coveralls report

Summary:
Ever since the switch from `test.sh` to `test.py`, we've been generating
the `.coverage` file inside `tests/`. Update `.travis.yml` to run
`coveralls` in the right place.

Also, handle the `--with-coverage` flag in a slightly more flexible way.

Test Plan:
Watch the build in my fork and make sure it looks like it's going to
work.

Reviewers: sean

Reviewed By: sean

Differential Revision: https://phabricator.buildinspace.com/D182
This commit is contained in:
Jack O'Connor 2015-01-26 16:20:08 -08:00
parent e951bf78e9
commit 3a9620e948
2 changed files with 3 additions and 3 deletions

View File

@ -8,4 +8,4 @@ install:
script: script:
- ./test.py --with-coverage -v - ./test.py --with-coverage -v
after_success: after_success:
coveralls cd tests && coveralls

View File

@ -35,8 +35,8 @@ def main():
env = os.environ.copy() env = os.environ.copy()
env['PYTHONPATH'] = REPO_ROOT env['PYTHONPATH'] = REPO_ROOT
args = sys.argv[1:] args = sys.argv[1:]
if len(args) > 0 and args[0] == '--with-coverage': if len(args) > 0 and '--with-coverage' in args:
args.pop(0) args.remove('--with-coverage')
command_start = ['coverage', 'run'] command_start = ['coverage', 'run']
else: else:
command_start = [sys.executable] command_start = [sys.executable]