Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Document runtests extra argv #7866

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

Merged
merged 4 commits into from
Jul 25, 2016

Conversation

ssanderson
Copy link

This is useful for things like passing --pdb to make nose drop into
a pdb post-mortem on exception.

I was looking for a way to do this while working on #7856 and ended up finding this by digging through the source.

@ssanderson
Copy link
Author

The AppVeyor failure here looks unrelated to the changes here:

Error: HTTPError: 500 Server Error: Internal Server Error for url: https://repo.continuum.io/pkgs/free/win-32/pytz-2016.6.1-py27_0.tar.bz2: https://repo.continuum.io/pkgs/free/win-32/pytz-2016.6.1-py27_0.tar.bz2
Command exited with code 1

Scott Sanderson added 2 commits July 23, 2016 16:35
This is useful for things like passing ``--pdb`` to make nose drop into
a pdb post-mortem on exception.
@ssanderson ssanderson force-pushed the document-runtests-extra-argv branch from 6d0431b to b312137 Compare July 23, 2016 20:35
the extra arguments after a bare ``--``. For example, to run a test method with
the ``--pdb`` flag forwarded to nose, run the following::

$ python runtests.py -t numpy/core/tests/test_ufunc.py:TestUfunc.test_sum -- --pdb
Copy link
Member

Choose a reason for hiding this comment

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

Can you find an example where the line length is < 80?

Copy link
Member

Choose a reason for hiding this comment

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

Maybe numpy/tests/test_scripts.py:test_f2py

@charris
Copy link
Member

charris commented Jul 25, 2016

The test errors look unrelated.

@ssanderson
Copy link
Author

@charris updated to use a shorter example and a more consistent variable name for the arguments forwarded to nose.

@charris
Copy link
Member

charris commented Jul 25, 2016

Travis is recovering, https://www.traviscistatus.com/incidents/mjyq8dprd2zx, but slowly. This looks good, so merging. Thanks @ssanderson .

@charris charris merged commit 4fb9a22 into numpy:master Jul 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants