Commit e3ba7214 authored by Rafael Monnerat's avatar Rafael Monnerat

fixup! slapos_cloud: Protect supply against multiple calls on the same request

parent d39f159c
...@@ -20,7 +20,6 @@ ...@@ -20,7 +20,6 @@
############################################################################## ##############################################################################
from erp5.component.test.SlapOSTestCaseMixin import SlapOSTestCaseMixin from erp5.component.test.SlapOSTestCaseMixin import SlapOSTestCaseMixin
import transaction import transaction
from unittest import expectedFailure
from Products.ERP5Type.Errors import UnsupportedWorkflowMethod from Products.ERP5Type.Errors import UnsupportedWorkflowMethod
from cryptography import x509 from cryptography import x509
from cryptography.x509.oid import NameOID from cryptography.x509.oid import NameOID
...@@ -234,6 +233,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflow(SlapOSTestCaseMixin): ...@@ -234,6 +233,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflow(SlapOSTestCaseMixin):
self.compute_node.requestSoftwareRelease(software_release_url=url, self.compute_node.requestSoftwareRelease(software_release_url=url,
state='available') state='available')
self.tic() self.tic()
self.cleanUpRequest()
self.login() self.login()
software_installation = self.compute_node.getAggregateRelatedValue( software_installation = self.compute_node.getAggregateRelatedValue(
portal_type='Software Installation') portal_type='Software Installation')
...@@ -267,7 +267,6 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflow(SlapOSTestCaseMixin): ...@@ -267,7 +267,6 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflow(SlapOSTestCaseMixin):
state='available') state='available')
transaction.abort() transaction.abort()
@expectedFailure
def test_requestSoftwareRelease_same_transaction(self): def test_requestSoftwareRelease_same_transaction(self):
self.person_user = self.makePerson(self.project) self.person_user = self.makePerson(self.project)
self.addProjectProductionManagerAssignment(self.person_user, self.project) self.addProjectProductionManagerAssignment(self.person_user, self.project)
...@@ -686,6 +685,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin): ...@@ -686,6 +685,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin):
software_installation.getReference()) software_installation.getReference())
self.tic() self.tic()
self.cleanUpRequest()
self.compute_node.requestSoftwareRelease(state="available", self.compute_node.requestSoftwareRelease(state="available",
software_release_url=software_release) software_release_url=software_release)
...@@ -718,6 +718,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin): ...@@ -718,6 +718,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin):
software_installation.getReference()) software_installation.getReference())
self.tic() self.tic()
self.cleanUpRequest()
self.compute_node.requestSoftwareRelease(state="destroyed", self.compute_node.requestSoftwareRelease(state="destroyed",
software_release_url=software_release) software_release_url=software_release)
...@@ -759,6 +760,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin): ...@@ -759,6 +760,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin):
software_installation.getReference()) software_installation.getReference())
self.tic() self.tic()
self.cleanUpRequest()
self.compute_node.requestSoftwareRelease(state="destroyed", self.compute_node.requestSoftwareRelease(state="destroyed",
software_release_url=software_release) software_release_url=software_release)
...@@ -806,6 +808,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin): ...@@ -806,6 +808,7 @@ class TestSlapOSCoreComputeNodeSlapInterfaceWorkflowSupply(SlapOSTestCaseMixin):
software_installation.getReference()) software_installation.getReference())
self.tic() self.tic()
self.cleanUpRequest()
self.compute_node.requestSoftwareRelease(state="destroyed", self.compute_node.requestSoftwareRelease(state="destroyed",
software_release_url=software_release) software_release_url=software_release)
......
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