Commit a5a9ad20 authored by Jérome Perrin's avatar Jérome Perrin

administration: simplify test by using an organisation

no need to do this magic with user_id if we are using an organisation
parent 42c7a16d
...@@ -83,11 +83,10 @@ class TestERP5Administration(InventoryAPITestCase): ...@@ -83,11 +83,10 @@ class TestERP5Administration(InventoryAPITestCase):
def test_check_consistency_alarm(self): def test_check_consistency_alarm(self):
alarm = self.portal.portal_alarms.check_consistency alarm = self.portal.portal_alarms.check_consistency
# Here we disable user_id so that Person_createUserPreference will not be called inconsistent_document = self.portal.organisation_module.newContent(
# automatically. portal_type='Organisation')
person = self.portal.person_module.newContent(portal_type='Person', user_id=None)
# this document will be non consistent, for PropertyTypeValidity # this document will be non consistent, for PropertyTypeValidity
person.title = 3 inconsistent_document.title = 3
# tic right now to make sure the person is indexed, indeed the alarm # tic right now to make sure the person is indexed, indeed the alarm
# could use catalog to retrieve objects to check # could use catalog to retrieve objects to check
self.tic() self.tic()
...@@ -112,7 +111,7 @@ class TestERP5Administration(InventoryAPITestCase): ...@@ -112,7 +111,7 @@ class TestERP5Administration(InventoryAPITestCase):
# case of PropertyTypeValidity # case of PropertyTypeValidity
alarm.solve() alarm.solve()
self.tic() self.tic()
self.assertEqual('3', person.title) self.assertEqual('3', inconsistent_document.title)
def test_missing_category_document_constraint(self): def test_missing_category_document_constraint(self):
person = self.portal.person_module.newContent(portal_type='Person') person = self.portal.person_module.newContent(portal_type='Person')
......
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