Mike Kolesnik has uploaded a new change for review. Change subject: core: Overhaul SetupNetworksHelper ......................................................................
core: Overhaul SetupNetworksHelper Now that tests are in order, we can rewrite SetupNetworksHelper a bit: 1. Changed fully to class fields since it looks cleaner and makes more sense now that tests don't rely on inner methods. 2. Some methods which were used only once got removed. 3. validations were fixed to correctly address cases where iface/bond/network were already processed. Change-Id: I96266c56b0d7f2d0c6aacb45e4a39087f94258f1 Signed-off-by: Mike Kolesnik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SetupNetworksHelper.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/SetupNetworksHelperTest.java 2 files changed, 94 insertions(+), 126 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/34/5234/1 -- To view, visit http://gerrit.ovirt.org/5234 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I96266c56b0d7f2d0c6aacb45e4a39087f94258f1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
