aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2012-01-06 11:52:26 -0800
committerAaron Patterson <aaron.patterson@gmail.com>2012-01-06 11:52:26 -0800
commit9699eeb496d6faff6f7768cbfe0e281ce7ce089c (patch)
tree0534d233c34889b181a64f641a02ea521f1eb928
parent6548fc66178dd3d46e3a575e6937659c07293764 (diff)
downloadrails-9699eeb496d6faff6f7768cbfe0e281ce7ce089c.tar.gz
rails-9699eeb496d6faff6f7768cbfe0e281ce7ce089c.tar.bz2
rails-9699eeb496d6faff6f7768cbfe0e281ce7ce089c.zip
convert build_message to sprintf
-rw-r--r--actionpack/lib/action_controller/test_case.rb14
1 files changed, 5 insertions, 9 deletions
diff --git a/actionpack/lib/action_controller/test_case.rb b/actionpack/lib/action_controller/test_case.rb
index 45e4805ff4..05e9362463 100644
--- a/actionpack/lib/action_controller/test_case.rb
+++ b/actionpack/lib/action_controller/test_case.rb
@@ -73,9 +73,8 @@ module ActionController
when NilClass, String, Symbol
options = options.to_s if Symbol === options
rendered = @templates
- msg = build_message(message,
- "expecting <?> but rendering with <?>",
- options, rendered.keys.join(', '))
+ msg = message || sprintf("expecting <%s> but rendering with <%s>",
+ options, rendered.keys)
assert_block(msg) do
if options
rendered.any? { |t,num| t.match(options) }
@@ -85,8 +84,7 @@ module ActionController
end
when Hash
if expected_layout = options[:layout]
- msg = build_message(message,
- "expecting layout <?> but action rendered <?>",
+ msg = message || sprintf("expecting layout <%s> but action rendered <%s>",
expected_layout, @layouts.keys)
case expected_layout
@@ -107,13 +105,11 @@ module ActionController
end
elsif expected_count = options[:count]
actual_count = @partials[expected_partial]
- msg = build_message(message,
- "expecting ? to be rendered ? time(s) but rendered ? time(s)",
+ msg = message || sprintf("expecting %s to be rendered %s time(s) but rendered %s time(s)",
expected_partial, expected_count, actual_count)
assert(actual_count == expected_count.to_i, msg)
else
- msg = build_message(message,
- "expecting partial <?> but action rendered <?>",
+ msg = message || sprintf("expecting partial <%s> but action rendered <%s>",
options[:partial], @partials.keys)
assert_includes @partials, expected_partial, msg
end