Code review comment for lp:~thumper/launchpad/job-fail

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

It all looks fine apart from this comment in the test:

# Abort the transaction to confirm that the job status has been
# updated.

I suggest ".. to confirm that the update of the job status has been committed."

review: Approve

« Back to merge proposal