Allon Mureinik has submitted this change and it was merged.
Change subject: core: QuotaDaoTest: Remove Config dependency
......................................................................
core: QuotaDaoTest: Remove Config dependency
The QuotaDaoTest tests performing database CRUD operations on Quota
objects. The actual values set on the Quota object are
inconsequential for this test, and there's no reason to query them from
Config values in the database.
This patch removes the quota dependency and replaces the Config calls
with the actual values that would have been returned from the test
database.
Change-Id: If5a13d9675f0b49d663fa6a1e46e75c49c5b3bb9
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/QuotaDaoTest.java
1 file changed, 12 insertions(+), 14 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Allon Mureinik: Verified; Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/65016
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If5a13d9675f0b49d663fa6a1e46e75c49c5b3bb9
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>