admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

test-bot: fix coverage on Travis CI

Browse Source 
Make sure to call `brew tests` only once with `--coverage` to avoid
expensive multiple runs and to prevent later runs from overwriting
previously sent results to Coveralls. (The previous setup overwrote the
results from a regular run with results from the `--generic` run.)

The `--no-compat` variant without any other options specified seemed to
be the most appropriate for a coverage report.

Closes #546.

Signed-off-by: Martin Afanasjew <martin@afanasjew.de>
This commit is contained in:
Martin Afanasjew 2016-07-17 23:01:43 +02:00
parent 9cf508c6e0
commit 5768e32ad2
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Library/Homebrew/dev-cmd/test-bot.rb
View File

@ -659,14 +659,15 @@ module Homebrew
if @tap.nil? if @tap.nil?
tests_args = [] tests_args = []
tests_args_coverage = []
if RUBY_TWO if RUBY_TWO
tests_args << "--official-cmd-taps" tests_args << "--official-cmd-taps"
tests_args << "--coverage" if ENV["TRAVIS"] tests_args_coverage << "--coverage" if ENV["TRAVIS"]
end end
test "brew", "tests", *tests_args test "brew", "tests", *tests_args
test "brew", "tests", "--generic", "--only=integration_cmds", test "brew", "tests", "--generic", "--only=integration_cmds",
*tests_args *tests_args
test "brew", "tests", "--no-compat" test "brew", "tests", "--no-compat", *tests_args_coverage
test "brew", "readall", "--syntax" test "brew", "readall", "--syntax"
# test update from origin/master to current commit. # test update from origin/master to current commit.
test "brew", "update-test" test "brew", "update-test"