annotate .svn/pristine/02/02ceadd1b8d78a178efab59eb94d40a87ff683f1.svn-base @ 1628:9c5f8e24dadc live tip

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