Skip to content

Commit

Permalink
Fix the unit test connection
Browse files Browse the repository at this point in the history
  • Loading branch information
zeroSteiner committed Dec 7, 2024
1 parent fb6a818 commit de167b6
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:
push:
branches:
- master
- testing

jobs:
test:
Expand Down
5 changes: 3 additions & 2 deletions test/integration/test_bind.rb
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
require 'resolv'
require_relative '../test_helper'

class TestBindIntegration < LDAPIntegrationTestCase
Expand Down Expand Up @@ -50,7 +51,7 @@ def test_bind_tls_with_cafile
def test_bind_tls_with_bad_hostname_no_verify_hostname_no_ca_passes
omit_unless TLS_OPTS.key?(:verify_hostname)

@ldap.host = '127.0.0.1'
@ldap.host = Resolv.getaddress(INTEGRATION_HOSTNAME)
@ldap.encryption(
method: :start_tls,
tls_options: { verify_mode: OpenSSL::SSL::VERIFY_PEER,
Expand All @@ -64,7 +65,7 @@ def test_bind_tls_with_bad_hostname_no_verify_hostname_no_ca_passes
def test_bind_tls_with_bad_hostname_no_verify_hostname_no_ca_opt_merge_passes
omit_unless TLS_OPTS.key?(:verify_hostname)

@ldap.host = '127.0.0.1'
@ldap.host = Resolv.getaddress(INTEGRATION_HOSTNAME)
@ldap.encryption(
method: :start_tls,
tls_options: TLS_OPTS.merge(verify_mode: OpenSSL::SSL::VERIFY_PEER,
Expand Down

0 comments on commit de167b6

Please sign in to comment.