Commit abac54ed authored by Rafael Monnerat's avatar Rafael Monnerat

slapos_cloud: User is only valid after its indexation

  This test lacks the use case of same user transaction
parent de67aec3
...@@ -93,7 +93,7 @@ class TestSlapOSComputerSecurity(TestSlapOSSecurityMixin): ...@@ -93,7 +93,7 @@ class TestSlapOSComputerSecurity(TestSlapOSSecurityMixin):
computer.newContent(portal_type='ERP5 Login', computer.newContent(portal_type='ERP5 Login',
reference=reference).validate() reference=reference).validate()
self.commit() self.tic()
self._assertUserExists(user_id, reference, None) self._assertUserExists(user_id, reference, None)
...@@ -112,7 +112,7 @@ class TestSlapOSComputerSecurity(TestSlapOSSecurityMixin): ...@@ -112,7 +112,7 @@ class TestSlapOSComputerSecurity(TestSlapOSSecurityMixin):
computer.setUserId(user_id) computer.setUserId(user_id)
computer.newContent(portal_type='ERP5 Login', computer.newContent(portal_type='ERP5 Login',
reference=reference) reference=reference)
self.commit() self.tic()
self._assertUserDoesNotExists(user_id, reference, None) self._assertUserDoesNotExists(user_id, reference, None)
...@@ -128,7 +128,7 @@ class TestSlapOSSoftwareInstanceSecurity(TestSlapOSSecurityMixin): ...@@ -128,7 +128,7 @@ class TestSlapOSSoftwareInstanceSecurity(TestSlapOSSecurityMixin):
instance.validate() instance.validate()
instance.newContent(portal_type='ERP5 Login', instance.newContent(portal_type='ERP5 Login',
reference=reference).validate() reference=reference).validate()
self.commit() self.tic()
self._assertUserExists(user_id, reference, None) self._assertUserExists(user_id, reference, None)
...@@ -147,7 +147,7 @@ class TestSlapOSSoftwareInstanceSecurity(TestSlapOSSecurityMixin): ...@@ -147,7 +147,7 @@ class TestSlapOSSoftwareInstanceSecurity(TestSlapOSSecurityMixin):
reference=subscription_reference) reference=subscription_reference)
subscription.validate() subscription.validate()
instance.setSpecialise(subscription.getRelativeUrl()) instance.setSpecialise(subscription.getRelativeUrl())
self.commit() self.tic()
# clear cache in order to reset calculation # clear cache in order to reset calculation
self.portal.portal_caches.clearAllCache() self.portal.portal_caches.clearAllCache()
...@@ -164,7 +164,7 @@ class TestSlapOSSoftwareInstanceSecurity(TestSlapOSSecurityMixin): ...@@ -164,7 +164,7 @@ class TestSlapOSSoftwareInstanceSecurity(TestSlapOSSecurityMixin):
instance = self.portal.getDefaultModule(portal_type=self.portal_type)\ instance = self.portal.getDefaultModule(portal_type=self.portal_type)\
.newContent(portal_type=self.portal_type, reference=reference) .newContent(portal_type=self.portal_type, reference=reference)
instance.setUserId(user_id) instance.setUserId(user_id)
self.commit() self.tic()
self._assertUserDoesNotExists(user_id, reference, None) self._assertUserDoesNotExists(user_id, reference, None)
...@@ -183,7 +183,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin): ...@@ -183,7 +183,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin):
person.newContent(portal_type='ERP5 Login', person.newContent(portal_type='ERP5 Login',
reference=reference, password=password).validate() reference=reference, password=password).validate()
self.commit() self.tic()
self._assertUserExists(user_id, reference, password) self._assertUserExists(user_id, reference, password)
...@@ -196,7 +196,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin): ...@@ -196,7 +196,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin):
# add to group category # add to group category
self.login() self.login()
person.newContent(portal_type='Assignment', group='company').open() person.newContent(portal_type='Assignment', group='company').open()
self.commit() self.tic()
self.tic() self.tic()
self.portal.portal_caches.clearAllCache() self.portal.portal_caches.clearAllCache()
...@@ -208,7 +208,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin): ...@@ -208,7 +208,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin):
# add to role category # add to role category
self.login() self.login()
person.newContent(portal_type='Assignment', role='member').open() person.newContent(portal_type='Assignment', role='member').open()
self.commit() self.tic()
self.portal.portal_caches.clearAllCache() self.portal.portal_caches.clearAllCache()
self.login(person.getUserId()) self.login(person.getUserId())
...@@ -224,7 +224,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin): ...@@ -224,7 +224,7 @@ class TestSlapOSPersonSecurity(TestSlapOSSecurityMixin):
person = self.portal.person_module.newContent(portal_type='Person', person = self.portal.person_module.newContent(portal_type='Person',
reference=reference, password=password) reference=reference, password=password)
self.commit() self.tic()
self._assertUserDoesNotExists(user_id, reference, password) self._assertUserDoesNotExists(user_id, reference, password)
......
...@@ -45,7 +45,7 @@ class TestSlapOSShadowPerson(TestSlapOSSecurityMixin): ...@@ -45,7 +45,7 @@ class TestSlapOSShadowPerson(TestSlapOSSecurityMixin):
person.newContent(portal_type='ERP5 Login', person.newContent(portal_type='ERP5 Login',
reference=reference, password=password).validate() reference=reference, password=password).validate()
self.commit() self.tic()
self._assertUserExists(user_id, reference, password) self._assertUserExists(user_id, reference, password)
...@@ -68,7 +68,7 @@ class TestSlapOSShadowPerson(TestSlapOSSecurityMixin): ...@@ -68,7 +68,7 @@ class TestSlapOSShadowPerson(TestSlapOSSecurityMixin):
portal_type='Person') portal_type='Person')
person.setUserId(user_id) person.setUserId(user_id)
self.commit() self.tic()
self._assertUserDoesNotExists(user_id, reference, password) self._assertUserDoesNotExists(user_id, reference, password)
self._assertUserDoesNotExists(shadow_user_id, reference, None) self._assertUserDoesNotExists(shadow_user_id, reference, None)
...@@ -88,7 +88,7 @@ class TestSlapOSShadowComputer(TestSlapOSSecurityMixin): ...@@ -88,7 +88,7 @@ class TestSlapOSShadowComputer(TestSlapOSSecurityMixin):
reference=reference).validate() reference=reference).validate()
computer.validate() computer.validate()
self.commit() self.tic()
self._assertUserExists(user_id, reference, None) self._assertUserExists(user_id, reference, None)
self._assertUserExists(shadow_user_id, reference, None) self._assertUserExists(shadow_user_id, reference, None)
...@@ -109,7 +109,7 @@ class TestSlapOSShadowComputer(TestSlapOSSecurityMixin): ...@@ -109,7 +109,7 @@ class TestSlapOSShadowComputer(TestSlapOSSecurityMixin):
reference=reference) reference=reference)
computer.setUserId(user_id) computer.setUserId(user_id)
self.commit() self.tic()
self._assertUserDoesNotExists(user_id, reference, None) self._assertUserDoesNotExists(user_id, reference, None)
self._assertUserDoesNotExists(user_id, shadow_reference, None) self._assertUserDoesNotExists(user_id, shadow_reference, None)
...@@ -128,7 +128,7 @@ class TestSlapOSShadowSoftwareInstance(TestSlapOSSecurityMixin): ...@@ -128,7 +128,7 @@ class TestSlapOSShadowSoftwareInstance(TestSlapOSSecurityMixin):
instance.newContent(portal_type='ERP5 Login', instance.newContent(portal_type='ERP5 Login',
reference=reference).validate() reference=reference).validate()
instance.validate() instance.validate()
self.commit() self.tic()
self._assertUserExists(user_id, reference, None) self._assertUserExists(user_id, reference, None)
self._assertUserExists(shadow_user_id, reference, None) self._assertUserExists(shadow_user_id, reference, None)
...@@ -148,7 +148,7 @@ class TestSlapOSShadowSoftwareInstance(TestSlapOSSecurityMixin): ...@@ -148,7 +148,7 @@ class TestSlapOSShadowSoftwareInstance(TestSlapOSSecurityMixin):
instance = self.portal.getDefaultModule(portal_type=self.portal_type)\ instance = self.portal.getDefaultModule(portal_type=self.portal_type)\
.newContent(portal_type=self.portal_type, reference=reference) .newContent(portal_type=self.portal_type, reference=reference)
instance.setUserId(user_id) instance.setUserId(user_id)
self.commit() self.tic()
self._assertUserDoesNotExists(user_id, reference, None) self._assertUserDoesNotExists(user_id, reference, None)
self._assertUserDoesNotExists(user_id, shadow_reference, None) self._assertUserDoesNotExists(user_id, shadow_reference, None)
......
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