diff options
| author | Ralph Bean <rbean@redhat.com> | 2016-10-03 12:50:57 +0000 |
|---|---|---|
| committer | Ralph Bean <rbean@redhat.com> | 2016-10-03 12:50:57 +0000 |
| commit | 4b56744f76dd3e637e59e94dea7160d9f705c6d4 (patch) | |
| tree | ee2ef1c961a7965822667c6088b3b43f55dde964 /roles/badges | |
| parent | 76474e985fd0710d540e5dd9bac6a6cd61a770bd (diff) | |
Add more debug information to this sanity check.
Diffstat (limited to 'roles/badges')
| -rw-r--r-- | roles/badges/backend/files/cron/award-oldschool-badges | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/roles/badges/backend/files/cron/award-oldschool-badges b/roles/badges/backend/files/cron/award-oldschool-badges index 49fb651d1..6fa0554c7 100644 --- a/roles/badges/backend/files/cron/award-oldschool-badges +++ b/roles/badges/backend/files/cron/award-oldschool-badges @@ -153,7 +153,8 @@ def main(): # First, some validation that the badge ids actually exist. for fas_group, badge_id in mapping.items(): badge = tahrir.get_badge(badge_id=badge_id) - assert(badge.id) + if not badge: + raise ValueError("%r is not a valid badge id" % badge_id) # Then, do a long query against FAS for our candidates. results = get_fas_groupings(fas_credentials=fm_config['fas_credentials'], |