Skip to content

Commit 1b595b2

Browse files
committed
let's assume folks will have to modify /etc/hosts
it's a little price to pay in the current setup and allows us to have the same tests locally and dev.
1 parent f38c39c commit 1b595b2

File tree

1 file changed

+1
-9
lines changed

1 file changed

+1
-9
lines changed

test/integration/test_bind.rb

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -151,13 +151,9 @@ def test_bind_tls_with_bogus_hostname_system_ca_fails
151151
)
152152
end
153153

154-
# The following depend on /etc/hosts hacking.
155-
# We can do that on CI, but it's less than cool on people's dev boxes
156154
def test_bind_tls_with_multiple_hosts
157-
omit_unless ENV['TRAVIS'] == 'true'
158-
159155
@ldap.host = nil
160-
@ldap.hosts = [['ldap01.example.com', 389], ['ldap02.example.com', 389]]
156+
@ldap.hosts = [[INTEGRATION_HOSTNAME, 389], [INTEGRATION_HOSTNAME, 389]]
161157
@ldap.encryption(
162158
method: :start_tls,
163159
tls_options: TLS_OPTS.merge(verify_mode: OpenSSL::SSL::VERIFY_PEER,
@@ -168,8 +164,6 @@ def test_bind_tls_with_multiple_hosts
168164
end
169165

170166
def test_bind_tls_with_multiple_bogus_hosts
171-
omit_unless ENV['TRAVIS'] == 'true'
172-
173167
@ldap.host = nil
174168
@ldap.hosts = [['127.0.0.1', 389], ['bogus.example.com', 389]]
175169
@ldap.encryption(
@@ -186,8 +180,6 @@ def test_bind_tls_with_multiple_bogus_hosts
186180
end
187181

188182
def test_bind_tls_with_multiple_bogus_hosts_no_verification
189-
omit_unless ENV['TRAVIS'] == 'true'
190-
191183
@ldap.host = nil
192184
@ldap.hosts = [['127.0.0.1', 389], ['bogus.example.com', 389]]
193185
@ldap.encryption(

0 commit comments

Comments
 (0)