search

fedora-infra / fas

Fedora Account System
https://admin.fedoraproject.org/accounts
GNU General Public License v2.0
40 stars 50 forks source link

Fas 3.0 unittests #192

Closed skrzepto closed 7 years ago

skrzepto commented 8 years ago

Adding unittests to fas3

It's at about 56% coverage.

How to execute unittests

py.test --cov=fas tests/ --cov-report html:cov_html
skrzepto commented 8 years ago

looks like after rebasing to most recent versionm of FAS_3.0 my unittests are now failing. Need to fix them now

skrzepto commented 8 years ago

Hmm it seems that line 295 https://github.com/skrzepto/fas/blob/115507f3748a70313d4c7dbac48f8ae0f1930268/fas/security.py#L295 causes failure. When I commented that line out, more tests pass.

skrzepto commented 8 years ago

After #193 gets merged in. Login works fine and I need to update the tests to reflect the changes in that branch.

skrzepto commented 7 years ago

Closing this since this needs to be modified for the recent changes in FAS_3 branch