Skip to content

Copy this doc up from the bok_choy tests. - #11519

Merged
jibsheet merged 1 commit into
masterfrom
jibsheet/document-nose-breakpoints-twice
Feb 12, 2016
Merged

Copy this doc up from the bok_choy tests.#11519
jibsheet merged 1 commit into
masterfrom
jibsheet/document-nose-breakpoints-twice

Conversation

@jibsheet

Copy link
Copy Markdown
Contributor

It's just as relevant to unit tests and I wish I had seen it earlier (this is where I originally learned about --pdb which was super helpful for all my test debugging, but I was doing awful things to trigger the debugger from tests, like raise Exception('kevin') since a normal import pdb doesn't work under nose).

@benpatterson @jzoldak

@benpatterson

Copy link
Copy Markdown
Contributor

👍

Comment thread docs/en_us/internal/testing.rst Outdated
and see what the values of the variables are. Check out `the pdb
documentation <http://docs.python.org/library/pdb.html>`__

Use this command to put a debugging breakpoint in a test.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should point out that if someone actually checks in this code, his/her test will hang (i.e., don't check it in).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed up wording changes for this one (and copied them to the second doc of this feature).

@jzoldak

jzoldak commented Feb 12, 2016

Copy link
Copy Markdown
Contributor

👍

It's just as relevant to unit tests and I wish I had seen it earlier.

Stress the transient nature of this debugging step.
@jibsheet
jibsheet force-pushed the jibsheet/document-nose-breakpoints-twice branch from 3cf1d61 to 21b75f8 Compare February 12, 2016 15:27
jibsheet added a commit that referenced this pull request Feb 12, 2016
…-twice

Copy this doc up from the bok_choy tests.
@jibsheet
jibsheet merged commit 4037b16 into master Feb 12, 2016
@jibsheet
jibsheet deleted the jibsheet/document-nose-breakpoints-twice branch February 12, 2016 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants