806600 - Faster tests, no need to bcrypt in integration tests

Status: RESOLVED FIXED

(Webtools Graveyard :: Elmo, defect)

Description

[Peter Bengtsson [:peterbe]]

With bcrypt as the hasher storing our users' passwords:

    Ran 170 tests in 27.822s

With MD5...:

    Ran 170 tests in 14.478s

I ran some profiling on the test suite and django_sha2's hasher.py is the most guilty one.

We don't need secure passwords in tests. Besides, we don't really care much about passwords because our real users are stored with unusable passwords anyway (using LDAP instead)

Comment 1

[Peter Bengtsson [:peterbe]]

Attachment: works wonderfully

I'm applying this same technique on all my other projects.

Comment 2

[Peter Bengtsson [:peterbe]]

An alternative is to leave in settings/local.py-dist, but commented out, this setting so individual developers who are confident with what they do can switch this on for faster tests. 
30 seconds it's much to wait for for jenkins.

Comment 3

[Axel Hecht [:Pike]]

Comment on attachment 676367 [details] [diff] [review]
works wonderfully

Review of attachment 676367 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, r=me.

Comment 4

[[github robot]]

Commit pushed to develop at https://github.com/mozilla/elmo

https://github.com/mozilla/elmo/commit/c4e24bd7c77c7c236556fb041f1fe954364f51bd
bug 806600 - faster tests with md5 hasher, r=Pike