test: use acl_users.getUserById() instead of getUser().

product/ERP5/tests/testBug.py

@@ -111,7 +111,7 @@ class TestBug(ERP5TypeTestCase):
     portal = self.getPortal()
     uf = portal.acl_users
     uf._doAddUser('mame', '', ['Assignor','Assignee'], [])
-    if not uf.getUser('dummy'):
+    if not uf.getUserById('dummy'):
       uf._doAddUser('manager', '', ['Manager'], [])
       self.login('manager')
       person_module = portal.getDefaultModule(self.person_portal_type)

product/ERP5/tests/testQueryModule.py

@@ -99,8 +99,8 @@ class TestQueryModule(ERP5TypeTestCase):
     assignment.validate()
     self.tic()
     uf = self.portal.acl_users
-    owner_user = uf.getUser('owner_user').__of__(uf)
-    question_user = uf.getUser('question_user').__of__(uf)
+    owner_user = uf.getUserById('owner_user').__of__(uf)
+    question_user = uf.getUserById('question_user').__of__(uf)
 
     # add Author local roles on person and query modules
     self.portal.person_module.manage_setLocalRoles('owner_user', ['Author'])

product/ERP5/tests/testTask.py

@@ -127,7 +127,7 @@ class TestTaskMixin:
 #  def stepLogin(self, **kw):
 #    portal = self.getPortal()
 #    uf = portal.acl_users
-#    if not uf.getUser('dummy'):
+#    if not uf.getUserById('dummy'):
 #      uf._doAddUser('manager', '', ['Manager'], [])
 #      self.login('manager')
 #      person_module = portal.getDefaultModule(self.person_portal_type)
@@ -718,7 +718,7 @@ class TestTask(TestTaskMixin, ERP5TypeTestCase):
 
     simulation_tool = self.getPortal().portal_simulation
     uf = self.getPortal().acl_users
-    if not uf.getUser('manager'):
+    if not uf.getUserById('manager'):
       uf._doAddUser('manager', '', ['Manager'], [])
     self.login('manager')
     try:

product/ERP5/tests/testTaskReportDivergence.py

@@ -99,7 +99,7 @@ class TestTaskReportDivergenceMixin(TestTaskMixin, SecurityTestCase):
     simulation_tool = portal.portal_simulation
     rule_tool = portal.portal_rules
     uf = self.getPortal().acl_users
-    if not uf.getUser('manager'):
+    if not uf.getUserById('manager'):
       uf._doAddUser('manager', '', ['Manager'], [])
     self.login('manager')
     simulation_tool.Base_setDefaultSecurity()

product/ERP5/tests/testZODBHistory.py

@@ -73,7 +73,7 @@ class TestZODBHistory(ERP5TypeTestCase):
   def addUser(self, user_name, role=['Member', 'Owner', 'Assignor']):
     """ Create a test user."""
     uf = self.portal.acl_users
-    if not uf.getUser(user_name):
+    if not uf.getUserById(user_name):
       uf._doAddUser(user_name, '', role, [])
 
   def _clearCache(self):

product/ERP5OOo/tests/testDms.py

@@ -2090,7 +2090,7 @@ return 1
     """
     # Create a root level zope user
     root_user_folder = self.getPortalObject().aq_parent.acl_users
-    if not root_user_folder.getUser('zope_user'):
+    if not root_user_folder.getUserById('zope_user'):
       root_user_folder._doAddUser('zope_user', '', ['Manager',], [])
       self.commit()
     # Create document with good content

product/ERP5Type/tests/ERP5TypeTestCase.py

@@ -260,7 +260,7 @@ class ERP5TypeTestCaseMixin(ProcessingNodeTestCase, PortalTestCase):
       '''Creates the default user.'''
       uf = self.portal.acl_users
       # do nothing if the user already exists
-      if not uf.getUser(user_name):
+      if not uf.getUserById(user_name):
         uf._doAddUser(user_name, 'secret', ['Member'], [])
 
     def _setUpDummyMailHost(self):

product/ERP5Type/tests/testDynamicClassGeneration.py

@@ -1366,7 +1366,7 @@ class _TestZodbComponent(SecurityTestCase):
     otherwise it should not be importable at all
     """
     uf = self.portal.acl_users
-    if not uf.getUser('ERP5TypeTestCase_NonDeveloper'):
+    if not uf.getUserById('ERP5TypeTestCase_NonDeveloper'):
       uf._doAddUser('ERP5TypeTestCase_NonDeveloper',
                     '', ['Manager', 'Member', 'Assignee',
                          'Assignor', 'Author', 'Auditor', 'Associate'], [])