From 5768e32ad20208e8c5c1029ef2c9b47abd2caa2d Mon Sep 17 00:00:00 2001 From: Martin Afanasjew Date: Sun, 17 Jul 2016 23:01:43 +0200 Subject: test-bot: fix coverage on Travis CI 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 --- Library/Homebrew/dev-cmd/test-bot.rb | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'Library') diff --git a/Library/Homebrew/dev-cmd/test-bot.rb b/Library/Homebrew/dev-cmd/test-bot.rb index d2c5b317b..548c973a3 100644 --- a/Library/Homebrew/dev-cmd/test-bot.rb +++ b/Library/Homebrew/dev-cmd/test-bot.rb @@ -659,14 +659,15 @@ module Homebrew if @tap.nil? tests_args = [] + tests_args_coverage = [] if RUBY_TWO tests_args << "--official-cmd-taps" - tests_args << "--coverage" if ENV["TRAVIS"] + tests_args_coverage << "--coverage" if ENV["TRAVIS"] end test "brew", "tests", *tests_args test "brew", "tests", "--generic", "--only=integration_cmds", *tests_args - test "brew", "tests", "--no-compat" + test "brew", "tests", "--no-compat", *tests_args_coverage test "brew", "readall", "--syntax" # test update from origin/master to current commit. test "brew", "update-test" -- cgit v1.2.3