Slow running Devise unit tests #969

wants to merge 1 commit into

2 participants


Found that devise was running slowly through unit tests due to bcrypt stretches. Bernard Schaefer purposed a less than ideal solution ( and I found that modifying streches in the config was sufficient to reduce test cost (

I'm still fairly new to ruby/rails so excuse an error in my solution - as well my quick scan didn't find any unit tests for this case, but I'd gladly write them if someone pointed me to the file required.

Gavin Miller

Plataformatec member

I wouldn't like to ship that code in devise itself. Why? Devise is used a lot by people starting with Rails and that would cause confusion and potential misuse. Thanks for the pull request!

@josevalim josevalim closed this Apr 5, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment