@@ -102,7 +102,7 @@ module HTTParty # format :json # end def format(f) - raise UnsupportedFormat, "Must be one of: #{AllowedFormats.values.join(', ')}" unless AllowedFormats.value?(f) + raise UnsupportedFormat, "Must be one of: #{AllowedFormats.values.uniq.join(', ')}" unless AllowedFormats.value?(f) default_options[:format] = f end lib/httparty.rb @@ -159,6 +159,13 @@ describe HTTParty do @klass.format :foobar end.should raise_error(HTTParty::UnsupportedFormat) end + + it 'should only print each format once with an exception' do + lambda do + @klass.format :foobar + end.should raise_error(HTTParty::UnsupportedFormat, "Must be one of: json, xml, html, yaml, plain") + end + end describe "with explicit override of automatic redirect handling" do spec/httparty_spec.rb acd306859d31d97d469d41500a2526c8f9a6e39a jgeiger jgeiger@gmail.com http://github.com/alexvollmer/httparty/commit/0b067f7a2ccfc0832021d5c09e558f4bac6c84ca 0b067f7a2ccfc0832021d5c09e558f4bac6c84ca 2009-03-30T16:57:46-07:00 2009-03-05T13:06:02-08:00 only show the acceptable format one time per format on exception Signed-off-by: John Nunemaker <nunemaker@gmail.com> 24ebc7cfffb1df4ed7f5a38ec7f5d7ae950e904d John Nunemaker nunemaker@gmail.com