From: Toshi MARUYAMA <marutosijp2@yahoo.co.jp>
Date: Tue, 19 May 2020 08:23:03 +0000 (+0000)
Subject: add test of upload filename with plus (#11568)
X-Git-Tag: 4.2.0~1003
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=dd35671202bead1510d3e84577ebd7861029f19e;p=redmine.git

add test of upload filename with plus (#11568)

git-svn-id: http://svn.redmine.org/redmine/trunk@19786 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/test/integration/attachments_test.rb b/test/integration/attachments_test.rb
index d123c9ef1..49b99421f 100644
--- a/test/integration/attachments_test.rb
+++ b/test/integration/attachments_test.rb
@@ -130,6 +130,30 @@ class AttachmentsTest < Redmine::IntegrationTest
     assert_equal 'File content'.length, attachment.filesize
   end
 
+  def test_upload_filename_with_plus
+    log_user('jsmith', 'jsmith')
+    filename = 'a+b.txt'
+    token = ajax_upload(filename, 'File content')
+    assert_difference 'Issue.count' do
+      post(
+        '/projects/ecookbook/issues',
+        :params => {
+          :issue => {:tracker_id => 1, :subject => 'Issue with upload'},
+          :attachments => {'p0' => {:filename => filename, :token => token}}
+        }
+      )
+      assert_response 302
+    end
+    issue = Issue.order('id DESC').first
+    assert_equal 'Issue with upload', issue.subject
+    assert_equal 1, issue.attachments.count
+
+    attachment = issue.attachments.first
+    assert_equal filename, attachment.filename
+    assert_equal '', attachment.description
+    assert_equal 'File content'.length, attachment.filesize
+  end
+
   def test_upload_as_js_and_destroy
     log_user('jsmith', 'jsmith')