kobotoolbox / kpi / 15534384297 / 2

81%

master: 76%

push

github

fix(massEmails): make sure sends are not interrupted when configs change TASK-1835 (#5810)

### 📣 Summary
Ensure emails continue to be sent throughout the day even when
configurations are changed before the send is over.



### 📖 Description
Fixes a bug wherein if sending all mass emails took more than an hour
(either because of volume or because the process was killed) and new
configurations were added before all emails had been sent, emails would
stop sending for the rest of the day. The same thing would happen if an
existing configuration was removed from the daily send list.


### 💭 Notes
Before sending any emails, the `send_emails` job tries to verify that
all send lists for the day have been generated. Previously, it did this
by making sure the list of all live email configurations matched the
list of processed configurations that had been cached by
`generate_mass_email_user_lists`. However, this created a problem when
the list of live email configurations changed between the run of
`generate_mass_email_user_lists` and a run of send_emails occurring
later on the same day, either because someone added a new configuration
and set it live or because they took an existing configuration and took
it out of the daily send.

This PR fixes this issue by updating the `send_emails` task to only
check that the `generate_mass_email_user_lists` task has put *something*
in the cache, rather than checking that all live configs are there. This
is because setting the cache key is the last thing the
`generate_mass_email_user_lists` task does, so if the cache key is set,
that means we have already created all the send lists we need for the
day. This also updates the `generate_mass_email_user_lists` task to only
generate the list of configs once rather than re-checking it every time.
This ensures that while it will still try again if it fails, it will not
add new configurations in the middle of the day.

Note this does mean that even if an email configuration is rem... (continued)

6333 of 10014 branches covered (63.24%)

23924 of 29523 relevant lines covered (81.04%)

0.81 hits per line