Chris@909
|
1 # Redmine - project management software
|
Chris@1115
|
2 # Copyright (C) 2006-2012 Jean-Philippe Lang
|
Chris@909
|
3 #
|
Chris@909
|
4 # This program is free software; you can redistribute it and/or
|
Chris@909
|
5 # modify it under the terms of the GNU General Public License
|
Chris@909
|
6 # as published by the Free Software Foundation; either version 2
|
Chris@909
|
7 # of the License, or (at your option) any later version.
|
Chris@909
|
8 #
|
Chris@909
|
9 # This program is distributed in the hope that it will be useful,
|
Chris@909
|
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
|
Chris@909
|
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
Chris@909
|
12 # GNU General Public License for more details.
|
Chris@909
|
13 #
|
Chris@909
|
14 # You should have received a copy of the GNU General Public License
|
Chris@909
|
15 # along with this program; if not, write to the Free Software
|
Chris@909
|
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
Chris@909
|
17
|
Chris@909
|
18 require File.expand_path('../../../test_helper', __FILE__)
|
Chris@909
|
19
|
Chris@909
|
20 class ApiTest::AttachmentsTest < ActionController::IntegrationTest
|
Chris@909
|
21 fixtures :projects, :trackers, :issue_statuses, :issues,
|
Chris@909
|
22 :enumerations, :users, :issue_categories,
|
Chris@909
|
23 :projects_trackers,
|
Chris@909
|
24 :roles,
|
Chris@909
|
25 :member_roles,
|
Chris@909
|
26 :members,
|
Chris@909
|
27 :enabled_modules,
|
Chris@909
|
28 :workflows,
|
Chris@909
|
29 :attachments
|
Chris@909
|
30
|
Chris@909
|
31 def setup
|
Chris@909
|
32 Setting.rest_api_enabled = '1'
|
Chris@1115
|
33 set_fixtures_attachments_directory
|
Chris@909
|
34 end
|
Chris@909
|
35
|
Chris@1115
|
36 def teardown
|
Chris@1115
|
37 set_tmp_attachments_directory
|
Chris@1115
|
38 end
|
Chris@1115
|
39
|
Chris@1115
|
40 test "GET /attachments/:id.xml should return the attachment" do
|
Chris@1115
|
41 get '/attachments/7.xml', {}, credentials('jsmith')
|
Chris@1115
|
42 assert_response :success
|
Chris@1115
|
43 assert_equal 'application/xml', @response.content_type
|
Chris@1115
|
44 assert_tag :tag => 'attachment',
|
Chris@1115
|
45 :child => {
|
Chris@1115
|
46 :tag => 'id',
|
Chris@1115
|
47 :content => '7',
|
Chris@1115
|
48 :sibling => {
|
Chris@1115
|
49 :tag => 'filename',
|
Chris@1115
|
50 :content => 'archive.zip',
|
Chris@1115
|
51 :sibling => {
|
Chris@1115
|
52 :tag => 'content_url',
|
Chris@1115
|
53 :content => 'http://www.example.com/attachments/download/7/archive.zip'
|
Chris@909
|
54 }
|
Chris@1115
|
55 }
|
Chris@1115
|
56 }
|
Chris@1115
|
57 end
|
Chris@909
|
58
|
Chris@1115
|
59 test "GET /attachments/:id.xml should deny access without credentials" do
|
Chris@1115
|
60 get '/attachments/7.xml'
|
Chris@1115
|
61 assert_response 401
|
Chris@1115
|
62 set_tmp_attachments_directory
|
Chris@1115
|
63 end
|
Chris@1115
|
64
|
Chris@1115
|
65 test "GET /attachments/download/:id/:filename should return the attachment content" do
|
Chris@1115
|
66 get '/attachments/download/7/archive.zip', {}, credentials('jsmith')
|
Chris@1115
|
67 assert_response :success
|
Chris@1115
|
68 assert_equal 'application/octet-stream', @response.content_type
|
Chris@1115
|
69 set_tmp_attachments_directory
|
Chris@1115
|
70 end
|
Chris@1115
|
71
|
Chris@1115
|
72 test "GET /attachments/download/:id/:filename should deny access without credentials" do
|
Chris@1115
|
73 get '/attachments/download/7/archive.zip'
|
Chris@1115
|
74 assert_response 302
|
Chris@1115
|
75 set_tmp_attachments_directory
|
Chris@1115
|
76 end
|
Chris@1115
|
77
|
Chris@1115
|
78 test "POST /uploads.xml should return the token" do
|
Chris@1115
|
79 set_tmp_attachments_directory
|
Chris@1115
|
80 assert_difference 'Attachment.count' do
|
Chris@1115
|
81 post '/uploads.xml', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
|
Chris@1115
|
82 assert_response :created
|
Chris@1115
|
83 assert_equal 'application/xml', response.content_type
|
Chris@1115
|
84 end
|
Chris@1115
|
85
|
Chris@1115
|
86 xml = Hash.from_xml(response.body)
|
Chris@1115
|
87 assert_kind_of Hash, xml['upload']
|
Chris@1115
|
88 token = xml['upload']['token']
|
Chris@1115
|
89 assert_not_nil token
|
Chris@1115
|
90
|
Chris@1115
|
91 attachment = Attachment.first(:order => 'id DESC')
|
Chris@1115
|
92 assert_equal token, attachment.token
|
Chris@1115
|
93 assert_nil attachment.container
|
Chris@1115
|
94 assert_equal 2, attachment.author_id
|
Chris@1115
|
95 assert_equal 'File content'.size, attachment.filesize
|
Chris@1115
|
96 assert attachment.content_type.blank?
|
Chris@1115
|
97 assert attachment.filename.present?
|
Chris@1115
|
98 assert_match /\d+_[0-9a-z]+/, attachment.diskfile
|
Chris@1115
|
99 assert File.exist?(attachment.diskfile)
|
Chris@1115
|
100 assert_equal 'File content', File.read(attachment.diskfile)
|
Chris@1115
|
101 end
|
Chris@1115
|
102
|
Chris@1115
|
103 test "POST /uploads.json should return the token" do
|
Chris@1115
|
104 set_tmp_attachments_directory
|
Chris@1115
|
105 assert_difference 'Attachment.count' do
|
Chris@1115
|
106 post '/uploads.json', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
|
Chris@1115
|
107 assert_response :created
|
Chris@1115
|
108 assert_equal 'application/json', response.content_type
|
Chris@1115
|
109 end
|
Chris@1115
|
110
|
Chris@1115
|
111 json = ActiveSupport::JSON.decode(response.body)
|
Chris@1115
|
112 assert_kind_of Hash, json['upload']
|
Chris@1115
|
113 token = json['upload']['token']
|
Chris@1115
|
114 assert_not_nil token
|
Chris@1115
|
115
|
Chris@1115
|
116 attachment = Attachment.first(:order => 'id DESC')
|
Chris@1115
|
117 assert_equal token, attachment.token
|
Chris@1115
|
118 end
|
Chris@1115
|
119
|
Chris@1115
|
120 test "POST /uploads.xml should accept :filename param as the attachment filename" do
|
Chris@1115
|
121 set_tmp_attachments_directory
|
Chris@1115
|
122 assert_difference 'Attachment.count' do
|
Chris@1115
|
123 post '/uploads.xml?filename=test.txt', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
|
Chris@1115
|
124 assert_response :created
|
Chris@1115
|
125 end
|
Chris@1115
|
126
|
Chris@1115
|
127 attachment = Attachment.order('id DESC').first
|
Chris@1115
|
128 assert_equal 'test.txt', attachment.filename
|
Chris@1115
|
129 assert_match /_test\.txt$/, attachment.diskfile
|
Chris@1115
|
130 end
|
Chris@1115
|
131
|
Chris@1115
|
132 test "POST /uploads.xml should not accept other content types" do
|
Chris@1115
|
133 set_tmp_attachments_directory
|
Chris@1115
|
134 assert_no_difference 'Attachment.count' do
|
Chris@1115
|
135 post '/uploads.xml', 'PNG DATA', {"CONTENT_TYPE" => 'image/png'}.merge(credentials('jsmith'))
|
Chris@1115
|
136 assert_response 406
|
Chris@1115
|
137 end
|
Chris@1115
|
138 end
|
Chris@1115
|
139
|
Chris@1115
|
140 test "POST /uploads.xml should return errors if file is too big" do
|
Chris@1115
|
141 set_tmp_attachments_directory
|
Chris@1115
|
142 with_settings :attachment_max_size => 1 do
|
Chris@1115
|
143 assert_no_difference 'Attachment.count' do
|
Chris@1115
|
144 post '/uploads.xml', ('x' * 2048), {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
|
Chris@1115
|
145 assert_response 422
|
Chris@1115
|
146 assert_tag 'error', :content => /exceeds the maximum allowed file size/
|
Chris@909
|
147 end
|
Chris@909
|
148 end
|
Chris@909
|
149 end
|
Chris@909
|
150 end
|