multipart-post 1.1.1 → 1.1.2

data/lib/parts.rb

@@ -40,8 +40,9 @@ module Parts
     def initialize(boundary, name, io)
       file_length = io.respond_to?(:length) ?  io.length : File.size(io.local_path)
       @head = build_head(boundary, name, io.original_filename, io.content_type, file_length)
-      @length = @head.length + file_length
-      @io = CompositeReadIO.new(StringIO.new(@head), io, StringIO.new("\r\n"))
+      @foot = "\r\n"
+      @length = @head.length + file_length + @foot.length
+      @io = CompositeReadIO.new(StringIO.new(@head), io, StringIO.new(@foot))
     end
 
     def build_head(boundary, name, filename, type, content_len)
@@ -59,7 +60,7 @@ module Parts
   class EpiloguePart
     include Part
     def initialize(boundary)
-      @part = "--#{boundary}--\r\n"
+      @part = "--#{boundary}--\r\n\r\n"
       @io = StringIO.new(@part)
     end
   end

data/test/test_parts.rb

@@ -0,0 +1,23 @@
+require 'test/unit'
+
+require 'parts'
+require 'stringio'
+require 'composite_io'
+
+class FilePartTest < Test::Unit::TestCase
+  TEMP_FILE = "temp.txt"
+
+  def setup
+    File.open(TEMP_FILE, "w") {|f| f << "1234567890"}
+    io =  UploadIO.new(TEMP_FILE, "text/plain")
+    @part = Parts::FilePart.new("boundary", "afile", io)
+  end
+
+  def teardown
+    File.delete(TEMP_FILE) rescue nil
+  end
+
+  def test_correct_length
+    assert_equal @part.length, @part.to_io.read.length
+  end
+end

metadata

@@ -2,7 +2,7 @@
 name: multipart-post
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors: 
 - Nick Sieger
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-13 00:00:00 Z
+date: 2011-05-24 00:00:00 Z
 dependencies: []
 
 description: "Use with Net::HTTP to do multipart form posts.  IO values that have #content_type, #original_filename, and #local_path will be posted as a binary file."
@@ -32,6 +32,7 @@ files:
 - README.txt
 - test/test_composite_io.rb
 - test/net/http/post/test_multipart.rb
+- test/test_parts.rb
 homepage: http://github.com/nicksieger/multipart-post
 licenses: []
 
@@ -46,7 +47,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -4205397351257484112
+      hash: 2078343466590708221
       segments: 
       - 0
       version: "0"
@@ -66,3 +67,4 @@ summary: Creates a multipart form post accessory for Net::HTTP.
 test_files: 
 - test/net/http/post/test_multipart.rb
 - test/test_composite_io.rb
+- test/test_parts.rb