vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix build should_perform for empty JSON

Browse source
This commit is contained in:
Joseph Schorr 2016-09-21 14:59:01 -04:00
parent 2505de8720
commit 6fffc22b8a
2 changed files with 11 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
endpoints/notificationevent.py
View file

@ -156,6 +156,9 @@ class BaseBuildEvent(NotificationEvent):
return None return None
def should_perform(self, event_data, notification_data): def should_perform(self, event_data, notification_data):
if not notification_data.event_config_json:
return True
event_config = json.loads(notification_data.event_config_json) event_config = json.loads(notification_data.event_config_json)
ref_regex = event_config.get('ref-regex') or None ref_regex = event_config.get('ref-regex') or None
if ref_regex is None: if ref_regex is None:

8
test/test_notifications.py
View file

@ -13,6 +13,14 @@ class TestCreate(unittest.TestCase):
class TestShouldPerform(unittest.TestCase): class TestShouldPerform(unittest.TestCase):
def test_build_emptyjson(self):
notification_data = AttrDict({
'event_config_json': None,
})
# No build data at all.
self.assertTrue(BuildSuccessEvent().should_perform({}, notification_data))
def test_build_nofilter(self): def test_build_nofilter(self):
notification_data = AttrDict({ notification_data = AttrDict({
'event_config_json': '{}', 'event_config_json': '{}',