diff options
| -rw-r--r-- | test/integration/api_test/files_test.rb | 45 |
1 files changed, 27 insertions, 18 deletions
diff --git a/test/integration/api_test/files_test.rb b/test/integration/api_test/files_test.rb index 260541094..378dd7bfa 100644 --- a/test/integration/api_test/files_test.rb +++ b/test/integration/api_test/files_test.rb @@ -37,9 +37,10 @@ class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base test "POST /projects/:project_id/files.json should create a file" do set_tmp_attachments_directory - post '/uploads.xml', + post( + '/uploads.xml', :params => 'File content', - :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))) token = Attachment.last.token payload = <<~JSON { "file": { @@ -47,9 +48,10 @@ class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base } } JSON - post '/projects/1/files.json', + post( + '/projects/1/files.json', :params => payload, - :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith'))) assert_response :success assert_equal 1, Attachment.last.container_id assert_equal "Project", Attachment.last.container_type @@ -57,18 +59,20 @@ class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base test "POST /projects/:project_id/files.xml should create a file" do set_tmp_attachments_directory - post '/uploads.xml', + post( + '/uploads.xml', :params => 'File content', - :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))) token = Attachment.last.token payload = <<~XML <file> <token>#{token}</token> </file> XML - post '/projects/1/files.xml', + post( + '/projects/1/files.xml', :params => payload, - :headers => {"CONTENT_TYPE" => 'application/xml'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/xml'}.merge(credentials('jsmith'))) assert_response :success assert_equal 1, Attachment.last.container_id assert_equal "Project", Attachment.last.container_type @@ -81,17 +85,19 @@ class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base } } JSON - post '/projects/1/files.json', + post( + '/projects/1/files.json', :params => payload, - :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith'))) assert_response :bad_request end test "POST /projects/:project_id/files.json should accept :filename, :description, :content_type as optional parameters" do set_tmp_attachments_directory - post '/uploads.xml', + post( + '/uploads.xml', :params => 'File content', - :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))) token = Attachment.last.token payload = <<~JSON { "file": { @@ -102,9 +108,10 @@ class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base } } JSON - post '/projects/1/files.json', + post( + '/projects/1/files.json', :params => payload, - :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith'))) assert_response :success assert_equal "New filename", Attachment.last.filename assert_equal "New description", Attachment.last.description @@ -113,9 +120,10 @@ class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base test "POST /projects/:project_id/files.json should accept :version_id to attach the files to a version" do set_tmp_attachments_directory - post '/uploads.xml', + post( + '/uploads.xml', :params => 'File content', - :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))) token = Attachment.last.token payload = <<~JSON { "file": { @@ -126,9 +134,10 @@ class Redmine::ApiTest::FilesTest < Redmine::ApiTest::Base } } JSON - post '/projects/1/files.json', + post( + '/projects/1/files.json', :params => payload, - :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith')) + :headers => {"CONTENT_TYPE" => 'application/json'}.merge(credentials('jsmith'))) assert_equal 3, Attachment.last.container_id assert_equal "Version", Attachment.last.container_type end |