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

Revision history for this message
Tim Penhey (thumper) wrote :

On Thu, 24 Sep 2009 17:45:15 Michael Hudson wrote:
> Review: Approve
> 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."

Thanks. Updated.

« Back to merge proposal