File: 0002-Skip-tests-relying-on-a-DNS-setup.patch

package info (click to toggle)
ruby-valid-email 0.2.1-1
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid
  • size: 288 kB
  • sloc: ruby: 631; makefile: 4
file content (48 lines) | stat: -rw-r--r-- 1,825 bytes parent folder | download
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