Commit 7e0e44bf authored by Vincent Pelletier's avatar Vincent Pelletier Committed by Kazuhiko Shiozaki

tests: Make role_category_list argument actually a list

The setter does this already so no change is expected, but these are API
misuses and they just look wrong.
parent abbfa19a
......@@ -1881,7 +1881,7 @@ class TestCRMMailSend(BaseTestCRM):
portal_type_object.newContent(id='manager_role',
portal_type='Role Information',
role_name_list=('Manager',),
role_category_list='function/crm_agent')
role_category_list=('function/crm_agent', ))
portal_type_object.updateRoleMapping()
user = self.createSimpleUser('Agent', 'crm_agent', 'crm_agent')
self.tic()
......
......@@ -115,7 +115,7 @@ CREATE TABLE alternate_roles_and_users (
self.portal.portal_types.Person.newContent(
portal_type='Role Information',
role_name='Unknown',
role_category_list=('group/g1'),
role_category_list=('group/g1', ),
role_base_category='group',
local_role_group_value=self.portal.portal_categories.local_role_group.Alternate)
......
......@@ -166,8 +166,9 @@ class UserManagementTestCase(ERP5TypeTestCase):
)
dummy_type.newContent(
portal_type='Role Information',
role_category_list=self.portal.portal_categories.\
group.dummy_group.getRelativeUrl(),
role_category_list=(
self.portal.portal_categories.group.dummy_group.getRelativeUrl(),
),
role_name_list=('Assignee', )
)
if 'Dummy Module' not in types_tool.objectIds():
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment