Fixed tests to handle the new error return from LDAP#bind.
This commit is contained in:
parent
0117d386c0
commit
030bb9c6d4
|
@ -65,15 +65,21 @@ class TestLdapClient < Test::Unit::TestCase
|
||||||
# TODO, use constants for the LDAP result codes, rather than hardcoding them.
|
# TODO, use constants for the LDAP result codes, rather than hardcoding them.
|
||||||
def test_bind
|
def test_bind
|
||||||
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => @auth
|
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => @auth
|
||||||
assert_equal( 0, ldap.bind )
|
assert_equal( true, ldap.bind )
|
||||||
|
assert_equal( 0, ldap.get_operation_result.code )
|
||||||
|
assert_equal( "Success", ldap.get_operation_result.message )
|
||||||
|
|
||||||
bad_username = @auth.merge( {:username => "cn=badguy,dc=imposters,dc=com"} )
|
bad_username = @auth.merge( {:username => "cn=badguy,dc=imposters,dc=com"} )
|
||||||
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => bad_username
|
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => bad_username
|
||||||
assert_equal( 48, ldap.bind )
|
assert_equal( false, ldap.bind )
|
||||||
|
assert_equal( 48, ldap.get_operation_result.code )
|
||||||
|
assert_equal( "Inappropriate Authentication", ldap.get_operation_result.message )
|
||||||
|
|
||||||
bad_password = @auth.merge( {:password => "cornhusk"} )
|
bad_password = @auth.merge( {:password => "cornhusk"} )
|
||||||
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => bad_password
|
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => bad_password
|
||||||
assert_equal( 49, ldap.bind )
|
assert_equal( false, ldap.bind )
|
||||||
|
assert_equal( 49, ldap.get_operation_result.code )
|
||||||
|
assert_equal( "Invalid Credentials", ldap.get_operation_result.message )
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
||||||
|
@ -98,7 +104,7 @@ class TestLdapClient < Test::Unit::TestCase
|
||||||
# This is a helper routine for test_search_attributes.
|
# This is a helper routine for test_search_attributes.
|
||||||
def internal_test_search_attributes attrs_to_search
|
def internal_test_search_attributes attrs_to_search
|
||||||
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => @auth
|
ldap = Net::LDAP.new :host => @host, :port => @port, :auth => @auth
|
||||||
assert_equal( 0, ldap.bind )
|
assert( ldap.bind )
|
||||||
|
|
||||||
search = {
|
search = {
|
||||||
:base => "dc=bayshorenetworks,dc=com",
|
:base => "dc=bayshorenetworks,dc=com",
|
||||||
|
|
Loading…
Reference in a new issue