2018-12-10 13:53:25 -08:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'rails_helper'
|
|
|
|
|
|
|
|
describe EmailMxValidator do
|
|
|
|
describe '#validate' do
|
2022-02-24 08:28:23 -08:00
|
|
|
let(:user) { double(email: 'foo@example.com', sign_up_ip: '1.2.3.4', errors: double(add: nil)) }
|
|
|
|
|
|
|
|
context 'for an e-mail domain that is explicitly allowed' do
|
|
|
|
around do |block|
|
|
|
|
tmp = Rails.configuration.x.email_domains_whitelist
|
|
|
|
Rails.configuration.x.email_domains_whitelist = 'example.com'
|
|
|
|
block.call
|
|
|
|
Rails.configuration.x.email_domains_whitelist = tmp
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not add errors if there are no DNS records' do
|
|
|
|
resolver = double
|
|
|
|
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::MX).and_return([])
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::A).and_return([])
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
|
|
|
allow(resolver).to receive(:timeouts=).and_return(nil)
|
|
|
|
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
|
|
|
|
|
|
|
|
subject.validate(user)
|
|
|
|
expect(user.errors).to_not have_received(:add)
|
|
|
|
end
|
2021-03-19 15:48:47 -07:00
|
|
|
end
|
|
|
|
|
2023-01-24 11:18:41 -08:00
|
|
|
it 'adds no error if there are DNS records for the e-mail domain' do
|
|
|
|
resolver = double
|
|
|
|
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::MX).and_return([])
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::A).and_return([Resolv::DNS::Resource::IN::A.new('192.0.2.42')])
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
|
|
|
allow(resolver).to receive(:timeouts=).and_return(nil)
|
|
|
|
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
|
|
|
|
|
|
|
|
subject.validate(user)
|
2023-02-19 17:33:27 -08:00
|
|
|
expect(user.errors).to_not have_received(:add)
|
2023-01-24 11:18:41 -08:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'adds an error if the email domain name contains empty labels' do
|
|
|
|
resolver = double
|
|
|
|
|
|
|
|
allow(resolver).to receive(:getresources).with('example..com', Resolv::DNS::Resource::IN::MX).and_return([])
|
|
|
|
allow(resolver).to receive(:getresources).with('example..com', Resolv::DNS::Resource::IN::A).and_return([Resolv::DNS::Resource::IN::A.new('192.0.2.42')])
|
|
|
|
allow(resolver).to receive(:getresources).with('example..com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
|
|
|
allow(resolver).to receive(:timeouts=).and_return(nil)
|
|
|
|
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
|
|
|
|
|
|
|
|
user = double(email: 'foo@example..com', sign_up_ip: '1.2.3.4', errors: double(add: nil))
|
|
|
|
subject.validate(user)
|
|
|
|
expect(user.errors).to have_received(:add)
|
|
|
|
end
|
|
|
|
|
2018-12-10 13:53:25 -08:00
|
|
|
it 'adds an error if there are no DNS records for the e-mail domain' do
|
|
|
|
resolver = double
|
|
|
|
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::MX).and_return([])
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::A).and_return([])
|
2019-02-12 05:48:04 -08:00
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
2018-12-10 13:53:25 -08:00
|
|
|
allow(resolver).to receive(:timeouts=).and_return(nil)
|
|
|
|
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
|
|
|
|
|
|
|
|
subject.validate(user)
|
|
|
|
expect(user.errors).to have_received(:add)
|
|
|
|
end
|
|
|
|
|
2022-02-24 08:28:23 -08:00
|
|
|
it 'adds an error if a MX record does not lead to an IP' do
|
2018-12-10 13:53:25 -08:00
|
|
|
resolver = double
|
|
|
|
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::MX).and_return([double(exchange: 'mail.example.com')])
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::A).and_return([])
|
2019-02-12 05:48:04 -08:00
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
2018-12-10 13:53:25 -08:00
|
|
|
allow(resolver).to receive(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::A).and_return([])
|
2019-02-12 05:48:04 -08:00
|
|
|
allow(resolver).to receive(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
2018-12-10 13:53:25 -08:00
|
|
|
allow(resolver).to receive(:timeouts=).and_return(nil)
|
|
|
|
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
|
|
|
|
|
|
|
|
subject.validate(user)
|
|
|
|
expect(user.errors).to have_received(:add)
|
|
|
|
end
|
|
|
|
|
2022-02-24 08:28:23 -08:00
|
|
|
it 'adds an error if the MX record is blacklisted' do
|
2018-12-10 13:53:25 -08:00
|
|
|
EmailDomainBlock.create!(domain: 'mail.example.com')
|
|
|
|
resolver = double
|
|
|
|
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::MX).and_return([double(exchange: 'mail.example.com')])
|
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::A).and_return([])
|
2019-02-12 05:48:04 -08:00
|
|
|
allow(resolver).to receive(:getresources).with('example.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
|
2018-12-10 13:53:25 -08:00
|
|
|
allow(resolver).to receive(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::A).and_return([double(address: '2.3.4.5')])
|
2019-02-12 05:48:04 -08:00
|
|
|
allow(resolver).to receive(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::AAAA).and_return([double(address: 'fd00::2')])
|
2018-12-10 13:53:25 -08:00
|
|
|
allow(resolver).to receive(:timeouts=).and_return(nil)
|
|
|
|
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
|
|
|
|
|
|
|
|
subject.validate(user)
|
|
|
|
expect(user.errors).to have_received(:add)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|