update codereview 1961002
This commit is contained in:
parent
c7ef784609
commit
933f29f4cf
|
@ -1,4 +1,4 @@
|
||||||
Index: trytond/trytond/tests/test_transaction.py
|
Index: trytond/tests/test_transaction.py
|
||||||
===================================================================
|
===================================================================
|
||||||
|
|
||||||
--- a/trytond/trytond/tests/test_transaction.py
|
--- a/trytond/trytond/tests/test_transaction.py
|
||||||
|
@ -6,7 +6,7 @@ Index: trytond/trytond/tests/test_transaction.py
|
||||||
@@ -56,6 +56,41 @@
|
@@ -56,6 +56,41 @@
|
||||||
Exception, manipulate_cursor, DB_NAME, USER, context=CONTEXT)
|
Exception, manipulate_cursor, DB_NAME, USER, context=CONTEXT)
|
||||||
self.assertTrue(empty_transaction(DB_NAME, USER, context=CONTEXT))
|
self.assertTrue(empty_transaction(DB_NAME, USER, context=CONTEXT))
|
||||||
|
|
||||||
+ def test0030set_user(self):
|
+ def test0030set_user(self):
|
||||||
+ '''
|
+ '''
|
||||||
+ Test set_user
|
+ Test set_user
|
||||||
|
@ -42,18 +42,18 @@ Index: trytond/trytond/tests/test_transaction.py
|
||||||
+ with Transaction().set_user(2):
|
+ with Transaction().set_user(2):
|
||||||
+ self.assertEqual(transaction.user, 2)
|
+ self.assertEqual(transaction.user, 2)
|
||||||
+
|
+
|
||||||
|
|
||||||
def suite():
|
def suite():
|
||||||
return unittest.TestLoader().loadTestsFromTestCase(TransactionTestCase)
|
return unittest.TestLoader().loadTestsFromTestCase(TransactionTestCase)
|
||||||
|
|
||||||
Index: trytond/trytond/transaction.py
|
Index: trytond/transaction.py
|
||||||
===================================================================
|
===================================================================
|
||||||
|
|
||||||
--- a/trytond/trytond/transaction.py
|
--- a/trytond/trytond/transaction.py
|
||||||
+++ b/trytond/trytond/transaction.py
|
+++ b/trytond/trytond/transaction.py
|
||||||
@@ -118,10 +118,16 @@
|
@@ -118,10 +118,16 @@
|
||||||
return manager
|
return manager
|
||||||
|
|
||||||
def set_user(self, user, set_context=False):
|
def set_user(self, user, set_context=False):
|
||||||
+ if user != 0 and set_context:
|
+ if user != 0 and set_context:
|
||||||
+ raise ValueError('set_context only allowed for root')
|
+ raise ValueError('set_context only allowed for root')
|
||||||
|
@ -69,5 +69,5 @@ Index: trytond/trytond/transaction.py
|
||||||
+ self.context.pop('user', None)
|
+ self.context.pop('user', None)
|
||||||
self.user = user
|
self.user = user
|
||||||
return manager
|
return manager
|
||||||
|
|
||||||
|
|
4
series
4
series
|
@ -1,7 +1,7 @@
|
||||||
issue971002_217001.diff
|
issue971002_217001.diff
|
||||||
issue2341002_1_10001.diff
|
#issue2341002_1_10001.diff
|
||||||
issue2311002_40001.diff
|
issue2311002_40001.diff
|
||||||
issue2321002_1.diff
|
issue2321002_1.diff
|
||||||
issue1961002_40001.diff
|
|
||||||
issue10739.diff
|
issue10739.diff
|
||||||
issue10467.diff
|
issue10467.diff
|
||||||
|
issue1961002_60001.diff
|
||||||
|
|
Loading…
Reference in New Issue