Fix: avoiding issue with PytestUnraisableExceptionWarning that is raised because of __del__() calling self.close() in Redis class, as the self.connection attribute was not set due to early failure in the Redis() constructor. Example: calling redis.StrictRedis(**connectionInfo) in a constructor, with connectionInfo={'hog':'cat'}

Author: advance512

redis/client.py

@@ -923,6 +923,11 @@ def __del__(self):
         self.close()
 
     def close(self):
+        # In case a connection property does not yet exist (due to a crash earlier in the Redis() constructor), return
+        # immediately as there is nothing to clean-up.
+        if not hasattr(self, 'connection'):
+            return
+
         conn = self.connection
         if conn:
             self.connection = None