fix #1437: return pids() in sorted order

HISTORY.rst

@@ -18,6 +18,7 @@ XXXX-XX-XX
 - 1426_: [Windows] PAGESIZE and number of processors is now calculated on
   startup.
 - 1433_: new Process.parents() method.  (idea by Ghislain Le Meur)
+- 1437_: pids() are returned in sorted order.
 
 **Bug fixes**
 

docs/index.rst

@@ -817,13 +817,17 @@ Functions
 
 .. function:: pids()
 
-  Return a list of current running PIDs. To iterate over all processes
-  and avoid race conditions :func:`process_iter()` should be preferred.
+  Return a sorted list of current running PIDs.
+  To iterate over all processes and avoid race conditions :func:`process_iter()`
+  should be preferred.
 
   >>> import psutil
   >>> psutil.pids()
   [1, 2, 3, 5, 7, 8, 9, 10, 11, 12, 13, 14, 15, 17, 18, 19, ..., 32498]
 
+  .. versionchanged::
+    5.6.0 PIDs are returned in sorted order
+
 .. function:: process_iter(attrs=None, ad_value=None)
 
   Return an iterator yielding a :class:`Process` class instance for all running

psutil/__init__.py

@@ -399,7 +399,7 @@ def _pprint_secs(secs):
 
 @lru_cache()
 def _first_pid():
-    return sorted(pids())[0]
+    return pids()[0]
 
 
 # =====================================================================
@@ -1500,7 +1500,7 @@ def wait(self, timeout=None):
 
 def pids():
     """Return a list of current running PIDs."""
-    return _psplatform.pids()
+    return sorted(_psplatform.pids())
 
 
 def pid_exists(pid):

psutil/tests/test_posix.py

@@ -321,10 +321,8 @@ class TestSystemAPIs(unittest.TestCase):
     def test_pids(self):
         # Note: this test might fail if the OS is starting/killing
         # other processes in the meantime
-        pids_ps = ps("pid")
+        pids_ps = sorted(ps("pid"))
         pids_psutil = psutil.pids()
-        pids_ps.sort()
-        pids_psutil.sort()
 
         # on MACOS and OPENBSD ps doesn't show pid 0
         if MACOS or OPENBSD and 0 not in pids_ps:

psutil/tests/test_system.py

@@ -242,11 +242,11 @@ def test_pid_exists_2(self):
             self.assertFalse(psutil.pid_exists(pid), msg=pid)
 
     def test_pids(self):
-        plist = [x.pid for x in psutil.process_iter()]
-        pidlist = psutil.pids()
-        self.assertEqual(plist.sort(), pidlist.sort())
+        pidslist = psutil.pids()
+        procslist = [x.pid for x in psutil.process_iter()]
         # make sure every pid is unique
-        self.assertEqual(len(pidlist), len(set(pidlist)))
+        self.assertEqual(sorted(set(pidslist)), pidslist)
+        self.assertEqual(pidslist, procslist)
 
     def test_test(self):
         # test for psutil.test() function