Commit 435601a6 authored by Łukasz Nowak's avatar Łukasz Nowak

pep8ized.

parent 4dced9de
# Copyright (c) 2002-2012 Nexedi SA and Contributors. All Rights Reserved. # Copyright (c) 2002-2012 Nexedi SA and Contributors. All Rights Reserved.
from Products.SlapOS.tests.testSlapOSMixin import \ from Products.SlapOS.tests.testSlapOSMixin import \
testSlapOSMixin testSlapOSMixin
import transaction import transaction
from Products.ERP5Type.tests.backportUnittest import expectedFailure from Products.ERP5Type.tests.backportUnittest import expectedFailure
class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
def generateNewId(self): def generateNewId(self):
return self.getPortalObject().portal_ids.generateNewId( return self.getPortalObject().portal_ids.generateNewId(
id_group=('slapos_core_test')) id_group=('slapos_core_test'))
def generateNewSoftwareReleaseUrl(self): def generateNewSoftwareReleaseUrl(self):
return 'http://example.org/test%s.cfg' % self.generateNewId() return 'http://example.org/test%s.cfg' % self.generateNewId()
...@@ -42,7 +43,6 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -42,7 +43,6 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
self.software_instance = portal.software_instance_module\ self.software_instance = portal.software_instance_module\
.template_software_instance.Base_createCloneDocument(batch_mode=1) .template_software_instance.Base_createCloneDocument(batch_mode=1)
hosting_subscription.edit( hosting_subscription.edit(
title=self.request_kw['software_title'], title=self.request_kw['software_title'],
reference="TESTHS-%s" % new_id, reference="TESTHS-%s" % new_id,
...@@ -90,37 +90,37 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -90,37 +90,37 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
request_kw = good_request_kw.copy() request_kw = good_request_kw.copy()
request_kw.pop('software_release') request_kw.pop('software_release')
self.assertRaises(KeyError, self.software_instance.requestInstance, self.assertRaises(KeyError, self.software_instance.requestInstance,
**request_kw) **request_kw)
request_kw = good_request_kw.copy() request_kw = good_request_kw.copy()
request_kw.pop('software_title') request_kw.pop('software_title')
self.assertRaises(KeyError, self.software_instance.requestInstance, self.assertRaises(KeyError, self.software_instance.requestInstance,
**request_kw) **request_kw)
request_kw = good_request_kw.copy() request_kw = good_request_kw.copy()
request_kw.pop('software_type') request_kw.pop('software_type')
self.assertRaises(KeyError, self.software_instance.requestInstance, self.assertRaises(KeyError, self.software_instance.requestInstance,
**request_kw) **request_kw)
request_kw = good_request_kw.copy() request_kw = good_request_kw.copy()
request_kw.pop('instance_xml') request_kw.pop('instance_xml')
self.assertRaises(KeyError, self.software_instance.requestInstance, self.assertRaises(KeyError, self.software_instance.requestInstance,
**request_kw) **request_kw)
request_kw = good_request_kw.copy() request_kw = good_request_kw.copy()
request_kw.pop('sla_xml') request_kw.pop('sla_xml')
self.assertRaises(KeyError, self.software_instance.requestInstance, self.assertRaises(KeyError, self.software_instance.requestInstance,
**request_kw) **request_kw)
request_kw = good_request_kw.copy() request_kw = good_request_kw.copy()
request_kw.pop('shared') request_kw.pop('shared')
self.assertRaises(KeyError, self.software_instance.requestInstance, self.assertRaises(KeyError, self.software_instance.requestInstance,
**request_kw) **request_kw)
request_kw = good_request_kw.copy() request_kw = good_request_kw.copy()
request_kw.pop('state') request_kw.pop('state')
self.assertRaises(KeyError, self.software_instance.requestInstance, self.assertRaises(KeyError, self.software_instance.requestInstance,
**request_kw) **request_kw)
def test_request_createdInstance(self): def test_request_createdInstance(self):
request_kw = self.request_kw.copy() request_kw = self.request_kw.copy()
...@@ -135,26 +135,26 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -135,26 +135,26 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
self.assertNotEqual(None, requested_instance) self.assertNotEqual(None, requested_instance)
self.assertEqual(request_kw['software_title'], self.assertEqual(request_kw['software_title'],
requested_instance.getTitle()) requested_instance.getTitle())
self.assertEqual('Software Instance', self.assertEqual('Software Instance',
requested_instance.getPortalType()) requested_instance.getPortalType())
self.assertEqual('validated', self.assertEqual('validated',
requested_instance.getValidationState()) requested_instance.getValidationState())
self.assertEqual('start_requested', self.assertEqual('start_requested',
requested_instance.getSlapState()) requested_instance.getSlapState())
self.assertEqual(request_kw['software_release'], self.assertEqual(request_kw['software_release'],
requested_instance.getRootSoftwareReleaseUrl()) requested_instance.getRootSoftwareReleaseUrl())
self.assertEqual(request_kw['instance_xml'], self.assertEqual(request_kw['instance_xml'],
requested_instance.getTextContent()) requested_instance.getTextContent())
self.assertEqual(request_kw['sla_xml'], self.assertEqual(request_kw['sla_xml'],
requested_instance.getSlaXml()) requested_instance.getSlaXml())
self.assertEqual(request_kw['software_type'], self.assertEqual(request_kw['software_type'],
requested_instance.getSourceReference()) requested_instance.getSourceReference())
def test_request_sameTitle(self): def test_request_sameTitle(self):
# check that correct request does not raise # check that correct request does not raise
self.assertRaises(ValueError, self.software_instance.requestInstance, self.assertRaises(ValueError, self.software_instance.requestInstance,
**self.request_kw) **self.request_kw)
def test_request_shared_True(self): def test_request_shared_True(self):
request_kw = self.request_kw.copy() request_kw = self.request_kw.copy()
...@@ -170,21 +170,21 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -170,21 +170,21 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
self.assertNotEqual(None, requested_instance) self.assertNotEqual(None, requested_instance)
self.assertEqual(request_kw['software_title'], self.assertEqual(request_kw['software_title'],
requested_instance.getTitle()) requested_instance.getTitle())
self.assertEqual('Slave Instance', self.assertEqual('Slave Instance',
requested_instance.getPortalType()) requested_instance.getPortalType())
self.assertEqual('validated', self.assertEqual('validated',
requested_instance.getValidationState()) requested_instance.getValidationState())
self.assertEqual('start_requested', self.assertEqual('start_requested',
requested_instance.getSlapState()) requested_instance.getSlapState())
self.assertEqual(request_kw['software_release'], self.assertEqual(request_kw['software_release'],
requested_instance.getRootSoftwareReleaseUrl()) requested_instance.getRootSoftwareReleaseUrl())
self.assertEqual(request_kw['instance_xml'], self.assertEqual(request_kw['instance_xml'],
requested_instance.getTextContent()) requested_instance.getTextContent())
self.assertEqual(request_kw['sla_xml'], self.assertEqual(request_kw['sla_xml'],
requested_instance.getSlaXml()) requested_instance.getSlaXml())
self.assertEqual(request_kw['software_type'], self.assertEqual(request_kw['software_type'],
requested_instance.getSourceReference()) requested_instance.getSourceReference())
def test_request_shared_unsupported(self): def test_request_shared_unsupported(self):
request_kw = self.request_kw.copy() request_kw = self.request_kw.copy()
...@@ -193,7 +193,7 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -193,7 +193,7 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
request_kw['shared'] = 'True' request_kw['shared'] = 'True'
self.assertRaises(ValueError, self.software_instance.requestInstance, self.assertRaises(ValueError, self.software_instance.requestInstance,
**request_kw) **request_kw)
def test_request_unindexed(self): def test_request_unindexed(self):
request_kw = self.request_kw.copy() request_kw = self.request_kw.copy()
...@@ -305,14 +305,14 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -305,14 +305,14 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
duplicate2.validate() duplicate2.validate()
self.portal.portal_workflow._jumpToStateFor(duplicate2, 'start_requested') self.portal.portal_workflow._jumpToStateFor(duplicate2, 'start_requested')
self.software_instance.getSpecialiseValue(\ self.software_instance.getSpecialiseValue(
portal_type='Hosting Subscription').edit( portal_type='Hosting Subscription').edit(
predecessor_list=[ predecessor_list=[
duplicate.getRelativeUrl(), duplicate.getRelativeUrl(),
duplicate2.getRelativeUrl(), duplicate2.getRelativeUrl(),
self.software_instance.getRelativeUrl() self.software_instance.getRelativeUrl()
] ]
) )
self.tic() self.tic()
self.assertRaises(ValueError, self.software_instance.requestInstance, self.assertRaises(ValueError, self.software_instance.requestInstance,
...@@ -355,22 +355,23 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -355,22 +355,23 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
requested_instance2.getRelativeUrl()) requested_instance2.getRelativeUrl())
self.assertSameSet( self.assertSameSet(
self.software_instance.getPredecessorList(), self.software_instance.getPredecessorList(),
[requested_instance.getRelativeUrl(), requested_instance2.getRelativeUrl()]) [requested_instance.getRelativeUrl(),
requested_instance2.getRelativeUrl()])
def test_request_tree_change_indexed(self): def test_request_tree_change_indexed(self):
"""Checks tree change forced by request """Checks tree change forced by request
For a tree like: For a tree like:
A A
| |
A A
|\ |\
B C B C
When B requests C tree shall change to: When B requests C tree shall change to:
A A
| |
A A
...@@ -389,8 +390,8 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -389,8 +390,8 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
C_instance = self.software_instance.REQUEST.get('request_instance') C_instance = self.software_instance.REQUEST.get('request_instance')
self.assertSameSet( self.assertSameSet(
self.software_instance.getPredecessorList(), self.software_instance.getPredecessorList(),
[B_instance.getRelativeUrl(), C_instance.getRelativeUrl()]) [B_instance.getRelativeUrl(), C_instance.getRelativeUrl()])
self.tic() # in order to recalculate tree self.tic() # in order to recalculate tree
...@@ -406,15 +407,15 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -406,15 +407,15 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
def test_request_tree_change_not_indexed(self): def test_request_tree_change_not_indexed(self):
"""Checks tree change forced by request """Checks tree change forced by request
For a tree like: For a tree like:
A A
| |
A A
|\ |\
B C B C
When B requests C tree in next transaction, but before indexation, When B requests C tree in next transaction, but before indexation,
the system shall disallow the operation.""" the system shall disallow the operation."""
request_kw = self.request_kw.copy() request_kw = self.request_kw.copy()
...@@ -428,8 +429,8 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -428,8 +429,8 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
C_instance = self.software_instance.REQUEST.get('request_instance') C_instance = self.software_instance.REQUEST.get('request_instance')
self.assertSameSet( self.assertSameSet(
self.software_instance.getPredecessorList(), self.software_instance.getPredecessorList(),
[B_instance.getRelativeUrl(), C_instance.getRelativeUrl()]) [B_instance.getRelativeUrl(), C_instance.getRelativeUrl()])
transaction.commit() transaction.commit()
...@@ -439,15 +440,15 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -439,15 +440,15 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
@expectedFailure @expectedFailure
def test_request_tree_change_same_transaction(self): def test_request_tree_change_same_transaction(self):
"""Checks tree change forced by request """Checks tree change forced by request
For a tree like: For a tree like:
A A
| |
A A
|\ |\
B C B C
When B requests C tree in the same transaction the system shall When B requests C tree in the same transaction the system shall
disallow the operation.""" disallow the operation."""
request_kw = self.request_kw.copy() request_kw = self.request_kw.copy()
...@@ -461,8 +462,8 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -461,8 +462,8 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
C_instance = self.software_instance.REQUEST.get('request_instance') C_instance = self.software_instance.REQUEST.get('request_instance')
self.assertSameSet( self.assertSameSet(
self.software_instance.getPredecessorList(), self.software_instance.getPredecessorList(),
[B_instance.getRelativeUrl(), C_instance.getRelativeUrl()]) [B_instance.getRelativeUrl(), C_instance.getRelativeUrl()])
self.assertRaises(NotImplementedError, B_instance.requestInstance, self.assertRaises(NotImplementedError, B_instance.requestInstance,
**request_kw) **request_kw)
...@@ -521,7 +522,6 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin): ...@@ -521,7 +522,6 @@ class TestSlapOSCoreSoftwareInstanceRequest(testSlapOSMixin):
self.assertEqual(request_kw['software_type'], self.assertEqual(request_kw['software_type'],
requested_instance2.getSourceReference()) requested_instance2.getSourceReference())
self.tic() self.tic()
request_kw['state'] = 'destroyed' request_kw['state'] = 'destroyed'
......
73 74
\ No newline at end of file \ No newline at end of file
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