Skip to content

Commit

Permalink
Add DomainHelpers spec support module for DNS/MX stub (mastodon#32690)
Browse files Browse the repository at this point in the history
  • Loading branch information
mjankowski authored Oct 29, 2024
1 parent de1d8dc commit df3b954
Show file tree
Hide file tree
Showing 4 changed files with 47 additions and 22 deletions.
12 changes: 1 addition & 11 deletions spec/controllers/auth/registrations_controller_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -233,17 +233,7 @@
Setting.registrations_mode = 'open'
Fabricate(:email_domain_block, allow_with_approval: true, domain: 'mail.example.com')
allow(User).to receive(:skip_mx_check?).and_return(false)

resolver = instance_double(Resolv::DNS, :timeouts= => nil)

allow(resolver).to receive(:getresources)
.with('example.com', Resolv::DNS::Resource::IN::MX)
.and_return([instance_double(Resolv::DNS::Resource::MX, exchange: 'mail.example.com')])
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(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::A).and_return([instance_double(Resolv::DNS::Resource::IN::A, address: '2.3.4.5')])
allow(resolver).to receive(:getresources).with('mail.example.com', Resolv::DNS::Resource::IN::AAAA).and_return([instance_double(Resolv::DNS::Resource::IN::AAAA, address: 'fd00::2')])
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
configure_mx(domain: 'example.com', exchange: 'mail.example.com')
end

it 'creates unapproved user and redirects to setup' do
Expand Down
1 change: 1 addition & 0 deletions spec/rails_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,7 @@ def sign_in(resource, _deprecated = nil, scope: nil)
config.include ActiveSupport::Testing::TimeHelpers
config.include Chewy::Rspec::Helpers
config.include Redisable
config.include DomainHelpers
config.include ThreadingHelpers
config.include SignedRequestHelpers, type: :request
config.include CommandLineHelpers, type: :cli
Expand Down
12 changes: 1 addition & 11 deletions spec/services/app_sign_up_service_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -53,17 +53,7 @@
Setting.registrations_mode = 'open'
Fabricate(:email_domain_block, allow_with_approval: true, domain: 'smtp.email.com')
allow(User).to receive(:skip_mx_check?).and_return(false)

resolver = instance_double(Resolv::DNS, :timeouts= => nil)

allow(resolver).to receive(:getresources)
.with('email.com', Resolv::DNS::Resource::IN::MX)
.and_return([instance_double(Resolv::DNS::Resource::MX, exchange: 'smtp.email.com')])
allow(resolver).to receive(:getresources).with('email.com', Resolv::DNS::Resource::IN::A).and_return([])
allow(resolver).to receive(:getresources).with('email.com', Resolv::DNS::Resource::IN::AAAA).and_return([])
allow(resolver).to receive(:getresources).with('smtp.email.com', Resolv::DNS::Resource::IN::A).and_return([instance_double(Resolv::DNS::Resource::IN::A, address: '2.3.4.5')])
allow(resolver).to receive(:getresources).with('smtp.email.com', Resolv::DNS::Resource::IN::AAAA).and_return([instance_double(Resolv::DNS::Resource::IN::AAAA, address: 'fd00::2')])
allow(Resolv::DNS).to receive(:open).and_yield(resolver)
configure_mx(domain: 'email.com', exchange: 'smtp.email.com')
end

it 'creates an unapproved user', :aggregate_failures do
Expand Down
44 changes: 44 additions & 0 deletions spec/support/domain_helpers.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
# frozen_string_literal: true

module DomainHelpers
def configure_mx(domain:, exchange:, ip_v4_addr: '2.3.4.5', ip_v6_addr: 'fd00::2')
resolver = instance_double(Resolv::DNS, :timeouts= => nil)

allow(resolver).to receive(:getresources)
.with(domain, Resolv::DNS::Resource::IN::MX)
.and_return([double_mx(exchange)])
allow(resolver)
.to receive(:getresources)
.with(domain, Resolv::DNS::Resource::IN::A)
.and_return([])
allow(resolver)
.to receive(:getresources)
.with(domain, Resolv::DNS::Resource::IN::AAAA)
.and_return([])
allow(resolver)
.to receive(:getresources)
.with(exchange, Resolv::DNS::Resource::IN::A)
.and_return([double_resource_v4(ip_v4_addr)])
allow(resolver)
.to receive(:getresources)
.with(exchange, Resolv::DNS::Resource::IN::AAAA)
.and_return([double_resource_v6(ip_v6_addr)])
allow(Resolv::DNS)
.to receive(:open)
.and_yield(resolver)
end

private

def double_mx(exchange)
instance_double(Resolv::DNS::Resource::MX, exchange: exchange)
end

def double_resource_v4(addr)
instance_double(Resolv::DNS::Resource::IN::A, address: addr)
end

def double_resource_v6(addr)
instance_double(Resolv::DNS::Resource::IN::AAAA, address: addr)
end
end

0 comments on commit df3b954

Please sign in to comment.