No new separate test but I did update a test that was already there for the functionality that I introduced.
On Mon, Nov 3, 2014 at 10:44 AM, Blake Rouse <email address hidden> wrote:
> Review: Needs Fixing > > No tests? > > Also the commit message is a little long. I would give exactly what this > does or fixes. Explaining the better way in the description. > -- > > https://code.launchpad.net/~newell-jensen/maas/bulk-node-actions-messages-bug-1387968/+merge/240217 > You are the owner of > lp:~newell-jensen/maas/bulk-node-actions-messages-bug-1387968. >
« Back to merge proposal
No new separate test but I did update a test that was already there for the
functionality that I introduced.
On Mon, Nov 3, 2014 at 10:44 AM, Blake Rouse <email address hidden>
wrote:
> Review: Needs Fixing /code.launchpad .net/~newell- jensen/ maas/bulk- node-actions- messages- bug-1387968/ +merge/ 240217
>
> No tests?
>
> Also the commit message is a little long. I would give exactly what this
> does or fixes. Explaining the better way in the description.
> --
>
> https:/
> You are the owner of
> lp:~newell-jensen/maas/bulk-node-actions-messages-bug-1387968.
>