fix test_find_cmd_python

minrk · minrk · commit a1492cd06714 · 2013-05-20T10:49:56.000-07:00
It's a bit unclear what should be done here.
'python' is short-circuited in find_cmd to sys.executable.
This commit does the same short-circuit with basename(sys.executable), which can be any of Python, python, python3, python3.exe, etc.

An alternate fix would be to change the test back to testing what the code actually did.
diff --git a/IPython/utils/process.py b/IPython/utils/process.py
@@ -59,7 +59,7 @@ def find_cmd(cmd):
     cmd : str
         The command line program to look for.
     """
-    if cmd == 'python':
+    if cmd in ('python', os.path.basename(sys.executable)):
         return os.path.abspath(sys.executable)
     try:
         path = _find_cmd(cmd).rstrip()
