Follow up: conditions optional fields

This is a follow-up on the patch:

  I08606cea676ecf57bbb3b73077c4832240fbe0d2

As requested in the reviews.
Wondering why I didn't fix it right away :D

Change-Id: Iab377f5fa4a1d448615914e1e9aa5a49053da693
This commit is contained in:
dparalen 2017-05-18 14:28:22 +02:00
parent 65945d1086
commit 00635e2344
1 changed files with 4 additions and 3 deletions

View File

@ -396,9 +396,7 @@ class Test(Base):
rule = {
'conditions': [
test_rules.BaseTest.condition_defaults(
{'op': 'eq', 'field': 'memory_mb', 'value': 1024}
)
{'op': 'eq', 'field': 'memory_mb', 'value': 1024},
],
'actions': [{'action': 'fail', 'message': 'boom'}],
'description': 'Cool actions'
@ -408,6 +406,9 @@ class Test(Base):
self.assertTrue(res['uuid'])
rule['uuid'] = res['uuid']
rule['links'] = res['links']
rule['conditions'] = [
test_rules.BaseTest.condition_defaults(rule['conditions'][0]),
]
self.assertEqual(rule, res)
res = self.call('get', rule['links'][0]['href']).json()