Ticket #8493: render_component_shouldnt_set_etag.diff

lib/action_controller/base.rb

@@ -496 +496 @@
-        assign_default_content_type_and_charset
-
-        response.request = request
-
+ unless component_request?
-        response
-      ensure
-        process_cleanup

test/controller/components_test.rb

@@ -96 +96 @@
-    assert_equal "Ring, ring: Lady of the House, speaking", @response.body
-  end
-
+  def test_etag_is_set_for_parent_template_when_calling_from_template
+    get :calling_from_template
+    expected_etag = etag_for("Ring, ring: Lady of the House, speaking")
+    assert_equal expected_etag, @response.headers['ETag']
+  end
+
-  def test_internal_calling
-    get :internal_caller
-    assert_equal "Are you there? Yes, ma'am", @response.body
@@ -109 +115 @@
-    get :use_flash
-    assert_equal 'no flash', @response.body
-  end
-
+
-  def test_component_redirect_redirects
-    get :calling_redirected
-
@@ -126 +132 @@
-
-    assert_equal "Lady of the House, speaking", @response.body
-  end
+  
+  protected
+  
+  def etag_for(text)
+    %("#{Digest::MD5.hexdigest(text)}")
+  end
+  
-end