+1

On Thu, Dec 10, 2009 at 3:49 PM, Jesse Wolfe <[email protected]> wrote:
> the ldap/connection spec creates a mock LDAP class that causes `require 
> 'ldap'`
> to explode. If `require 'ldap'` happens first, then we're OK.
>
> Signed-off-by: Jesse Wolfe <[email protected]>
> ---
>  spec/unit/util/ldap/connection.rb |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/spec/unit/util/ldap/connection.rb 
> b/spec/unit/util/ldap/connection.rb
> index 8bc85a6..7a76dbc 100755
> --- a/spec/unit/util/ldap/connection.rb
> +++ b/spec/unit/util/ldap/connection.rb
> @@ -7,6 +7,11 @@ require File.dirname(__FILE__) + '/../../../spec_helper'
>
>  require 'puppet/util/ldap/connection'
>
> +begin
> +    require 'ldap'
> +rescue LoadError
> +end
> +
>  # So our mocks and such all work, even when ldap isn't available.
>  unless defined?(LDAP::Conn)
>     class LDAP
> --
> 1.6.5
>
> --
>
> You received this message because you are subscribed to the Google Groups 
> "Puppet Developers" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to 
> [email protected].
> For more options, visit this group at 
> http://groups.google.com/group/puppet-dev?hl=en.
>
>
>

--

You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en.


Reply via email to