openstack/placement
Code Issues Proposed changes

Merge "Fix uuid replacement in aggregate notification test"

Browse Source
This commit is contained in:
Jenkins 2017-05-19 12:26:16 +00:00 committed by Gerrit Code Review
commit 99019e8089
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
nova/tests/functional/notification_sample_tests/test_aggregate.py
View File

@ -25,17 +25,15 @@ class TestAggregateNotificationSample(
aggregate = self.admin_api.post_aggregate(aggregate_req)
self.assertEqual(2, len(fake_notifier.VERSIONED_NOTIFICATIONS))
# The uuid hasn't been exposed on the REST API yet so we have no way to
# match it here, now.
self._verify_notification(
'aggregate-create-start',
replacements={
'uuid': self.ANY},
'uuid': aggregate['uuid']},
actual=fake_notifier.VERSIONED_NOTIFICATIONS[0])
self._verify_notification(
'aggregate-create-end',
replacements={
'uuid': self.ANY,
'uuid': aggregate['uuid'],
'id': aggregate['id']},
actual=fake_notifier.VERSIONED_NOTIFICATIONS[1])
@ -45,12 +43,12 @@ class TestAggregateNotificationSample(
self._verify_notification(
'aggregate-delete-start',
replacements={
'uuid': self.ANY,
'uuid': aggregate['uuid'],
'id': aggregate['id']},
actual=fake_notifier.VERSIONED_NOTIFICATIONS[2])
self._verify_notification(
'aggregate-delete-end',
replacements={
'uuid': self.ANY,
'uuid': aggregate['uuid'],
'id': aggregate['id']},
actual=fake_notifier.VERSIONED_NOTIFICATIONS[3])