aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
authorMike McQuaid2014-10-21 15:34:20 +0100
committerMike McQuaid2014-10-21 15:34:20 +0100
commit5c88e8d654f2ab8c5c3158bf28f13f08cfe6af50 (patch)
tree4f833e8cd68183b14bd9f4b4ec6dd4da0d210f01 /Library
parent324ea3e4814ce02c76c6e87064084e03141cfb17 (diff)
downloadbrew-5c88e8d654f2ab8c5c3158bf28f13f08cfe6af50.tar.bz2
brew-test-bot: be more resilient to bad arguments.
Diffstat (limited to 'Library')
-rw-r--r--Library/Homebrew/cmd/test-bot.rb12
1 files changed, 9 insertions, 3 deletions
diff --git a/Library/Homebrew/cmd/test-bot.rb b/Library/Homebrew/cmd/test-bot.rb
index 740a112ff..bb7aa7f73 100644
--- a/Library/Homebrew/cmd/test-bot.rb
+++ b/Library/Homebrew/cmd/test-bot.rb
@@ -175,7 +175,7 @@ module Homebrew
elsif formula
@formulae = [argument]
else
- odie "#{argument} is not a pull request URL, commit URL or formula name."
+ raise ArgumentError.new("#{argument} is not a pull request URL, commit URL or formula name.")
end
@category = __method__
@@ -597,8 +597,14 @@ module Homebrew
tests << test
else
ARGV.named.each do |argument|
- test = Test.new(argument, tap)
- test_error = !test.run
+ test_error = false
+ begin
+ test = Test.new(argument, tap)
+ test_error = !test.run
+ rescue ArgumentError => e
+ test_error = true
+ ofail e.message
+ end
any_errors ||= test_error
tests << test
end