From fff392abc59c15427bec9105eb5f0f0bbda6cff9 Mon Sep 17 00:00:00 2001 From: Raf Szalanski Date: Thu, 2 Jul 2015 22:31:40 +0000 Subject: [PATCH] Fix creation of tempfiles in Rack::Test::UploadedFile Tempfile's initializer is not very intuitive. If you want your tempfile to have an extension you have to pass an array with two elements: - basename - extension (with a dot at the beginning) This change makes it behave more like original Rack::Multipart::UploadedFile. --- lib/rack/test/uploaded_file.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/rack/test/uploaded_file.rb b/lib/rack/test/uploaded_file.rb index d0d3e2a..3339412 100644 --- a/lib/rack/test/uploaded_file.rb +++ b/lib/rack/test/uploaded_file.rb @@ -26,7 +26,7 @@ def initialize(path, content_type = "text/plain", binary = false) @content_type = content_type @original_filename = ::File.basename(path) - @tempfile = Tempfile.new(@original_filename) + @tempfile = Tempfile.new([@original_filename, ::File.extname(path)]) @tempfile.set_encoding(Encoding::BINARY) if @tempfile.respond_to?(:set_encoding) @tempfile.binmode if binary