Merge lp:~lmc/repoze.who.plugins.ldap/enable-nosetests into lp:repoze.who.plugins.ldap

Proposed by Lorenzo M. Catucci
Status: Needs review
Proposed branch: lp:~lmc/repoze.who.plugins.ldap/enable-nosetests
Merge into: lp:repoze.who.plugins.ldap
Diff against target: 10 lines (+4/-0)
1 file modified
setup.cfg (+4/-0)
To merge this branch: bzr merge lp:~lmc/repoze.who.plugins.ldap/enable-nosetests
Reviewer Review Type Date Requested Status
Gustavo Narea Pending
Review via email: mp+37464@code.launchpad.net

Description of the change

By enabling nosetests we get a two-fold improvement: first, nosetests' output is much cleaner than standard python unittest; second, we gain free support for coverage.py.
At revision 79, by simply calling setup.py with a --with-cover extra option:
  $ python setup.py nosetests --with-coverage
we will get the following output:
running nosetests
running egg_info
[...]
running build_ext
.....................................
Name Stmts Exec Cover Missing
---------------------------------------------------------------
repoze.who.plugins.ldap 3 3 100%
repoze.who.plugins.ldap.plugins 130 99 76% 84, 96, 111, 130-132, 147, 210-213, 271-274, 277, 302-305, 318, 322, 384-387, 408, 418-421, 424-426
---------------------------------------------------------------
TOTAL 133 102 76%
----------------------------------------------------------------------
Ran 37 tests in 0.096s

OK

To post a comment you must log in.

Unmerged revisions

80. By Lorenzo M. Catucci <lorenzo@sancho>

Add proper configuration for nosetests usage.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'setup.cfg'
2--- setup.cfg 2008-09-09 13:18:01 +0000
3+++ setup.cfg 2010-10-04 13:02:40 +0000
4@@ -1,2 +1,6 @@
5 [egg_info]
6 #tag_build = dev
7+[nosetests]
8+where = repoze/who/plugins/ldap
9+exclude = ^(unimplemented|error)_
10+cover-package = repoze.who.plugins.ldap

Subscribers

People subscribed via source and target branches