Address review feedback

Author: aclark4life

doc/changelog.rst

@@ -8,8 +8,8 @@ PyMongo 4.17 brings a number of changes including:
 
 - ``has_key``, ``iterkeys`` and ``itervalues`` in :class:`bson.son.SON` have
   been deprecated and will be removed in PyMongo 5.0. These methods were
-  deprecated in favor of the standard dictionary methods ``in``, ``keys``, and
-  ``values`` respectively.
+  deprecated in favor of the standard dictionary containment operator ``in``
+  and the ``keys()`` and ``values()`` methods, respectively.
 
 - Added the :meth:`~pymongo.asynchronous.client_session.AsyncClientSession.bind` and :meth:`~pymongo.client_session.ClientSession.bind` methods
   that allow users to bind a session to all database operations within the scope of a context manager instead of having to explicitly pass the session to each individual operation.

test/test_son.py

@@ -145,13 +145,11 @@ def test_iteration(self):
             self.assertEqual(ele * 100, test_son[ele])
 
     def test_contains_has(self):
-        """key in SON and __contains__"""
+        """Test key membership via 'in' and __contains__."""
         test_son = SON([(1, 100), (2, 200), (3, 300)])
         self.assertIn(1, test_son)
         self.assertIn(2, test_son, "in failed")
         self.assertNotIn(22, test_son, "in succeeded when it shouldn't")
-        self.assertTrue(2 in test_son, "'in' operator failed for key 2")
-        self.assertFalse(22 in test_son, "'in' operator succeeded when it shouldn't")
 
     def test_clears(self):
         """Test clear()"""