altf4arnold/datatracker
1
0
Fork 0
Code Issues Pull requests Actions 3 Packages Projects Releases Wiki Activity

fix: remove test so all deleted or swapped (onsite<->remote) registrations get removed from MeetingRegistrations (#4359)

Browse source
This commit is contained in:
Ryan Cross 2022-08-25 08:00:04 -07:00 committed by GitHub
parent 41ae0cfac1
commit a94a87f336
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
ietf/stats/utils.py
View file

@ -246,7 +246,8 @@ def get_meeting_registration_data(meeting):
decoded = response.json() decoded = response.json()
except ValueError: except ValueError:
if response.content.strip() == 'Invalid meeting': if response.content.strip() == 'Invalid meeting':
pass logger.info('Invalid meeting: {}'.format(meeting.number))
return (0,0,0)
else: else:
raise RuntimeError("Could not decode response from registrations API: '%s...'" % (response.content[:64], )) raise RuntimeError("Could not decode response from registrations API: '%s...'" % (response.content[:64], ))
@ -307,16 +308,14 @@ def get_meeting_registration_data(meeting):
num_created += 1 num_created += 1
num_processed += 1 num_processed += 1
# handle deleted registrations, if count is reasonable
# any registrations left in meeting_registrations no longer exist in reg # any registrations left in meeting_registrations no longer exist in reg
# so must have been deleted # so must have been deleted
if 0 < len(meeting_registrations) < 5: for r in meeting_registrations:
for r in meeting_registrations: try:
try: MeetingRegistration.objects.get(meeting=meeting,email=r[0],reg_type=r[1]).delete()
MeetingRegistration.objects.get(meeting=meeting,email=r[0],reg_type=r[1]).delete() logger.info('Removing deleted registration. email={}, reg_type={}'.format(r[0], r[1]))
logger.info('Removing deleted registration. email={}, reg_type={}'.format(r[0], r[1])) except MeetingRegistration.DoesNotExist:
except MeetingRegistration.DoesNotExist: pass
pass
else: else:
raise RuntimeError("Bad response from registrations API: %s, '%s'" % (response.status_code, response.content)) raise RuntimeError("Bad response from registrations API: %s, '%s'" % (response.status_code, response.content))
num_total = MeetingRegistration.objects.filter( num_total = MeetingRegistration.objects.filter(