try to prevent occasional failure on windows

psutil/__init__.py

@@ -245,13 +245,12 @@ def __init__(self, pid, name=None, ppid=None, msg=None):
         self.name = name
         self.msg = msg
         if msg is None:
-            if name and ppid:
-                details = "(pid=%s, name=%s, ppid=%s)" % (
-                    self.pid, repr(self.name), self.ppid)
-            elif name:
-                details = "(pid=%s, name=%s)" % (self.pid, repr(self.name))
-            else:
-                details = "(pid=%s)" % self.pid
+            args = ["pid=%s" % pid]
+            if name:
+                args.append("name=%s" % repr(self.name))
+            if ppid:
+                args.append("ppid=%s" % self.ppid)
+            details = "(%s)" % ", ".join(args)
             self.msg = "process still exists but it's a zombie " + details
 
 

test/test_psutil.py

@@ -1795,11 +1795,12 @@ def test_prog_w_funky_name(self):
         sproc = get_test_subprocess(cmdline)
         p = psutil.Process(sproc.pid)
         # ...in order to try to prevent occasional failures on travis
-        wait_for_pid(p.pid)
-        normcase = os.path.normcase
-        self.assertEqual(p.name(), os.path.basename(funky_path))
-        self.assertEqual(normcase(p.exe()), normcase(funky_path))
+        if TRAVIS:
+            wait_for_pid(p.pid)
         self.assertEqual(p.cmdline(), cmdline)
+        self.assertEqual(p.name(), os.path.basename(funky_path))
+        self.assertEqual(os.path.normcase(p.exe()),
+                         os.path.normcase(funky_path))
 
     @unittest.skipUnless(POSIX, 'posix only')
     def test_uids(self):