Change in ovirt-engine[master]: core: Cleanup SetupNetworksHelperTest

mkolesni at redhat.com mkolesni at redhat.com
Mon May 21 09:40:58 UTC 2012


Mike Kolesnik has submitted this change and it was merged.

Change subject: core: Cleanup SetupNetworksHelperTest
......................................................................


core: Cleanup SetupNetworksHelperTest

1. Removed use of static import where it is not our convention.
2. Reordered annotations to be after javadoc, as is common in the java
   world.
3. Fixed typos in test names.
4. Added break-line (<br>) to javadocs, so that formatter won't ruin
them.
5. Ran formatter (got rid of some blank lines).

Change-Id: Ifbd42ab37118df5de2a9811e9952b10489527304
Signed-off-by: Mike Kolesnik <mkolesni at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/SetupNetworksHelperTest.java
1 file changed, 41 insertions(+), 49 deletions(-)

Approvals:
  Mike Kolesnik: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/4601
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifbd42ab37118df5de2a9811e9952b10489527304
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>



More information about the Engine-commits mailing list