They actually are already handled.  I added this as part of the pulp 2.4 stuff (notice its not in master) and since realized my mistake.  

The done method above should handle it:
```
       def done?
         !!external_task[:finish_time]
       end
```