devise_ldap_authenticatable icon indicating copy to clipboard operation
devise_ldap_authenticatable copied to clipboard

Remove testing dependency on openldap

Open cschiewek opened this issue 9 years ago • 3 comments

Attn: @acurley @gustaflindqvist

I'd like to remove the openldap dependency from the test suite. This would allow us to setup on travis and greatly simplify the test suite in general.

I think https://github.com/NUBIC/ladle would facilitate this, but haven't done any significant investigation into it.

Let me know your thoughts...

cschiewek avatar Mar 19 '15 17:03 cschiewek

I think your idea is correct. Our friends at Github use ladle for testing apps that they integrate with their repo github-ldap. But on travis, they use openldap. Travis allows you to install packages and numerous ldap related projects have a script, invoked during the install section of the build lifecycle, that installs openldap.
https://github.com/github/github-ldap/blob/master/.travis.yml and https://github.com/github/github-ldap/blob/master/script/install-openldap

Perhaps that is the best first step?

acurley avatar Mar 19 '15 17:03 acurley

Yeah, I agrees. Good first step. 

— Sent from Mailbox

On Thu, Mar 19, 2015 at 6:59 PM, Andrew Curley [email protected] wrote:

I think your idea is correct. Our friends at Github use ladle for testing apps that they integrate with their repo github-ldap. But on travis, they use openldap. Travis allows you to install packages and numerous ldap related projects have a script, invoked during the install section of the build lifecycle, that installs openldap.
https://github.com/github/github-ldap/blob/master/.travis.yml and https://github.com/github/github-ldap/blob/master/script/install-openldap

Perhaps that is the best first step?

Reply to this email directly or view it on GitHub: https://github.com/cschiewek/devise_ldap_authenticatable/issues/197#issuecomment-83694835

gustaflindqvist avatar Mar 19 '15 18:03 gustaflindqvist

:+1:

cschiewek avatar Mar 19 '15 18:03 cschiewek