Merge pull request #1323 from coreos-inc/secworkerreturn

Move security notification work into its own method to allow for retu…
This commit is contained in:
josephschorr 2016-06-02 13:59:25 -04:00
commit ec492bb683
3 changed files with 15 additions and 3 deletions

View file

@ -587,7 +587,7 @@ class TestSecurityScanner(unittest.TestCase):
# Test with an unknown notification.
with HTTMock(get_notification, unknown_notification):
worker = SecurityNotificationWorker(None)
self.assertFalse(worker.process_queue_item({
self.assertFalse(worker.perform_notification_work({
'Name': 'unknownnotification'
}))
@ -598,7 +598,7 @@ class TestSecurityScanner(unittest.TestCase):
with HTTMock(get_notification, delete_notification, unknown_notification):
worker = SecurityNotificationWorker(None)
self.assertTrue(worker.process_queue_item(data))
self.assertTrue(worker.perform_notification_work(data))
self.assertEquals(['GET-1', 'GET-2', 'DELETE'], pages_called)