Revert [6086] and [6087] since they caused a major regression with functional tests in 1.2.2. References #7372.

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@6145 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

actionpack/CHANGELOG

@@ -25,8 +25,6 @@ superclass' view_paths.  [Rick]
 
 * Added FormBuilder#submit as a delegate for FormTagHelper#submit_tag [DHH]
 
-* TestSession supports indifferent access so session['foo'] == session[:foo] in your tests.  #7372 [julik, jean.helou]
-
 * Allow Routes to generate all urls for a set of options by specifying :generate_all => true. Allows caching to properly set or expire all paths for a resource. References #1739. [Nicholas Seckar]
 
 * Change the query parser to map empty GET params to "" rather than nil. Closes #5694. [Nicholas Seckar]

actionpack/lib/action_controller/test_process.rb

@@ -292,11 +292,11 @@ module ActionController #:nodoc:
     end
 
     def [](key)
-      data[key.to_s]
+      data[key]
     end
 
     def []=(key, value)
-      data[key.to_s] = value
+      data[key] = value
     end
 
     def update

actionpack/test/controller/test_test.rb

@@ -8,12 +8,6 @@ class TestTest < Test::Unit::TestCase
       render :text => 'ignore me'
     end
 
-    def set_session
-      session['string'] = 'A wonder'
-      session[:symbol] = 'it works'
-      render :text => 'Success'
-    end
-
     def render_raw_post
       raise Test::Unit::AssertionFailedError, "#raw_post is blank" if request.raw_post.blank?
       render :text => request.raw_post
@@ -117,14 +111,6 @@ HTML
     assert_equal '>value<', flash['test']
   end
 
-  def test_process_with_session
-    process :set_session
-    assert_equal 'A wonder', session['string'], "A value stored in the session should be available by string key"
-    assert_equal 'A wonder', session[:string], "Test session hash should allow indifferent access"
-    assert_equal 'it works', session['symbol'], "Test session hash should allow indifferent access"
-    assert_equal 'it works', session[:symbol], "Test session hash should allow indifferent access"
-  end
-
   def test_process_with_request_uri_with_no_params
     process :test_uri
     assert_equal "/test_test/test/test_uri", @response.body