1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48
|
From: Lucas Kanashiro <lucas.kanashiro@canonical.com>
Date: Thu, 2 Jul 2020 15:08:00 -0300
Subject: Skip tests relying on a DNS setup
The tests skipped failed in Ubuntu infrastructure and it passed in
Debian buildd and local chroots. Those tests rely on a DNS to perform
some MX record lookups. As far as I can see in Debian and local chroots
some DNS information is leaking and that is why the tests pass. I filed
a bug upstream to see if they can try to mock the MX record lookup some
way:
https://github.com/hallelujah/valid_email/issues/107
Forwarded: not-needed
---
spec/email_validator_spec.rb | 6 +++---
spec/validate_email_spec.rb | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
--- a/spec/email_validator_spec.rb
+++ b/spec/email_validator_spec.rb
@@ -188,13 +188,13 @@ describe EmailValidator do
describe "validating email with MX" do
subject { person_class_mx.new }
- it "passes when email domain has MX record" do
+ xit "passes when email domain has MX record" do
subject.email = 'john@gmail.com'
expect(subject.valid?).to be_truthy
expect(subject.errors[:email]).to be_empty
end
- it "fails when email domain has no MX record" do
+ xit "fails when email domain has no MX record" do
subject.email = 'john@subdomain.rubyonrails.org'
expect(subject.valid?).to be_falsey
expect(subject).to have_error_messages(:email, errors)
--- a/spec/validate_email_spec.rb
+++ b/spec/validate_email_spec.rb
@@ -22,7 +22,7 @@ describe ValidateEmail do
allow(Resolv::DNS).to receive(:new).and_return(dns)
end
- it 'returns true when mx record exist' do
+ xit 'returns true when mx record exist' do
expect(ValidateEmail.valid?('user@gmail.com', mx: true)).to be_truthy
end
|