remove flake8 exclude for W601 and fix that

setup.cfg

@@ -28,7 +28,6 @@ ignore =
     E721
     F841
     E722
-    W601
     F401
 
 

src/Products/Sessions/tests/testSessionDataManager.py

@@ -190,7 +190,10 @@ def testInvalidateSessionDataObject(self):
         sd = sdm.getSessionData()
         sd['test'] = 'Its alive!  Alive!'
         sd.invalidate()
-        self.assertTrue(not sdm.getSessionData().has_key('test'))
+        # getSessionData() object did not implement a ``__contains__`` method
+        # so a new style ```'test' not in sdm.getSessionData() did not work.```
+        self.assertTrue(not sdm.getSessionData().has_key('test'))  # NOQA: W601
+        # self.assertTrue('test' not in sdm.getSessionData())
 
     def testGhostUnghostSessionManager(self):
         import transaction
@@ -250,7 +253,7 @@ def testAutoReqPopulate(self):
         self.app.REQUEST['PARENTS'] = [self.app]
         self.app.REQUEST['URL'] = 'a'
         self.app.REQUEST.traverse('/')
-        self.assertTrue(self.app.REQUEST.has_key('TESTOFSESSION'))
+        self.assertTrue('TESTOFSESSION' in self.app.REQUEST)
 
     def testUnlazifyAutoPopulated(self):
         from Acquisition import aq_base

src/Products/Transience/tests/testTransientObject.py

@@ -109,8 +109,13 @@ def test_dictionaryLike(self):
         with self.assertRaises(AttributeError):
             self._genLenError(t)
         self.assertIsNone(t.get('foobie', None))
-        self.assertTrue(t.has_key('a'))
-        self.assertFalse(t.has_key('foobie'))
+        # Test with .has_key() are necessary:
+        # as t did not have a __contains__ method
+        # normally you would test like that:
+        # >>> self.assertTrue('a' in t)
+        # >>> self.assertFalse('foobie' in t)
+        self.assertTrue(t.has_key('a'))  # NOQA: W601
+        self.assertFalse(t.has_key('foobie'))  # NOQA: W601
         t.clear()
         self.assertEqual(len(t.keys()), 0)