Add missing some cursor close and rollback in unittests
This commit is contained in:
1 parent
00677cc383
commit
c185cc309d
2 files changed
+13
No files matched your search
@@ -540,6 +540,9 @@ class FieldsTestCase(unittest.TestCase):
|
||||
CONTEXT)
|
||||
self.assert_(float5['float'] == 0)
|
||||
|
||||
cursor.rollback()
|
||||
cursor.close()
|
||||
|
||||
def test0040numeric(self):
|
||||
'''
|
||||
Test Numeric.
|
||||
@@ -950,6 +953,7 @@ class FieldsTestCase(unittest.TestCase):
|
||||
|
||||
self.failUnlessRaises(Exception, self.char_required.create, cursor,
|
||||
USER, {}, CONTEXT)
|
||||
cursor.rollback()
|
||||
|
||||
char5_id = self.char_required.create(cursor, USER, {
|
||||
'char': 'Test',
|
||||
@@ -969,6 +973,7 @@ class FieldsTestCase(unittest.TestCase):
|
||||
char6_id, {
|
||||
'char': 'foobar',
|
||||
}, CONTEXT)
|
||||
cursor.rollback()
|
||||
|
||||
char7_id = self.char.create(cursor, USER, {
|
||||
'char': u'é',
|
||||
@@ -1211,6 +1216,7 @@ class FieldsTestCase(unittest.TestCase):
|
||||
|
||||
self.failUnlessRaises(Exception, self.text_required.create, cursor,
|
||||
USER, {}, CONTEXT)
|
||||
cursor.rollback()
|
||||
|
||||
text5_id = self.text_required.create(cursor, USER, {
|
||||
'text': 'Test',
|
||||
@@ -1476,6 +1482,7 @@ class FieldsTestCase(unittest.TestCase):
|
||||
|
||||
self.failUnlessRaises(Exception, self.sha_required.create, cursor,
|
||||
USER, {}, CONTEXT)
|
||||
cursor.rollback()
|
||||
|
||||
sha5_id = self.sha_required.create(cursor, USER, {
|
||||
'sha': 'Test',
|
||||
@@ -1753,6 +1760,7 @@ class FieldsTestCase(unittest.TestCase):
|
||||
|
||||
self.failUnlessRaises(Exception, self.date_required.create, cursor,
|
||||
USER, {}, CONTEXT)
|
||||
cursor.rollback()
|
||||
|
||||
date6_id = self.date_required.create(cursor, USER, {
|
||||
'date': today,
|
||||
@@ -2026,6 +2034,7 @@ class FieldsTestCase(unittest.TestCase):
|
||||
|
||||
self.failUnlessRaises(Exception, self.datetime_required.create, cursor,
|
||||
USER, {}, CONTEXT)
|
||||
cursor.rollback()
|
||||
|
||||
datetime6_id = self.datetime_required.create(cursor, USER, {
|
||||
'datetime': today,
|
||||
|
||||
@@ -105,6 +105,7 @@ def install_module(name):
|
||||
cursor = database.cursor()
|
||||
if DB_NAME not in database.list(cursor):
|
||||
create(DB_NAME, 'admin', 'en_US', CONFIG['admin_passwd'])
|
||||
cursor.commit()
|
||||
cursor.close()
|
||||
cursor = DB.cursor()
|
||||
module_obj = POOL.get('ir.module.module')
|
||||
@@ -114,6 +115,7 @@ def install_module(name):
|
||||
])
|
||||
|
||||
if not module_ids:
|
||||
cursor.close()
|
||||
return
|
||||
|
||||
module_obj.button_install(cursor, USER, module_ids, CONTEXT)
|
||||
@@ -152,6 +154,8 @@ def test_view(module_name):
|
||||
field = element.get(attr)
|
||||
if field:
|
||||
assert field in res['fields']
|
||||
cursor.rollback()
|
||||
cursor.close()
|
||||
|
||||
def suite():
|
||||
'''
|
||||
|
||||
Reference in new issue
Block a user