Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No output inside nose tests #8

Comments

@akaihola
Copy link

When I insert an "import ipdb;ipdb.set_trace()" inside a nose test case, running of the tests pauses but nothing is output on the terminal.

I've made a patch to fix this in my fork of ipdb in February 2010: see akaihola/ipdb@0894324

Unfortunately my repository isn't a fork of yours. IIRC I created it by cloning svn.gotcha.python-hosting.com since there were no GitHub mirrors at that time.

@akaihola
Copy link
Author

I adapted my original patch to your current code and made it work with IPython >=0.10.2 as well:
akaihola/ipdb@bcf1e99

@gotcha
Copy link
Owner

gotcha commented Aug 25, 2011

Can you turn it into a pull request ?

@akaihola
Copy link
Author

Can you turn it into a pull request ?

Unfortunately I can't before GitHub staff changes my repository to be a
fork of yours. They said it will take some time.

I can git format-patch it for you if it helps.

-Antti

@gotcha
Copy link
Owner

gotcha commented Aug 25, 2011

Ok, I'll manage myself then !

@gotcha
Copy link
Owner

gotcha commented Aug 26, 2011

I made a branch where I use less code.

Can you test my 'issue8' branch ?

@akaihola
Copy link
Author

Your 'issue8' branch seems to work correctly (Python 2.7, nose 1.1.2, IPython 0.11).

@akaihola
Copy link
Author

Ah, one problem! If I have set_trace() in a loop, the first time it works ok, but if I hit c and arrive at set_trace() on the next iteration of the loop, the ipdb> prompt is missing (although all other output works correctly).

In my original patch this problem doesn't exist.

@gotcha
Copy link
Owner

gotcha commented Aug 26, 2011

Last push should fix the loop issue. Can you confirm ?

@gotcha
Copy link
Owner

gotcha commented Sep 1, 2011

As I also tested myself, I consider this issue closed.

Released as ipdb 0.6

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