##// END OF EJS Templates
Fixed test/functional/mail_handler_controller_test.rb breaking when run alone (#12285)...
Jean-Baptiste Barth -
r10561:9280182bdef8
parent child
Show More
@@ -1,80 +1,81
1 # Redmine - project management software
1 # Redmine - project management software
2 # Copyright (C) 2006-2012 Jean-Philippe Lang
2 # Copyright (C) 2006-2012 Jean-Philippe Lang
3 #
3 #
4 # This program is free software; you can redistribute it and/or
4 # This program is free software; you can redistribute it and/or
5 # modify it under the terms of the GNU General Public License
5 # modify it under the terms of the GNU General Public License
6 # as published by the Free Software Foundation; either version 2
6 # as published by the Free Software Foundation; either version 2
7 # of the License, or (at your option) any later version.
7 # of the License, or (at your option) any later version.
8 #
8 #
9 # This program is distributed in the hope that it will be useful,
9 # This program is distributed in the hope that it will be useful,
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 # GNU General Public License for more details.
12 # GNU General Public License for more details.
13 #
13 #
14 # You should have received a copy of the GNU General Public License
14 # You should have received a copy of the GNU General Public License
15 # along with this program; if not, write to the Free Software
15 # along with this program; if not, write to the Free Software
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17
17
18 require File.expand_path('../../test_helper', __FILE__)
18 require File.expand_path('../../test_helper', __FILE__)
19 require 'mail_handler_controller'
19 require 'mail_handler_controller'
20
20
21 # Re-raise errors caught by the controller.
21 # Re-raise errors caught by the controller.
22 class MailHandlerController; def rescue_action(e) raise e end; end
22 class MailHandlerController; def rescue_action(e) raise e end; end
23
23
24 class MailHandlerControllerTest < ActionController::TestCase
24 class MailHandlerControllerTest < ActionController::TestCase
25 fixtures :users, :projects, :enabled_modules, :roles, :members, :member_roles, :issues, :issue_statuses, :trackers, :enumerations
25 fixtures :users, :projects, :enabled_modules, :roles, :members, :member_roles, :issues, :issue_statuses,
26 :trackers, :projects_trackers, :enumerations
26
27
27 FIXTURES_PATH = File.dirname(__FILE__) + '/../fixtures/mail_handler'
28 FIXTURES_PATH = File.dirname(__FILE__) + '/../fixtures/mail_handler'
28
29
29 def setup
30 def setup
30 @controller = MailHandlerController.new
31 @controller = MailHandlerController.new
31 @request = ActionController::TestRequest.new
32 @request = ActionController::TestRequest.new
32 @response = ActionController::TestResponse.new
33 @response = ActionController::TestResponse.new
33 User.current = nil
34 User.current = nil
34 end
35 end
35
36
36 def test_should_create_issue
37 def test_should_create_issue
37 # Enable API and set a key
38 # Enable API and set a key
38 Setting.mail_handler_api_enabled = 1
39 Setting.mail_handler_api_enabled = 1
39 Setting.mail_handler_api_key = 'secret'
40 Setting.mail_handler_api_key = 'secret'
40
41
41 assert_difference 'Issue.count' do
42 assert_difference 'Issue.count' do
42 post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
43 post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
43 end
44 end
44 assert_response 201
45 assert_response 201
45 end
46 end
46
47
47 def test_should_respond_with_422_if_not_created
48 def test_should_respond_with_422_if_not_created
48 Project.find('onlinestore').destroy
49 Project.find('onlinestore').destroy
49
50
50 Setting.mail_handler_api_enabled = 1
51 Setting.mail_handler_api_enabled = 1
51 Setting.mail_handler_api_key = 'secret'
52 Setting.mail_handler_api_key = 'secret'
52
53
53 assert_no_difference 'Issue.count' do
54 assert_no_difference 'Issue.count' do
54 post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
55 post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
55 end
56 end
56 assert_response 422
57 assert_response 422
57 end
58 end
58
59
59 def test_should_not_allow_with_api_disabled
60 def test_should_not_allow_with_api_disabled
60 # Disable API
61 # Disable API
61 Setting.mail_handler_api_enabled = 0
62 Setting.mail_handler_api_enabled = 0
62 Setting.mail_handler_api_key = 'secret'
63 Setting.mail_handler_api_key = 'secret'
63
64
64 assert_no_difference 'Issue.count' do
65 assert_no_difference 'Issue.count' do
65 post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
66 post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
66 end
67 end
67 assert_response 403
68 assert_response 403
68 end
69 end
69
70
70 def test_should_not_allow_with_wrong_key
71 def test_should_not_allow_with_wrong_key
71 # Disable API
72 # Disable API
72 Setting.mail_handler_api_enabled = 1
73 Setting.mail_handler_api_enabled = 1
73 Setting.mail_handler_api_key = 'secret'
74 Setting.mail_handler_api_key = 'secret'
74
75
75 assert_no_difference 'Issue.count' do
76 assert_no_difference 'Issue.count' do
76 post :index, :key => 'wrong', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
77 post :index, :key => 'wrong', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml'))
77 end
78 end
78 assert_response 403
79 assert_response 403
79 end
80 end
80 end
81 end
General Comments 0
You need to be logged in to leave comments. Login now