mail_handler_test.rb
607 lines
| 23.3 KiB
| text/x-ruby
|
RubyLexer
Jean-Philippe Lang
|
r3232 | # encoding: utf-8 | ||
| # | ||||
|
Jean-Philippe Lang
|
r2689 | # Redmine - project management software | ||
Toshi MARUYAMA
|
r5689 | # Copyright (C) 2006-2011 Jean-Philippe Lang | ||
|
Jean-Philippe Lang
|
r521 | # | ||
| # 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
|
r5689 | # | ||
|
Jean-Philippe Lang
|
r521 | # 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
|
r5689 | # | ||
|
Jean-Philippe Lang
|
r521 | # 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. | ||||
Jean-Baptiste Barth
|
r4395 | require File.expand_path('../../test_helper', __FILE__) | ||
|
Jean-Philippe Lang
|
r521 | |||
Eric Davis
|
r2773 | class MailHandlerTest < ActiveSupport::TestCase | ||
|
Toshi MARUYAMA
|
r8618 | fixtures :users, :projects, :enabled_modules, :roles, | ||
| :members, :member_roles, :users, | ||||
| :issues, :issue_statuses, | ||||
| :workflows, :trackers, :projects_trackers, | ||||
| :versions, :enumerations, :issue_categories, | ||||
| :custom_fields, :custom_fields_trackers, :custom_fields_projects, | ||||
| :boards, :messages | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r1554 | FIXTURES_PATH = File.dirname(__FILE__) + '/../fixtures/mail_handler' | ||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r521 | def setup | ||
|
Jean-Philippe Lang
|
r1554 | ActionMailer::Base.deliveries.clear | ||
|
Eric Davis
|
r4112 | Setting.notified_events = Redmine::Notifiable.all.collect(&:name) | ||
|
Jean-Philippe Lang
|
r521 | end | ||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r1554 | def test_add_issue | ||
|
Jean-Philippe Lang
|
r3001 | ActionMailer::Base.deliveries.clear | ||
|
Jean-Philippe Lang
|
r1554 | # This email contains: 'Project: onlinestore' | ||
| issue = submit_email('ticket_on_given_project.eml') | ||||
| assert issue.is_a?(Issue) | ||||
| assert !issue.new_record? | ||||
| issue.reload | ||||
|
Jean-Philippe Lang
|
r4461 | assert_equal Project.find(2), issue.project | ||
| assert_equal issue.project.trackers.first, issue.tracker | ||||
|
Jean-Philippe Lang
|
r1554 | assert_equal 'New ticket on a given project', issue.subject | ||
| assert_equal User.find_by_login('jsmith'), issue.author | ||||
|
Jean-Philippe Lang
|
r2365 | assert_equal IssueStatus.find_by_name('Resolved'), issue.status | ||
|
Jean-Philippe Lang
|
r1554 | assert issue.description.include?('Lorem ipsum dolor sit amet, consectetuer adipiscing elit.') | ||
|
Eric Davis
|
r3649 | assert_equal '2010-01-01', issue.start_date.to_s | ||
| assert_equal '2010-12-31', issue.due_date.to_s | ||||
|
Eric Davis
|
r3650 | assert_equal User.find_by_login('jsmith'), issue.assigned_to | ||
|
Jean-Philippe Lang
|
r6079 | assert_equal Version.find_by_name('Alpha'), issue.fixed_version | ||
|
Jean-Philippe Lang
|
r4280 | assert_equal 2.5, issue.estimated_hours | ||
| assert_equal 30, issue.done_ratio | ||||
|
Jean-Philippe Lang
|
r4302 | assert_equal [issue.id, 1, 2], [issue.root_id, issue.lft, issue.rgt] | ||
|
Jean-Philippe Lang
|
r2365 | # keywords should be removed from the email body | ||
| assert !issue.description.match(/^Project:/i) | ||||
| assert !issue.description.match(/^Status:/i) | ||||
|
Jean-Philippe Lang
|
r4985 | assert !issue.description.match(/^Start Date:/i) | ||
|
Jean-Philippe Lang
|
r3001 | # Email notification should be sent | ||
| mail = ActionMailer::Base.deliveries.last | ||||
| assert_not_nil mail | ||||
| assert mail.subject.include?('New ticket on a given project') | ||||
|
Jean-Philippe Lang
|
r1554 | end | ||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r4461 | def test_add_issue_with_default_tracker | ||
| # This email contains: 'Project: onlinestore' | ||||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_on_given_project.eml', | ||||
| :issue => {:tracker => 'Support request'} | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r4461 | assert issue.is_a?(Issue) | ||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'Support request', issue.tracker.name | ||||
| end | ||||
|
Eric Davis
|
r1750 | |||
| def test_add_issue_with_status | ||||
| # This email contains: 'Project: onlinestore' and 'Status: Resolved' | ||||
| issue = submit_email('ticket_on_given_project.eml') | ||||
| assert issue.is_a?(Issue) | ||||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal Project.find(2), issue.project | ||||
| assert_equal IssueStatus.find_by_name("Resolved"), issue.status | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r1629 | def test_add_issue_with_attributes_override | ||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_with_attributes.eml', | ||||
| :allow_override => 'tracker,category,priority' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r1629 | assert issue.is_a?(Issue) | ||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'New ticket on a given project', issue.subject | ||||
| assert_equal User.find_by_login('jsmith'), issue.author | ||||
| assert_equal Project.find(2), issue.project | ||||
| assert_equal 'Feature request', issue.tracker.to_s | ||||
| assert_equal 'Stock management', issue.category.to_s | ||||
| assert_equal 'Urgent', issue.priority.to_s | ||||
| assert issue.description.include?('Lorem ipsum dolor sit amet, consectetuer adipiscing elit.') | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6807 | |||
|
Jean-Philippe Lang
|
r6186 | def test_add_issue_with_group_assignment | ||
| with_settings :issue_group_assignment => '1' do | ||||
| issue = submit_email('ticket_on_given_project.eml') do |email| | ||||
| email.gsub!('Assigned to: John Smith', 'Assigned to: B Team') | ||||
| end | ||||
| assert issue.is_a?(Issue) | ||||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal Group.find(11), issue.assigned_to | ||||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r1629 | def test_add_issue_with_partial_attributes_override | ||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_with_attributes.eml', | ||||
| :issue => {:priority => 'High'}, | ||||
| :allow_override => ['tracker'] | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r1629 | assert issue.is_a?(Issue) | ||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'New ticket on a given project', issue.subject | ||||
| assert_equal User.find_by_login('jsmith'), issue.author | ||||
| assert_equal Project.find(2), issue.project | ||||
| assert_equal 'Feature request', issue.tracker.to_s | ||||
| assert_nil issue.category | ||||
| assert_equal 'High', issue.priority.to_s | ||||
| assert issue.description.include?('Lorem ipsum dolor sit amet, consectetuer adipiscing elit.') | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Eric Davis
|
r2732 | def test_add_issue_with_spaces_between_attribute_and_separator | ||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_with_spaces_between_attribute_and_separator.eml', | ||||
| :allow_override => 'tracker,category,priority' | ||||
| ) | ||||
|
Eric Davis
|
r2732 | assert issue.is_a?(Issue) | ||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'New ticket on a given project', issue.subject | ||||
| assert_equal User.find_by_login('jsmith'), issue.author | ||||
| assert_equal Project.find(2), issue.project | ||||
| assert_equal 'Feature request', issue.tracker.to_s | ||||
| assert_equal 'Stock management', issue.category.to_s | ||||
| assert_equal 'Urgent', issue.priority.to_s | ||||
| assert issue.description.include?('Lorem ipsum dolor sit amet, consectetuer adipiscing elit.') | ||||
| end | ||||
|
Jean-Philippe Lang
|
r1554 | def test_add_issue_with_attachment_to_specific_project | ||
|
Jean-Philippe Lang
|
r1629 | issue = submit_email('ticket_with_attachment.eml', :issue => {:project => 'onlinestore'}) | ||
|
Jean-Philippe Lang
|
r1554 | assert issue.is_a?(Issue) | ||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'Ticket created by email with attachment', issue.subject | ||||
| assert_equal User.find_by_login('jsmith'), issue.author | ||||
| assert_equal Project.find(2), issue.project | ||||
| assert_equal 'This is a new ticket with attachments', issue.description | ||||
| # Attachment properties | ||||
| assert_equal 1, issue.attachments.size | ||||
| assert_equal 'Paella.jpg', issue.attachments.first.filename | ||||
| assert_equal 'image/jpeg', issue.attachments.first.content_type | ||||
| assert_equal 10790, issue.attachments.first.filesize | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2209 | def test_add_issue_with_custom_fields | ||
| issue = submit_email('ticket_with_custom_fields.eml', :issue => {:project => 'onlinestore'}) | ||||
| assert issue.is_a?(Issue) | ||||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'New ticket with custom field values', issue.subject | ||||
|
Toshi MARUYAMA
|
r8621 | assert_equal 'Value for a custom field', | ||
| issue.custom_value_for(CustomField.find_by_name('Searchable field')).value | ||||
|
Jean-Philippe Lang
|
r2365 | assert !issue.description.match(/^searchable field:/i) | ||
|
Jean-Philippe Lang
|
r2209 | end | ||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2075 | def test_add_issue_with_cc | ||
| issue = submit_email('ticket_with_cc.eml', :issue => {:project => 'ecookbook'}) | ||||
| assert issue.is_a?(Issue) | ||||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert issue.watched_by?(User.find_by_mail('dlopper@somenet.foo')) | ||||
|
Eric Davis
|
r4104 | assert_equal 1, issue.watcher_user_ids.size | ||
|
Jean-Philippe Lang
|
r2075 | end | ||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2689 | def test_add_issue_by_unknown_user | ||
| assert_no_difference 'User.count' do | ||||
|
Toshi MARUYAMA
|
r8621 | assert_equal false, | ||
| submit_email( | ||||
| 'ticket_by_unknown_user.eml', | ||||
| :issue => {:project => 'ecookbook'} | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r2689 | end | ||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2689 | def test_add_issue_by_anonymous_user | ||
| Role.anonymous.add_permission!(:add_issues) | ||||
| assert_no_difference 'User.count' do | ||||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_by_unknown_user.eml', | ||||
| :issue => {:project => 'ecookbook'}, | ||||
| :unknown_user => 'accept' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r2689 | assert issue.is_a?(Issue) | ||
| assert issue.author.anonymous? | ||||
| end | ||||
| end | ||||
|
Eric Davis
|
r3669 | |||
| def test_add_issue_by_anonymous_user_with_no_from_address | ||||
| Role.anonymous.add_permission!(:add_issues) | ||||
| assert_no_difference 'User.count' do | ||||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_by_empty_user.eml', | ||||
| :issue => {:project => 'ecookbook'}, | ||||
| :unknown_user => 'accept' | ||||
| ) | ||||
|
Eric Davis
|
r3669 | assert issue.is_a?(Issue) | ||
| assert issue.author.anonymous? | ||||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r3081 | def test_add_issue_by_anonymous_user_on_private_project | ||
| Role.anonymous.add_permission!(:add_issues) | ||||
| assert_no_difference 'User.count' do | ||||
| assert_no_difference 'Issue.count' do | ||||
|
Toshi MARUYAMA
|
r8621 | assert_equal false, | ||
| submit_email( | ||||
| 'ticket_by_unknown_user.eml', | ||||
| :issue => {:project => 'onlinestore'}, | ||||
| :unknown_user => 'accept' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r3081 | end | ||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r3081 | def test_add_issue_by_anonymous_user_on_private_project_without_permission_check | ||
| assert_no_difference 'User.count' do | ||||
| assert_difference 'Issue.count' do | ||||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_by_unknown_user.eml', | ||||
| :issue => {:project => 'onlinestore'}, | ||||
| :no_permission_check => '1', | ||||
| :unknown_user => 'accept' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r3081 | assert issue.is_a?(Issue) | ||
| assert issue.author.anonymous? | ||||
| assert !issue.project.is_public? | ||||
|
Jean-Philippe Lang
|
r4302 | assert_equal [issue.id, 1, 2], [issue.root_id, issue.lft, issue.rgt] | ||
|
Jean-Philippe Lang
|
r3081 | end | ||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2689 | def test_add_issue_by_created_user | ||
| Setting.default_language = 'en' | ||||
| assert_difference 'User.count' do | ||||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_by_unknown_user.eml', | ||||
| :issue => {:project => 'ecookbook'}, | ||||
| :unknown_user => 'create' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r2689 | assert issue.is_a?(Issue) | ||
| assert issue.author.active? | ||||
| assert_equal 'john.doe@somenet.foo', issue.author.mail | ||||
| assert_equal 'John', issue.author.firstname | ||||
| assert_equal 'Doe', issue.author.lastname | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2689 | # account information | ||
| email = ActionMailer::Base.deliveries.first | ||||
| assert_not_nil email | ||||
| assert email.subject.include?('account activation') | ||||
|
Jean-Philippe Lang
|
r8972 | login = mail_body(email).match(/\* Login: (.*)$/)[1] | ||
| password = mail_body(email).match(/\* Password: (.*)$/)[1] | ||||
|
Jean-Philippe Lang
|
r2689 | assert_equal issue.author, User.try_to_login(login, password) | ||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2485 | def test_add_issue_without_from_header | ||
| Role.anonymous.add_permission!(:add_issues) | ||||
| assert_equal false, submit_email('ticket_without_from_header.eml') | ||||
| end | ||||
|
Jean-Philippe Lang
|
r4281 | |||
|
Jean-Philippe Lang
|
r4282 | def test_add_issue_with_invalid_attributes | ||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_with_invalid_attributes.eml', | ||||
| :allow_override => 'tracker,category,priority' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r4282 | assert issue.is_a?(Issue) | ||
| assert !issue.new_record? | ||||
| issue.reload | ||||
|
Jean-Philippe Lang
|
r4290 | assert_nil issue.assigned_to | ||
|
Jean-Philippe Lang
|
r4282 | assert_nil issue.start_date | ||
| assert_nil issue.due_date | ||||
| assert_equal 0, issue.done_ratio | ||||
| assert_equal 'Normal', issue.priority.to_s | ||||
| assert issue.description.include?('Lorem ipsum dolor sit amet, consectetuer adipiscing elit.') | ||||
| end | ||||
|
Jean-Philippe Lang
|
r4281 | def test_add_issue_with_localized_attributes | ||
| User.find_by_mail('jsmith@somenet.foo').update_attribute 'language', 'fr' | ||||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'ticket_with_localized_attributes.eml', | ||||
| :allow_override => 'tracker,category,priority' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r4281 | assert issue.is_a?(Issue) | ||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'New ticket on a given project', issue.subject | ||||
| assert_equal User.find_by_login('jsmith'), issue.author | ||||
| assert_equal Project.find(2), issue.project | ||||
| assert_equal 'Feature request', issue.tracker.to_s | ||||
| assert_equal 'Stock management', issue.category.to_s | ||||
| assert_equal 'Urgent', issue.priority.to_s | ||||
| assert issue.description.include?('Lorem ipsum dolor sit amet, consectetuer adipiscing elit.') | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r3208 | def test_add_issue_with_japanese_keywords | ||
|
Toshi MARUYAMA
|
r8398 | ja_dev = "\xe9\x96\x8b\xe7\x99\xba" | ||
| ja_dev.force_encoding('UTF-8') if ja_dev.respond_to?(:force_encoding) | ||||
| tracker = Tracker.create!(:name => ja_dev) | ||||
|
Jean-Philippe Lang
|
r3208 | Project.find(1).trackers << tracker | ||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'japanese_keywords_iso_2022_jp.eml', | ||||
| :issue => {:project => 'ecookbook'}, | ||||
| :allow_override => 'tracker' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r3208 | assert_kind_of Issue, issue | ||
| assert_equal tracker, issue.tracker | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6807 | |||
|
Jean-Philippe Lang
|
r6204 | def test_add_issue_from_apple_mail | ||
|
Toshi MARUYAMA
|
r8621 | issue = submit_email( | ||
| 'apple_mail_with_attachment.eml', | ||||
| :issue => {:project => 'ecookbook'} | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r6204 | assert_kind_of Issue, issue | ||
| assert_equal 1, issue.attachments.size | ||||
|
Toshi MARUYAMA
|
r6807 | |||
|
Jean-Philippe Lang
|
r6204 | attachment = issue.attachments.first | ||
| assert_equal 'paella.jpg', attachment.filename | ||||
| assert_equal 10790, attachment.filesize | ||||
|
Toshi MARUYAMA
|
r8599 | assert File.exist?(attachment.diskfile) | ||
| assert_equal 10790, File.size(attachment.diskfile) | ||||
| assert_equal 'caaf384198bcbc9563ab5c058acd73cd', attachment.digest | ||||
|
Jean-Philippe Lang
|
r6204 | end | ||
|
Eric Davis
|
r2548 | |||
|
Jean-Philippe Lang
|
r2908 | def test_should_ignore_emails_from_emission_address | ||
| Role.anonymous.add_permission!(:add_issues) | ||||
| assert_no_difference 'User.count' do | ||||
|
Toshi MARUYAMA
|
r8621 | assert_equal false, | ||
| submit_email( | ||||
| 'ticket_from_emission_address.eml', | ||||
| :issue => {:project => 'ecookbook'}, | ||||
| :unknown_user => 'create' | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r2908 | end | ||
| end | ||||
|
Eric Davis
|
r2548 | def test_add_issue_should_send_email_notification | ||
|
Eric Davis
|
r4107 | Setting.notified_events = ['issue_added'] | ||
|
Eric Davis
|
r2548 | ActionMailer::Base.deliveries.clear | ||
| # This email contains: 'Project: onlinestore' | ||||
| issue = submit_email('ticket_on_given_project.eml') | ||||
| assert issue.is_a?(Issue) | ||||
| assert_equal 1, ActionMailer::Base.deliveries.size | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r6192 | def test_update_issue | ||
|
Jean-Philippe Lang
|
r1554 | journal = submit_email('ticket_reply.eml') | ||
| assert journal.is_a?(Journal) | ||||
| assert_equal User.find_by_login('jsmith'), journal.user | ||||
| assert_equal Issue.find(2), journal.journalized | ||||
|
Eric Davis
|
r1750 | assert_match /This is reply/, journal.notes | ||
|
Jean-Philippe Lang
|
r4461 | assert_equal 'Feature request', journal.issue.tracker.name | ||
|
Eric Davis
|
r1750 | end | ||
|
Jean-Philippe Lang
|
r6192 | def test_update_issue_with_attribute_changes | ||
|
Eric Davis
|
r1750 | # This email contains: 'Status: Resolved' | ||
| journal = submit_email('ticket_reply_with_status.eml') | ||||
| assert journal.is_a?(Journal) | ||||
| issue = Issue.find(journal.issue.id) | ||||
| assert_equal User.find_by_login('jsmith'), journal.user | ||||
| assert_equal Issue.find(2), journal.journalized | ||||
| assert_match /This is reply/, journal.notes | ||||
|
Jean-Philippe Lang
|
r4461 | assert_equal 'Feature request', journal.issue.tracker.name | ||
|
Eric Davis
|
r1750 | assert_equal IssueStatus.find_by_name("Resolved"), issue.status | ||
|
Eric Davis
|
r3649 | assert_equal '2010-01-01', issue.start_date.to_s | ||
| assert_equal '2010-12-31', issue.due_date.to_s | ||||
|
Eric Davis
|
r3650 | assert_equal User.find_by_login('jsmith'), issue.assigned_to | ||
|
Jean-Philippe Lang
|
r4461 | assert_equal "52.6", issue.custom_value_for(CustomField.find_by_name('Float field')).value | ||
|
Jean-Philippe Lang
|
r4985 | # keywords should be removed from the email body | ||
| assert !journal.notes.match(/^Status:/i) | ||||
| assert !journal.notes.match(/^Start Date:/i) | ||||
|
Jean-Philippe Lang
|
r521 | end | ||
|
Toshi MARUYAMA
|
r6807 | |||
|
Jean-Philippe Lang
|
r6192 | def test_update_issue_with_attachment | ||
| assert_difference 'Journal.count' do | ||||
| assert_difference 'JournalDetail.count' do | ||||
| assert_difference 'Attachment.count' do | ||||
| assert_no_difference 'Issue.count' do | ||||
| journal = submit_email('ticket_with_attachment.eml') do |raw| | ||||
| raw.gsub! /^Subject: .*$/, 'Subject: Re: [Cookbook - Feature #2] (New) Add ingredients categories' | ||||
| end | ||||
| end | ||||
| end | ||||
| end | ||||
| end | ||||
| journal = Journal.first(:order => 'id DESC') | ||||
| assert_equal Issue.find(2), journal.journalized | ||||
| assert_equal 1, journal.details.size | ||||
|
Toshi MARUYAMA
|
r6807 | |||
|
Jean-Philippe Lang
|
r6192 | detail = journal.details.first | ||
| assert_equal 'attachment', detail.property | ||||
| assert_equal 'Paella.jpg', detail.value | ||||
| end | ||||
|
Eric Davis
|
r2548 | |||
|
Jean-Philippe Lang
|
r6192 | def test_update_issue_should_send_email_notification | ||
|
Eric Davis
|
r2548 | ActionMailer::Base.deliveries.clear | ||
| journal = submit_email('ticket_reply.eml') | ||||
| assert journal.is_a?(Journal) | ||||
| assert_equal 1, ActionMailer::Base.deliveries.size | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r6192 | def test_update_issue_should_not_set_defaults | ||
|
Toshi MARUYAMA
|
r8621 | journal = submit_email( | ||
| 'ticket_reply.eml', | ||||
| :issue => {:tracker => 'Support request', :priority => 'High'} | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r4462 | assert journal.is_a?(Journal) | ||
| assert_match /This is reply/, journal.notes | ||||
| assert_equal 'Feature request', journal.issue.tracker.name | ||||
| assert_equal 'Normal', journal.issue.priority.name | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2287 | def test_reply_to_a_message | ||
| m = submit_email('message_reply.eml') | ||||
| assert m.is_a?(Message) | ||||
| assert !m.new_record? | ||||
| m.reload | ||||
| assert_equal 'Reply via email', m.subject | ||||
| # The email replies to message #2 which is part of the thread of message #1 | ||||
| assert_equal Message.find(1), m.parent | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2292 | def test_reply_to_a_message_by_subject | ||
| m = submit_email('message_reply_by_subject.eml') | ||||
| assert m.is_a?(Message) | ||||
| assert !m.new_record? | ||||
| m.reload | ||||
| assert_equal 'Reply to the first post', m.subject | ||||
| assert_equal Message.find(1), m.parent | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r2134 | def test_should_strip_tags_of_html_only_emails | ||
| issue = submit_email('ticket_html_only.eml', :issue => {:project => 'ecookbook'}) | ||||
| assert issue.is_a?(Issue) | ||||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| assert_equal 'HTML email', issue.subject | ||||
| assert_equal 'This is a html-only email.', issue.description | ||||
| end | ||||
|
Jean-Philippe Lang
|
r521 | |||
|
Jean-Philippe Lang
|
r3112 | context "truncate emails based on the Setting" do | ||
| context "with no setting" do | ||||
| setup do | ||||
| Setting.mail_handler_body_delimiters = '' | ||||
| end | ||||
| should "add the entire email into the issue" do | ||||
| issue = submit_email('ticket_on_given_project.eml') | ||||
| assert_issue_created(issue) | ||||
| assert issue.description.include?('---') | ||||
| assert issue.description.include?('This paragraph is after the delimiter') | ||||
| end | ||||
| end | ||||
| context "with a single string" do | ||||
| setup do | ||||
| Setting.mail_handler_body_delimiters = '---' | ||||
| end | ||||
| should "truncate the email at the delimiter for the issue" do | ||||
| issue = submit_email('ticket_on_given_project.eml') | ||||
| assert_issue_created(issue) | ||||
| assert issue.description.include?('This paragraph is before delimiters') | ||||
| assert issue.description.include?('--- This line starts with a delimiter') | ||||
| assert !issue.description.match(/^---$/) | ||||
| assert !issue.description.include?('This paragraph is after the delimiter') | ||||
| end | ||||
| end | ||||
|
Eric Davis
|
r4247 | context "with a single quoted reply (e.g. reply to a Redmine email notification)" do | ||
| setup do | ||||
| Setting.mail_handler_body_delimiters = '--- Reply above. Do not remove this line. ---' | ||||
| end | ||||
| should "truncate the email at the delimiter with the quoted reply symbols (>)" do | ||||
| journal = submit_email('issue_update_with_quoted_reply_above.eml') | ||||
| assert journal.is_a?(Journal) | ||||
| assert journal.notes.include?('An update to the issue by the sender.') | ||||
| assert !journal.notes.match(Regexp.escape("--- Reply above. Do not remove this line. ---")) | ||||
| assert !journal.notes.include?('Looks like the JSON api for projects was missed.') | ||||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Eric Davis
|
r4247 | context "with multiple quoted replies (e.g. reply to a reply of a Redmine email notification)" do | ||
| setup do | ||||
| Setting.mail_handler_body_delimiters = '--- Reply above. Do not remove this line. ---' | ||||
| end | ||||
| should "truncate the email at the delimiter with the quoted reply symbols (>)" do | ||||
| journal = submit_email('issue_update_with_multiple_quoted_reply_above.eml') | ||||
| assert journal.is_a?(Journal) | ||||
| assert journal.notes.include?('An update to the issue by the sender.') | ||||
| assert !journal.notes.match(Regexp.escape("--- Reply above. Do not remove this line. ---")) | ||||
| assert !journal.notes.include?('Looks like the JSON api for projects was missed.') | ||||
| end | ||||
| end | ||||
|
Jean-Philippe Lang
|
r3112 | context "with multiple strings" do | ||
| setup do | ||||
| Setting.mail_handler_body_delimiters = "---\nBREAK" | ||||
| end | ||||
| should "truncate the email at the first delimiter found (BREAK)" do | ||||
| issue = submit_email('ticket_on_given_project.eml') | ||||
| assert_issue_created(issue) | ||||
| assert issue.description.include?('This paragraph is before delimiters') | ||||
| assert !issue.description.include?('BREAK') | ||||
| assert !issue.description.include?('This paragraph is between delimiters') | ||||
| assert !issue.description.match(/^---$/) | ||||
| assert !issue.description.include?('This paragraph is after the delimiter') | ||||
| end | ||||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r5439 | |||
|
Eric Davis
|
r3687 | def test_email_with_long_subject_line | ||
| issue = submit_email('ticket_with_long_subject.eml') | ||||
| assert issue.is_a?(Issue) | ||||
| assert_equal issue.subject, 'New ticket on a given project with a very long subject line which exceeds 255 chars and should not be ignored but chopped off. And if the subject line is still not long enough, we just add more text. And more text. Wow, this is really annoying. Especially, if you have nothing to say...'[0,255] | ||||
| end | ||||
|
Jean-Philippe Lang
|
r3112 | |||
|
Jean-Philippe Lang
|
r7832 | def test_new_user_from_attributes_should_return_valid_user | ||
| to_test = { | ||||
| # [address, name] => [login, firstname, lastname] | ||||
| ['jsmith@example.net', nil] => ['jsmith@example.net', 'jsmith', '-'], | ||||
| ['jsmith@example.net', 'John'] => ['jsmith@example.net', 'John', '-'], | ||||
| ['jsmith@example.net', 'John Smith'] => ['jsmith@example.net', 'John', 'Smith'], | ||||
| ['jsmith@example.net', 'John Paul Smith'] => ['jsmith@example.net', 'John', 'Paul Smith'], | ||||
| ['jsmith@example.net', 'AVeryLongFirstnameThatExceedsTheMaximumLength Smith'] => ['jsmith@example.net', 'AVeryLongFirstnameThatExceedsT', 'Smith'], | ||||
|
Jean-Philippe Lang
|
r8658 | ['jsmith@example.net', 'John AVeryLongLastnameThatExceedsTheMaximumLength'] => ['jsmith@example.net', 'John', 'AVeryLongLastnameThatExceedsTh'] | ||
|
Jean-Philippe Lang
|
r7832 | } | ||
| to_test.each do |attrs, expected| | ||||
| user = MailHandler.new_user_from_attributes(attrs.first, attrs.last) | ||||
|
Jean-Philippe Lang
|
r8666 | assert user.valid?, user.errors.full_messages.to_s | ||
|
Jean-Philippe Lang
|
r7832 | assert_equal attrs.first, user.mail | ||
| assert_equal expected[0], user.login | ||||
| assert_equal expected[1], user.firstname | ||||
| assert_equal expected[2], user.lastname | ||||
| end | ||||
| end | ||||
| def test_new_user_from_attributes_should_respect_minimum_password_length | ||||
| with_settings :password_min_length => 15 do | ||||
| user = MailHandler.new_user_from_attributes('jsmith@example.net') | ||||
| assert user.valid? | ||||
| assert user.password.length >= 15 | ||||
| end | ||||
| end | ||||
|
Toshi MARUYAMA
|
r8399 | |||
|
Jean-Philippe Lang
|
r7832 | def test_new_user_from_attributes_should_use_default_login_if_invalid | ||
|
Jean-Philippe Lang
|
r8658 | user = MailHandler.new_user_from_attributes('foo+bar@example.net') | ||
|
Jean-Philippe Lang
|
r7832 | assert user.valid? | ||
| assert user.login =~ /^user[a-f0-9]+$/ | ||||
|
Jean-Philippe Lang
|
r8658 | assert_equal 'foo+bar@example.net', user.mail | ||
|
Jean-Philippe Lang
|
r7832 | end | ||
|
Jean-Philippe Lang
|
r8923 | def test_new_user_with_utf8_encoded_fullname_should_be_decoded | ||
| assert_difference 'User.count' do | ||||
| issue = submit_email( | ||||
| 'fullname_of_sender_as_utf8_encoded.eml', | ||||
| :issue => {:project => 'ecookbook'}, | ||||
| :unknown_user => 'create' | ||||
| ) | ||||
| end | ||||
| user = User.first(:order => 'id DESC') | ||||
| assert_equal "foo@example.org", user.mail | ||||
| assert_equal "Ää", user.firstname | ||||
| assert_equal "Öö", user.lastname | ||||
| end | ||||
|
Jean-Philippe Lang
|
r521 | private | ||
|
Toshi MARUYAMA
|
r5439 | |||
|
Jean-Philippe Lang
|
r1554 | def submit_email(filename, options={}) | ||
| raw = IO.read(File.join(FIXTURES_PATH, filename)) | ||||
|
Jean-Philippe Lang
|
r6186 | yield raw if block_given? | ||
|
Jean-Philippe Lang
|
r1554 | MailHandler.receive(raw, options) | ||
| end | ||||
|
Jean-Philippe Lang
|
r3112 | |||
| def assert_issue_created(issue) | ||||
| assert issue.is_a?(Issue) | ||||
| assert !issue.new_record? | ||||
| issue.reload | ||||
| end | ||||
|
Jean-Philippe Lang
|
r521 | end | ||