More win32 fixes and refinements to test suite.

IPython/utils/platutils.py

@@ -79,7 +79,7 b' def find_cmd(cmd):'
         return sys.executable
     try:
         path = _platutils.find_cmd(cmd)
-    except:
+    except OSError:
         raise FindCmdError('command could not be found: %s' % cmd)
     # which returns empty if not found
     if path == '':

IPython/utils/tests/test_platutils.py

@@ -1,4 +1,3 b''
-#!/usr/bin/env python
 # encoding: utf-8
 """
 Tests for platutils.py
@@ -31,29 +30,43 b' def test_find_cmd_python():'
     """Make sure we find sys.exectable for python."""
     nt.assert_equals(find_cmd('python'), sys.executable)
 
+    
 @dec.skip_win32
-def test_find_cmd():
+def test_find_cmd_ls():
     """Make sure we can find the full path to ls."""
     path = find_cmd('ls')
     nt.assert_true(path.endswith('ls'))
 
[e-mail]_if_not_win32
-def test_find_cmd():
+    
+def has_pywin32():
+    try:
+        import win32api
+    except ImportError:
+        return False
+    return True
+
+
[e-mail](has_pywin32, "This test requires win32api to run")
+def test_find_cmd_pythonw():
     """Try to find pythonw on Windows."""
     path = find_cmd('pythonw')
     nt.assert_true(path.endswith('pythonw.exe'))
 
+
[e-mail](lambda : sys.platform != 'win32' or has_pywin32(),
+            "This test runs on posix or in win32 with win32api installed")
 def test_find_cmd_fail():
     """Make sure that FindCmdError is raised if we can't find the cmd."""
     nt.assert_raises(FindCmdError,find_cmd,'asdfasdf')
 
+    
 @dec.skip_if_not_win32
 def test_get_long_path_name_win32():
     p = get_long_path_name('c:\\docume~1')
     nt.assert_equals(p,u'c:\\Documents and Settings') 
    
+    
 @dec.skip_win32
 def test_get_long_path_name():
     p = get_long_path_name('/usr/local')
     nt.assert_equals(p,'/usr/local')
-