Guarding against result.summary being unset

[mhauru]

I'm trying King Arthur out, and have run into a couple of instances where the job result is None or result.summary is None, but the code tries to get fields from it and crashes. I put in some quick if result.summary: clauses to the plug the hole in my local version, but I wonder if that's just masking some deeper issue about an assumed guarantee of when result.summary is set that is not being respected.

I've seen this happen with at least this snippet from scheduler.py:

            logger.error("Job #%s (task: %s) failed but will be resumed",
                         job_id, task_id)

            if result.summary.fetched > 0:
                task.backend_args['next_from_date'] = result.summary.max_updated_on

                if result.summary.max_offset:
                    task.backend_args['next_offset'] = result.summary.max_offset

which raises AttributeError: 'NoneType' object has no attribute 'summary', and this one in jobs.py

    logger.debug("Job #%s (task: %s) completed (%s) - %s/%s items (%s) fetched",
                 result.job_id, task_id, result.backend,
                 str(result.summary.fetched), str(result.summary.skipped),
                 result.category)

which raises AttributeError: 'NoneType' object has no attribute 'fetched'.