test_process.rb in actionpack-2.1.1

- old
+ new

@@ -169,11 +169,11 @@
       headers['Status'].to_s.split(' ',2)[1]
     end
 
     # Was the response successful?
     def success?
-      response_code == 200
+      (200..299).include?(response_code)
     end
 
     # Was the URL not found?
     def missing?
       response_code == 404
@@ -331,11 +331,11 @@
   class TestUploadedFile
     # The filename, *not* including the path, of the "uploaded" file
     attr_reader :original_filename
 
     # The content type of the "uploaded" file
-    attr_reader :content_type
+    attr_accessor :content_type
 
     def initialize(path, content_type = Mime::TEXT, binary = false)
       raise "#{path} file does not exist" unless File.exist?(path)
       @content_type = content_type
       @original_filename = path.sub(/^.*#{File::SEPARATOR}([^#{File::SEPARATOR}]+)$/) { $1 }
@@ -410,9 +410,11 @@
         raise "Can't follow redirects outside of current controller (from #{@controller.controller_name} to #{redirected_controller})"
       end
 
       get(@response.redirected_to.delete(:action), @response.redirected_to.stringify_keys)
     end
+
+    deprecate :follow_redirect => "If you wish to follow redirects, you should use integration tests"
 
     def assigns(key = nil) 
       if key.nil? 
         @response.template.assigns 
       else 