Code review comment for lp:~julian-edwards/maas/vm-pause-state-bug-1394382

Revision history for this message
Julian Edwards (julian-edwards) wrote :

On Tuesday 02 Dec 2014 07:21:07 you wrote:
> Review: Approve
>
> It's worth adding a tests that shows that for each VM state in VirshVMState,
> power_state_virsh() returns a value that makes sense. Other than that,
> LGTM.

Thanks for the review.

I'm not sure what that would be really testing. There are already tests for
the happy path and the unhappy path. The rest of it is just mappings that
I've added.

« Back to merge proposal