No need to check if fragment responds to :each

Author: pixeltrix

test/abstract_unit.rb

@@ -6,15 +6,3 @@
 
 FIXTURE_LOAD_PATH = File.expand_path('../fixtures', __FILE__)
 
-module RackTestUtils
-  def body_to_string(body)
-    if body.respond_to?(:each)
-      str = ''
-      body.each {|s| str << s }
-      str
-    else
-      body
-    end
-  end
-  extend self
-end

test/caching_test.rb

@@ -249,8 +249,6 @@ def test_simple_action_not_cached
     assert_not_equal cached_time, @response.body
   end
 
-  include RackTestUtils
-
   def test_action_cache_with_layout
     draw do
       get '/action_caching_test/with_layout', to: 'action_caching_test#with_layout'
@@ -265,8 +263,7 @@ def test_action_cache_with_layout
     get :with_layout
     assert_response :success
     assert_not_equal cached_time, @response.body
-    body = body_to_string(read_fragment('hostname.com/action_caching_test/with_layout'))
-    assert_equal @response.body, body
+    assert_equal @response.body, read_fragment('hostname.com/action_caching_test/with_layout')
   end
 
   def test_action_cache_with_layout_and_layout_cache_false
@@ -283,8 +280,7 @@ def test_action_cache_with_layout_and_layout_cache_false
     get :layout_false
     assert_response :success
     assert_not_equal cached_time, @response.body
-    body = body_to_string(read_fragment('hostname.com/action_caching_test/layout_false'))
-    assert_equal cached_time, body
+    assert_equal cached_time, read_fragment('hostname.com/action_caching_test/layout_false')
   end
 
   def test_action_cache_with_layout_and_layout_cache_false_via_proc
@@ -301,8 +297,7 @@ def test_action_cache_with_layout_and_layout_cache_false_via_proc
     get :with_layout_proc_param, params: { layout: false }
     assert_response :success
     assert_not_equal cached_time, @response.body
-    body = body_to_string(read_fragment('hostname.com/action_caching_test/with_layout_proc_param'))
-    assert_equal cached_time, body
+    assert_equal cached_time, read_fragment('hostname.com/action_caching_test/with_layout_proc_param')
   end
 
   def test_action_cache_with_layout_and_layout_cache_true_via_proc
@@ -319,8 +314,7 @@ def test_action_cache_with_layout_and_layout_cache_true_via_proc
     get :with_layout_proc_param, params: { layout: true }
     assert_response :success
     assert_not_equal cached_time, @response.body
-    body = body_to_string(read_fragment('hostname.com/action_caching_test/with_layout_proc_param'))
-    assert_equal @response.body, body
+    assert_equal @response.body, read_fragment('hostname.com/action_caching_test/with_layout_proc_param')
   end
 
   def test_action_cache_conditional_options