Commit d94135bc authored by Rafael Monnerat's avatar Rafael Monnerat

slapos_erp5: Accounting Period requires Assignee to be started

    Accounting Manager should be Assignee (not Assignor) to start Accounting Periods.
parent 66392e21
......@@ -5,7 +5,7 @@
<multi_property id='category'>function/accounting/agent</multi_property>
<multi_property id='base_category'>function</multi_property>
</role>
<role id='Assignor'>
<role id='Assignee'>
<property id='title'>Accountant Manager</property>
<property id='description'>Only the accountant can validate new accounts.</property>
<multi_property id='categories'>local_role_group/function</multi_property>
......
......@@ -129,7 +129,6 @@ class TestAccount(TestSlapOSGroupRoleSecurityMixin):
self.assertRoles(product, 'R-SHADOW-PERSON', ['Auditor'])
self.assertRoles(product, self.user_id, ['Owner'])
class TestAccountingPeriod(TestSlapOSGroupRoleSecurityMixin):
def test_AccountingFunction(self):
product = self.portal.organisation_module.newContent(
......@@ -137,7 +136,7 @@ class TestAccountingPeriod(TestSlapOSGroupRoleSecurityMixin):
portal_type='Accounting Period')
self.assertSecurityGroup(product,
['F-ACCMAN', 'F-ACCAGT', self.user_id], False)
self.assertRoles(product, 'F-ACCMAN', ['Assignor'])
self.assertRoles(product, 'F-ACCMAN', ['Assignee'])
self.assertRoles(product, 'F-ACCAGT', ['Assignee'])
self.assertRoles(product, self.user_id, ['Owner'])
......
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