-
Notifications
You must be signed in to change notification settings - Fork 4
Closed
Description
Connections opened in test_rollback and test_ExceptionsAsConnectionAttributes are opened yet never explicitly closed, which leads to unittest ResourceWarning messages.
Lines 181 to 198 in c259a6a
| def test_ExceptionsAsConnectionAttributes(self): | |
| # OPTIONAL EXTENSION | |
| # Test for the optional DB API 2.0 extension, where the exceptions | |
| # are exposed as attributes on the Connection object | |
| # I figure this optional extension will be implemented by any | |
| # driver author who is using this test suite, so it is enabled | |
| # by default. | |
| con = self._connect() | |
| drv = self.driver | |
| _failUnless(self,con.Warning is drv.Warning) | |
| _failUnless(self,con.Error is drv.Error) | |
| _failUnless(self,con.InterfaceError is drv.InterfaceError) | |
| _failUnless(self,con.DatabaseError is drv.DatabaseError) | |
| _failUnless(self,con.OperationalError is drv.OperationalError) | |
| _failUnless(self,con.IntegrityError is drv.IntegrityError) | |
| _failUnless(self,con.InternalError is drv.InternalError) | |
| _failUnless(self,con.ProgrammingError is drv.ProgrammingError) | |
| _failUnless(self,con.NotSupportedError is drv.NotSupportedError) |
Lines 209 to 217 in c259a6a
| def test_rollback(self): | |
| con = self._connect() | |
| # If rollback is defined, it should either work or throw | |
| # the documented exception | |
| if hasattr(con,'rollback'): | |
| try: | |
| con.rollback() | |
| except self.driver.NotSupportedError: | |
| pass |
Metadata
Metadata
Assignees
Labels
No labels