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@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@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>