Code review comment for lp:~salvatore-orlando/nova/bug723301

Revision history for this message
termie (termie) wrote :

These two docstrings should have a one line summary, followed by a newline and then the additional information.

37 + """Simulates an error while downloading an image.
38 + Verifies that VDIs created are properly cleaned up.
39 +
40 + """

For example, would become:

"""Simulates an error while downloading an image.

Verifies that VDIs created are properly cleaned up.

"""

50 + def test_spawn_fail_cleanup_2(self):
51 + """Simulates an error while creating VM record. It
52 + verifies that VDIs created are properly cleaned up.
53 +
54 + """

review: Needs Fixing

« Back to merge proposal