[Piuparts-devel] [piuparts] Rearrange old unittests.py; Use nosetests (#1)

[Holger Levsen]

Hi Matthias,

On Freitag, 10. Januar 2014, Matthias wrote:
> in order to get the tests running (and to get a base for adding more tests)
> i moved the unittests.py to a sub directory "tests/" in which
> nosetests searches for tests. I also added a configuration for nosetests
> that writes a test coverage report (the coverage report is saved in
> build/html).

awesome! very much so!

> * Move unittests.py to tests/ subdirectory
> * Split out dependencyparser test to an extra file
> * Add configuration for nosetests
> * Use nosetests in Makefile target "check"
> You can merge this Pull Request by running:
> 
>   git pull https://github.com/mattthias/piuparts develop_reorganise_tests

can you please add a debian/changelog entry? (based on the develop branch) - 
well, skip that, I've just merged your commit and added a debian/changelog 
entry... :)

Also, when is the check target called? Never during debian builds, right?

Hmm, and how to recommend python-nose? There are not build-recommends:...

Oh, and scroll down for the results of the first test-run:

cheers,
	Holger, leaving the rest quoted for the benefit of 
		piuparts-devel at lists.alioth.debian.org
 
> Or you can view, comment on it, or merge it online at:
> 
>   https://github.com/h01ger/piuparts/pull/1
> 
> -- Commit Summary --
> 
>   * Rearrange old unittests.py; Use nosetests
> 
> -- File Changes --
> 
>     M Makefile (2)
>     A nosetests.cfg (14)
>     A tests/test_dependencyparser.py (33)
>     R tests/unittests.py (32)
> 
> -- Patch Links --
> 
> https://github.com/h01ger/piuparts/pull/1.patch
> https://github.com/h01ger/piuparts/pull/1.diff
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/h01ger/piuparts/pull/1



$ make check
python piuparts.py unittest
..ERROR:root:ELOOP: Too many symbolic links in '/absolute-selfloop'
...ERROR:root:ELOOP: Too many symbolic links in 
'/explode/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb/bomb'
.......ERROR:root:ELOOP: Too many symbolic links in '/selfloop'
.ERROR:root:ELOOP: Too many symbolic links in '/dir/selfloop'
ERROR:root:ELOOP: Too many symbolic links in '/dir/selfloop'
ERROR:root:ELOOP: Too many symbolic links in '/dir/subdir/selfloop'
ERROR:root:ELOOP: Too many symbolic links in '/dir/subdir/selfloop'
ERROR:root:ELOOP: Too many symbolic links in '/dir/subdir/selfloop'
...
----------------------------------------------------------------------
Ran 16 tests in 0.071s

OK
nosetests -c nosetests.cfg
make: nosetests: Kommando nicht gefunden
make: *** [check] Fehler 127
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 828 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.alioth.debian.org/pipermail/piuparts-devel/attachments/20140111/3a7fe3b2/attachment.sig>