Change in ovirt-engine[master]: core: Remove VmStatic from VmStatisticsDaoTest

amureini at redhat.com amureini at redhat.com
Wed May 25 07:37:25 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove VmStatic from VmStatisticsDaoTest
......................................................................


core: Remove VmStatic from VmStatisticsDaoTest

VmStatisticsDaoTest is about testing VmStatisticsDao[Impl] - it should
not have anything to do with VmStatic. This is especially true as the
VmStatic is just retrieved from the database by ID, and then only used
to store that ID.

Change-Id: I07de1d5f823150acbf26179f769deebfae15a9be
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmStatisticsDaoTest.java
1 file changed, 3 insertions(+), 7 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Allon Mureinik: Verified
  Arik Hadas: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/58004
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I07de1d5f823150acbf26179f769deebfae15a9be
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list