Add tests to validate the use of the "connect" kwarg in client constructor

Author: alfinkel

tests/unit/client_tests.py

@@ -103,6 +103,44 @@ def test_connect(self):
             self.client.disconnect()
             self.assertIsNone(self.client.r_session)
 
+    def test_auto_connect(self):
+        """
+        Test connect during client instantiation option.
+        """
+        try:
+            self.set_up_client(auto_connect=True)
+            self.assertIsInstance(self.client.r_session, requests.Session)
+            if self.client.admin_party:
+                self.assertIsNone(self.client.r_session.auth)
+            else:
+                self.assertEqual(
+                    self.client.r_session.auth, (self.user, self.pwd)
+                )
+        finally:
+            self.client.disconnect()
+            self.assertIsNone(self.client.r_session)
+
+    def test_multiple_connect(self):
+        """
+        Test that issuing a connect call to an already connected client does
+        not cause any issue.
+        """
+        try:
+            self.client.connect()
+            self.set_up_client(auto_connect=True)
+            self.client.connect()
+            self.assertIsInstance(self.client.r_session, requests.Session)
+            if self.client.admin_party:
+                self.assertIsNone(self.client.r_session.auth)
+            else:
+                self.assertEqual(
+                    self.client.r_session.auth, (self.user, self.pwd)
+                )
+        finally:
+            self.client.disconnect()
+            self.assertIsNone(self.client.r_session)
+
+
     def test_session(self):
         """
         Test getting session information.  

tests/unit/unit_t_db_base.py

@@ -132,6 +132,9 @@ def setUp(self):
         """
         Set up test attributes for unit tests targeting a database
         """
+        self.set_up_client()
+
+    def set_up_client(self, auto_connect=False):
         if os.environ.get('RUN_CLOUDANT_TESTS') is None:
             admin_party = False
             if (os.environ.get('ADMIN_PARTY') and
@@ -140,7 +143,13 @@ def setUp(self):
             self.user = os.environ.get('DB_USER', None)
             self.pwd = os.environ.get('DB_PASSWORD', None)
             self.url = os.environ['DB_URL']
-            self.client = CouchDB(self.user, self.pwd, admin_party, url=self.url)
+            self.client = CouchDB(
+                self.user,
+                self.pwd,
+                admin_party,
+                url=self.url,
+                connect=auto_connect
+            )
         else:
             self.account = os.environ.get('CLOUDANT_ACCOUNT')
             self.user = os.environ.get('DB_USER')
@@ -152,7 +161,10 @@ def setUp(self):
                 self.user,
                 self.pwd,
                 url=self.url,
-                x_cloudant_user=self.account)
+                x_cloudant_user=self.account,
+                connect=auto_connect
+            )
+
 
     def tearDown(self):
         """