mirror of
https://github.com/codeninjasllc/discourse.git
synced 2024-11-30 19:08:10 -05:00
c8d322d1be
This change has a tradeoff. It increases our backscatter vulnerability - the subject could have spammy content - but it's extremely valuable to the user to know exactly which message was rejected. If you sent two at the same time, and only one was rejected, you would have no way of knowing which worked and which to resend without going to the website (which is what email-in is trying to avoid, kinda).
112 lines
2.7 KiB
Ruby
112 lines
2.7 KiB
Ruby
require 'spec_helper'
|
|
require_dependency 'jobs/regular/process_post'
|
|
|
|
describe Jobs::PollMailbox do
|
|
|
|
let!(:poller) { Jobs::PollMailbox.new }
|
|
|
|
describe ".execute" do
|
|
|
|
it "does no polling if pop3s_polling_enabled is false" do
|
|
SiteSetting.expects(:pop3s_polling_enabled?).returns(false)
|
|
poller.expects(:poll_pop3s).never
|
|
|
|
poller.execute({})
|
|
end
|
|
|
|
describe "with pop3s_polling_enabled" do
|
|
|
|
it "calls poll_pop3s" do
|
|
SiteSetting.expects(:pop3s_polling_enabled?).returns(true)
|
|
poller.expects(:poll_pop3s).once
|
|
|
|
poller.execute({})
|
|
end
|
|
end
|
|
|
|
end
|
|
|
|
describe ".poll_pop3s" do
|
|
|
|
it "logs an error on pop authentication error" do
|
|
error = Net::POPAuthenticationError.new
|
|
data = { limit_once_per: 1.hour, message_params: { error: error }}
|
|
|
|
Net::POP3.expects(:start).raises(error)
|
|
|
|
Discourse.expects(:handle_exception)
|
|
|
|
poller.poll_pop3s
|
|
end
|
|
|
|
end
|
|
|
|
describe "processing email" do
|
|
|
|
let!(:receiver) { mock }
|
|
let!(:email_string) { fixture_file("emails/valid_incoming.eml") }
|
|
let!(:email) { mock }
|
|
|
|
before do
|
|
email.stubs(:pop).returns(email_string)
|
|
Email::Receiver.expects(:new).with(email_string).returns(receiver)
|
|
end
|
|
|
|
describe "all goes fine" do
|
|
|
|
it "email gets deleted" do
|
|
receiver.expects(:process)
|
|
email.expects(:delete)
|
|
|
|
poller.handle_mail(email)
|
|
end
|
|
end
|
|
|
|
describe "raises Untrusted error" do
|
|
|
|
it "sends a reply and deletes the email" do
|
|
receiver.expects(:process).raises(Email::Receiver::UserNotSufficientTrustLevelError)
|
|
email.expects(:delete)
|
|
|
|
message = Mail::Message.new(email_string)
|
|
Mail::Message.expects(:new).with(email_string).returns(message)
|
|
|
|
client_message = mock
|
|
sender_object = mock
|
|
|
|
RejectionMailer.expects(:send_rejection).with(
|
|
message.from, message.body, message.subject, message.to, :email_reject_trust_level
|
|
).returns(client_message)
|
|
Email::Sender.expects(:new).with(client_message, :email_reject_trust_level).returns(sender_object)
|
|
sender_object.expects(:send)
|
|
|
|
poller.handle_mail(email)
|
|
end
|
|
end
|
|
|
|
describe "raises error" do
|
|
|
|
[ Email::Receiver::ProcessingError,
|
|
Email::Receiver::EmailUnparsableError,
|
|
Email::Receiver::EmptyEmailError,
|
|
Email::Receiver::UserNotFoundError,
|
|
Email::Receiver::EmailLogNotFound,
|
|
ActiveRecord::Rollback,
|
|
TypeError
|
|
].each do |exception|
|
|
|
|
it "deletes email on #{exception}" do
|
|
receiver.expects(:process).raises(exception)
|
|
email.expects(:delete)
|
|
|
|
Discourse.stubs(:handle_exception)
|
|
|
|
poller.handle_mail(email)
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
end
|
|
|
|
end
|