Allon Mureinik has uploaded a new change for review.

Change subject: core: Cleanup DiskDao and its tests
......................................................................

core: Cleanup DiskDao and its tests

General cleanup in the DAO and its tests:
1. Made private methods static where possible
2. migrated Guid string constants from the test to FixturesTool so
   they could be reused.
3. Renamed some test methods to be more readable

Change-Id: I9551288f38b537c8196f2a2d79012457a0220cc1
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskDaoDbFacadeImpl.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/DiskDaoTest.java
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/FixturesTool.java
3 files changed, 22 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/35/4435/1
--
To view, visit http://gerrit.ovirt.org/4435
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9551288f38b537c8196f2a2d79012457a0220cc1
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to