@@ -38,16 +38,43 class MailHandlerControllerTest < ActionController::TestCase | |||
|
38 | 38 | Setting.mail_handler_api_enabled = 1 |
|
39 | 39 | Setting.mail_handler_api_key = 'secret' |
|
40 | 40 | |
|
41 | post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml')) | |
|
41 | assert_difference 'Issue.count' do | |
|
42 | post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml')) | |
|
43 | end | |
|
42 | 44 | assert_response 201 |
|
43 | 45 | end |
|
44 | 46 | |
|
45 |
def test_should_ |
|
|
47 | def test_should_respond_with_422_if_not_created | |
|
48 | Project.find('onlinestore').destroy | |
|
49 | ||
|
50 | Setting.mail_handler_api_enabled = 1 | |
|
51 | Setting.mail_handler_api_key = 'secret' | |
|
52 | ||
|
53 | assert_no_difference 'Issue.count' do | |
|
54 | post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml')) | |
|
55 | end | |
|
56 | assert_response 422 | |
|
57 | end | |
|
58 | ||
|
59 | def test_should_not_allow_with_api_disabled | |
|
46 | 60 | # Disable API |
|
47 | 61 | Setting.mail_handler_api_enabled = 0 |
|
48 | 62 | Setting.mail_handler_api_key = 'secret' |
|
49 | 63 | |
|
50 | post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml')) | |
|
64 | assert_no_difference 'Issue.count' do | |
|
65 | post :index, :key => 'secret', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml')) | |
|
66 | end | |
|
67 | assert_response 403 | |
|
68 | end | |
|
69 | ||
|
70 | def test_should_not_allow_with_wrong_key | |
|
71 | # Disable API | |
|
72 | Setting.mail_handler_api_enabled = 1 | |
|
73 | Setting.mail_handler_api_key = 'secret' | |
|
74 | ||
|
75 | assert_no_difference 'Issue.count' do | |
|
76 | post :index, :key => 'wrong', :email => IO.read(File.join(FIXTURES_PATH, 'ticket_on_given_project.eml')) | |
|
77 | end | |
|
51 | 78 | assert_response 403 |
|
52 | 79 | end |
|
53 | 80 | end |
General Comments 0
You need to be logged in to leave comments.
Login now