Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Give tests copy-pastable identifiers in nose testing #307

Merged
merged 1 commit into from Apr 1, 2013

Conversation

Projects
None yet
3 participants
Contributor

ifreecarve commented Mar 30, 2013

This patch fixes an annoyance.

Currently, failing nosetests are printed like this

======================================================================
FAIL: test_the_name (ion.services.sa.test.test_nosetest_namimg.TestNaming)
----------------------------------------------------------------------

This patch prints them like this:

======================================================================
FAIL: test_the_name ( ion.services.sa.test.test_nosetest_namimg:TestNaming.test_the_name )
----------------------------------------------------------------------

Unlike the current format, "ion.services.sa.test.test_nosetest_namimg:TestNaming.test_the_name" can be pasted directly into the nosetests command line.

Contributor

daf commented Apr 1, 2013

YESSSSSSSSSSSSSSSSSSS

@daf daf added a commit that referenced this pull request Apr 1, 2013

@daf daf Merge pull request #307 from ifreecarve/2013-03-30_copy_pastable_names
Give tests copy-pastable identifiers in nose testing
43b4a8a

@daf daf merged commit 43b4a8a into ooici:master Apr 1, 2013

Member

lukecampbell commented Apr 1, 2013

Yes! Please merge.... err thanks!

Contributor

ifreecarve commented Apr 1, 2013

I'm hesitant to bump the version in coi-services though, I noticed there are a few commits between this patch and what we're currently using.

@ifreecarve ifreecarve deleted the unknown repository branch Apr 1, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment