size is not part of the input SPEC

If size is added as a formal requirement, revert this commit. Till then Lint should bomb if middleware tries to access env['rack.input'].size. Discussion about adding #size to rack.input http://groups.google.com/group/rack-devel/browse_thread/thread/9c06163a4b13ccad
jeromeetienne · Apr 7, 2010 · 11c9609 · 11c9609
1 parent 98a61aa
commit 11c9609
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 15 deletions.
diff --git a/lib/rack/lint.rb b/lib/rack/lint.rb
@@ -289,10 +289,6 @@ def initialize(input)
         @input = input
       end
 
-      def size
-        @input.size
-      end
-
       ## * +gets+ must be called without arguments and return a string,
       ##   or +nil+ on EOF.
       def gets(*args)

diff --git a/test/spec_rack_lint.rb b/test/spec_rack_lint.rb
@@ -506,17 +506,6 @@ def rewind
 end
 
 context "Rack::Lint::InputWrapper" do
-  specify "delegates :size to underlying IO object" do
-    class IOMock
-      def size
-        101
-      end
-    end
-
-    wrapper = Rack::Lint::InputWrapper.new(IOMock.new)
-    wrapper.size.should == 101
-  end
-
   specify "delegates :rewind to underlying IO object" do
     io = StringIO.new("123")
     wrapper = Rack::Lint::InputWrapper.new(io)