From df7246fdea9479d1131473b0babbecf35558dfda Mon Sep 17 00:00:00 2001 From: Bruce Steedman Date: Sat, 1 Oct 2016 09:26:25 +0100 Subject: [PATCH] make assertions conditional on ARGV.verbose? --- Library/Homebrew/test/test_download_strategies.rb | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/Library/Homebrew/test/test_download_strategies.rb b/Library/Homebrew/test/test_download_strategies.rb index 99c4ad1af2..e349401855 100644 --- a/Library/Homebrew/test/test_download_strategies.rb +++ b/Library/Homebrew/test/test_download_strategies.rb @@ -22,15 +22,21 @@ class AbstractDownloadStrategyTests < Homebrew::TestCase def test_expand_safe_system_args_with_explicit_quiet_flag @args << { quiet_flag: "--flag" } - ARGV.stubs(verbose?: false) expanded_args = @strategy.expand_safe_system_args(@args) - assert_equal %w[foo bar baz --flag], expanded_args + if ARGV.verbose? + assert_equal %w[foo bar baz], expanded_args + else + assert_equal %w[foo bar baz --flag], expanded_args + end end def test_expand_safe_system_args_with_implicit_quiet_flag - ARGV.stubs(verbose?: false) expanded_args = @strategy.expand_safe_system_args(@args) - assert_equal %w[foo bar -q baz], expanded_args + if ARGV.verbose? + assert_equal %w[foo bar baz], expanded_args + else + assert_equal %w[foo bar -q baz], expanded_args + end end def test_expand_safe_system_args_does_not_mutate_argument