-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Fix error in test suite with ip.system() #1895
Conversation
Test results for commit 789aae6 merged into master
Not available for testing: python2.6 |
@@ -2210,7 +2210,7 @@ def system_raw(self, cmd): | |||
# use piped system by default, because it is better behaved | |||
system = system_piped | |||
|
|||
def getoutput(self, cmd, split=True): | |||
def getoutput(self, cmd, split=True, _depth=0): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would expose depth
without the underscore and document it publicly, it can come in handy for external users. And since this is an official method anyway, might as well.
Thanks for this! Other than my minor comment above, this is good to go. |
Test results for commit 34f8b22 merged into master
Not available for testing: python2.6 |
Sweet, thanks! Merging now... |
Fix error in test suite with ip.system()
Fix error in test suite with ip.system()
As discussed in #1878. Another case of looking the wrong depth through the stack.
There's still some other problem with the doctesting machinery -
a
should be turning up in the user namespace in this test, but it isn't. I've tried to understand the code to run doctests before, though, and it gives me a headache.