Code review comment for lp:~julian-edwards/maas/download-service-exceptions

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Nicely done. Just one note: shouldn't the existing tests then call try_download instead of maybe_start_download, so that the success case is covered?

review: Approve

« Back to merge proposal