jro_apps/redmine Files · test/integration/api_test/attachments_test.rb

Make sure that attachments are created in the same order they were selected (#12310)....

Make sure that attachments are created in the same order they were selected (#12310). git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10793 e93f8b46-1217-0410-a6f0-8f06a7374b81

Jean-Philippe Lang - - Load All Authors

File last commit:

r10467:251f263abdc5


                r10571:a17f4c8375d1

Download file

             attachments_test.rb
        
                    150 lines
            
             | 5.3 KiB
            
                | text/x-ruby
            
             |
                RubyLexer
            
             / test / integration / api_test / attachments_test.rb
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
      # Redmine - project management software

        Jean-Philippe Lang
    
Copyright update....

              r9453
            
      # Copyright (C) 2006-2012  Jean-Philippe Lang

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
      #

      # This program is free software; you can redistribute it and/or

      # modify it under the terms of the GNU General Public License

      # as published by the Free Software Foundation; either version 2

      # of the License, or (at your option) any later version.

        Toshi MARUYAMA
    
remove trailing white-spaces from test/integration/api_test/attachments_test.rb....

              r6546
            
      #

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
      # This program is distributed in the hope that it will be useful,

      # but WITHOUT ANY WARRANTY; without even the implied warranty of

      # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

      # GNU General Public License for more details.

        Toshi MARUYAMA
    
remove trailing white-spaces from test/integration/api_test/attachments_test.rb....

              r6546
            
      #

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
      # You should have received a copy of the GNU General Public License

      # along with this program; if not, write to the Free Software

      # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.

      require File.expand_path('../../../test_helper', __FILE__)

      class ApiTest::AttachmentsTest < ActionController::IntegrationTest

        Toshi MARUYAMA
    
Rails3: replace "all" fixtures at test/integration/api_test/attachments_test.rb...

              r7386
            
        fixtures :projects, :trackers, :issue_statuses, :issues,

                 :enumerations, :users, :issue_categories,

                 :projects_trackers,

                 :roles,

                 :member_roles,

                 :members,

                 :enabled_modules,

                 :workflows,

                 :attachments

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
        def setup

          Setting.rest_api_enabled = '1'

        Jean-Philippe Lang
    
Make sure that tests restore the attachments path to the tmp dir so that fixture files don't get deleted....

              r8128
            
          set_fixtures_attachments_directory

        end

        def teardown

          set_tmp_attachments_directory

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
        end

        Toshi MARUYAMA
    
remove trailing white-spaces from test/integration/api_test/attachments_test.rb....

              r6546
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        test "GET /attachments/:id.xml should return the attachment" do

          get '/attachments/7.xml', {}, credentials('jsmith')

          assert_response :success

          assert_equal 'application/xml', @response.content_type

          assert_tag :tag => 'attachment',

            :child => {

              :tag => 'id',

              :content => '7',

              :sibling => {

                :tag => 'filename',

                :content => 'archive.zip',

                :sibling => {

                  :tag => 'content_url',

                  :content => 'http://www.example.com/attachments/download/7/archive.zip'

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
                }

        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
              }

            }

        end

        Toshi MARUYAMA
    
remove trailing white-spaces from test/integration/api_test/attachments_test.rb....

              r6546
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        test "GET /attachments/:id.xml should deny access without credentials" do

          get '/attachments/7.xml'

          assert_response 401

          set_tmp_attachments_directory

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
        end

        Toshi MARUYAMA
    
remove trailing white-spaces from test/integration/api_test/attachments_test.rb....

              r6546
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        test "GET /attachments/download/:id/:filename should return the attachment content" do

          get '/attachments/download/7/archive.zip', {}, credentials('jsmith')

          assert_response :success

          assert_equal 'application/octet-stream', @response.content_type

          set_tmp_attachments_directory

        end

        Toshi MARUYAMA
    
remove trailing white-spaces from test/integration/api_test/attachments_test.rb....

              r6546
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        test "GET /attachments/download/:id/:filename should deny access without credentials" do

          get '/attachments/download/7/archive.zip'

          assert_response 302

          set_tmp_attachments_directory

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
        end

        Jean-Philippe Lang
    
Adds support for adding attachments to issues through the REST API (#8171)....

              r8808
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        test "POST /uploads.xml should return the token" do

          set_tmp_attachments_directory

          assert_difference 'Attachment.count' do

            post '/uploads.xml', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))

            assert_response :created

            assert_equal 'application/xml', response.content_type

          end

        Jean-Philippe Lang
    
Adds support for adding attachments to issues through the REST API (#8171)....

              r8808
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
          xml = Hash.from_xml(response.body)

          assert_kind_of Hash, xml['upload']

          token = xml['upload']['token']

          assert_not_nil token

        Jean-Philippe Lang
    
Adds support for adding attachments to issues through the REST API (#8171)....

              r8808
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
          attachment = Attachment.first(:order => 'id DESC')

          assert_equal token, attachment.token

          assert_nil attachment.container

          assert_equal 2, attachment.author_id

          assert_equal 'File content'.size, attachment.filesize

          assert attachment.content_type.blank?

          assert attachment.filename.present?

          assert_match /\d+_[0-9a-z]+/, attachment.diskfile

          assert File.exist?(attachment.diskfile)

          assert_equal 'File content', File.read(attachment.diskfile)

        end

        Jean-Philippe Lang
    
Adds support for adding attachments to issues through the REST API (#8171)....

              r8808
            
        Jean-Philippe Lang
    
Added a test for JSON upload....

              r10224
            
        test "POST /uploads.json should return the token" do

          set_tmp_attachments_directory

          assert_difference 'Attachment.count' do

            post '/uploads.json', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))

            assert_response :created

            assert_equal 'application/json', response.content_type

          end

          json = ActiveSupport::JSON.decode(response.body)

          assert_kind_of Hash, json['upload']

          token = json['upload']['token']

          assert_not_nil token

          attachment = Attachment.first(:order => 'id DESC')

          assert_equal token, attachment.token

        end

        Jean-Philippe Lang
    
Let the attachment filename be specified on upload (#12125)....

              r10467
            
        test "POST /uploads.xml should accept :filename param as the attachment filename" do

          set_tmp_attachments_directory

          assert_difference 'Attachment.count' do

            post '/uploads.xml?filename=test.txt', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))

            assert_response :created

          end

          attachment = Attachment.order('id DESC').first

          assert_equal 'test.txt', attachment.filename

          assert_match /_test\.txt$/, attachment.diskfile

        end

        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        test "POST /uploads.xml should not accept other content types" do

          set_tmp_attachments_directory

          assert_no_difference 'Attachment.count' do

            post '/uploads.xml', 'PNG DATA', {"CONTENT_TYPE" => 'image/png'}.merge(credentials('jsmith'))

            assert_response 406

        Jean-Philippe Lang
    
Adds support for adding attachments to issues through the REST API (#8171)....

              r8808
            
          end

        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        end

        Jean-Philippe Lang
    
Better message for file size validation error....

              r8817
            
        Jean-Philippe Lang
    
Removed test contexts....

              r10223
            
        test "POST /uploads.xml should return errors if file is too big" do

          set_tmp_attachments_directory

          with_settings :attachment_max_size => 1 do

            assert_no_difference 'Attachment.count' do

              post '/uploads.xml', ('x' * 2048), {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))

              assert_response 422

              assert_tag 'error', :content => /exceeds the maximum allowed file size/

        Jean-Philippe Lang
    
Better message for file size validation error....

              r8817
            
            end

          end

        Jean-Philippe Lang
    
Adds support for adding attachments to issues through the REST API (#8171)....

              r8808
            
        end

        Jean-Philippe Lang
    
REST API for reading attachments (#7671)....

              r6175
            
      end

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	# Redmine - project management software
Jean-Philippe Lang Copyright update....	r9453	# Copyright (C) 2006-2012 Jean-Philippe Lang
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	#
		# This program is free software; you can redistribute it and/or
		# modify it under the terms of the GNU General Public License
		# as published by the Free Software Foundation; either version 2
		# of the License, or (at your option) any later version.
Toshi MARUYAMA remove trailing white-spaces from test/integration/api_test/attachments_test.rb....	r6546	#
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	# This program is distributed in the hope that it will be useful,
		# but WITHOUT ANY WARRANTY; without even the implied warranty of
		# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
		# GNU General Public License for more details.
Toshi MARUYAMA remove trailing white-spaces from test/integration/api_test/attachments_test.rb....	r6546	#
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	# You should have received a copy of the GNU General Public License
		# along with this program; if not, write to the Free Software
		# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.

		require File.expand_path('../../../test_helper', __FILE__)

		class ApiTest::AttachmentsTest < ActionController::IntegrationTest
Toshi MARUYAMA Rails3: replace "all" fixtures at test/integration/api_test/attachments_test.rb...	r7386	fixtures :projects, :trackers, :issue_statuses, :issues,
		:enumerations, :users, :issue_categories,
		:projects_trackers,
		:roles,
		:member_roles,
		:members,
		:enabled_modules,
		:workflows,
		:attachments
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175
		def setup
		Setting.rest_api_enabled = '1'
Jean-Philippe Lang Make sure that tests restore the attachments path to the tmp dir so that fixture files don't get deleted....	r8128	set_fixtures_attachments_directory
		end

		def teardown
		set_tmp_attachments_directory
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	end
Toshi MARUYAMA remove trailing white-spaces from test/integration/api_test/attachments_test.rb....	r6546
Jean-Philippe Lang Removed test contexts....	r10223	test "GET /attachments/:id.xml should return the attachment" do
		get '/attachments/7.xml', {}, credentials('jsmith')
		assert_response :success
		assert_equal 'application/xml', @response.content_type
		assert_tag :tag => 'attachment',
		:child => {
		:tag => 'id',
		:content => '7',
		:sibling => {
		:tag => 'filename',
		:content => 'archive.zip',
		:sibling => {
		:tag => 'content_url',
		:content => 'http://www.example.com/attachments/download/7/archive.zip'
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	}
Jean-Philippe Lang Removed test contexts....	r10223	}
		}
		end
Toshi MARUYAMA remove trailing white-spaces from test/integration/api_test/attachments_test.rb....	r6546
Jean-Philippe Lang Removed test contexts....	r10223	test "GET /attachments/:id.xml should deny access without credentials" do
		get '/attachments/7.xml'
		assert_response 401
		set_tmp_attachments_directory
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	end
Toshi MARUYAMA remove trailing white-spaces from test/integration/api_test/attachments_test.rb....	r6546
Jean-Philippe Lang Removed test contexts....	r10223	test "GET /attachments/download/:id/:filename should return the attachment content" do
		get '/attachments/download/7/archive.zip', {}, credentials('jsmith')
		assert_response :success
		assert_equal 'application/octet-stream', @response.content_type
		set_tmp_attachments_directory
		end
Toshi MARUYAMA remove trailing white-spaces from test/integration/api_test/attachments_test.rb....	r6546
Jean-Philippe Lang Removed test contexts....	r10223	test "GET /attachments/download/:id/:filename should deny access without credentials" do
		get '/attachments/download/7/archive.zip'
		assert_response 302
		set_tmp_attachments_directory
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	end
Jean-Philippe Lang Adds support for adding attachments to issues through the REST API (#8171)....	r8808
Jean-Philippe Lang Removed test contexts....	r10223	test "POST /uploads.xml should return the token" do
		set_tmp_attachments_directory
		assert_difference 'Attachment.count' do
		post '/uploads.xml', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
		assert_response :created
		assert_equal 'application/xml', response.content_type
		end
Jean-Philippe Lang Adds support for adding attachments to issues through the REST API (#8171)....	r8808
Jean-Philippe Lang Removed test contexts....	r10223	xml = Hash.from_xml(response.body)
		assert_kind_of Hash, xml['upload']
		token = xml['upload']['token']
		assert_not_nil token
Jean-Philippe Lang Adds support for adding attachments to issues through the REST API (#8171)....	r8808
Jean-Philippe Lang Removed test contexts....	r10223	attachment = Attachment.first(:order => 'id DESC')
		assert_equal token, attachment.token
		assert_nil attachment.container
		assert_equal 2, attachment.author_id
		assert_equal 'File content'.size, attachment.filesize
		assert attachment.content_type.blank?
		assert attachment.filename.present?
		assert_match /\d+_[0-9a-z]+/, attachment.diskfile
		assert File.exist?(attachment.diskfile)
		assert_equal 'File content', File.read(attachment.diskfile)
		end
Jean-Philippe Lang Adds support for adding attachments to issues through the REST API (#8171)....	r8808
Jean-Philippe Lang Added a test for JSON upload....	r10224	test "POST /uploads.json should return the token" do
		set_tmp_attachments_directory
		assert_difference 'Attachment.count' do
		post '/uploads.json', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
		assert_response :created
		assert_equal 'application/json', response.content_type
		end

		json = ActiveSupport::JSON.decode(response.body)
		assert_kind_of Hash, json['upload']
		token = json['upload']['token']
		assert_not_nil token

		attachment = Attachment.first(:order => 'id DESC')
		assert_equal token, attachment.token
		end

Jean-Philippe Lang Let the attachment filename be specified on upload (#12125)....	r10467	test "POST /uploads.xml should accept :filename param as the attachment filename" do
		set_tmp_attachments_directory
		assert_difference 'Attachment.count' do
		post '/uploads.xml?filename=test.txt', 'File content', {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
		assert_response :created
		end

		attachment = Attachment.order('id DESC').first
		assert_equal 'test.txt', attachment.filename
		assert_match /_test\.txt$/, attachment.diskfile
		end

Jean-Philippe Lang Removed test contexts....	r10223	test "POST /uploads.xml should not accept other content types" do
		set_tmp_attachments_directory
		assert_no_difference 'Attachment.count' do
		post '/uploads.xml', 'PNG DATA', {"CONTENT_TYPE" => 'image/png'}.merge(credentials('jsmith'))
		assert_response 406
Jean-Philippe Lang Adds support for adding attachments to issues through the REST API (#8171)....	r8808	end
Jean-Philippe Lang Removed test contexts....	r10223	end
Jean-Philippe Lang Better message for file size validation error....	r8817
Jean-Philippe Lang Removed test contexts....	r10223	test "POST /uploads.xml should return errors if file is too big" do
		set_tmp_attachments_directory
		with_settings :attachment_max_size => 1 do
		assert_no_difference 'Attachment.count' do
		post '/uploads.xml', ('x' * 2048), {"CONTENT_TYPE" => 'application/octet-stream'}.merge(credentials('jsmith'))
		assert_response 422
		assert_tag 'error', :content => /exceeds the maximum allowed file size/
Jean-Philippe Lang Better message for file size validation error....	r8817	end
		end
Jean-Philippe Lang Adds support for adding attachments to issues through the REST API (#8171)....	r8808	end
Jean-Philippe Lang REST API for reading attachments (#7671)....	r6175	end