Code review comment for lp:~trapnine/maas/fix-1501982

Revision history for this message
Blake Rouse (blake-rouse) wrote :

Looks good. But missing 3 unit tests. You have unit tests on the overall generated result, but you should unit test each of those new methods explicitly.

review: Needs Fixing

« Back to merge proposal