Change in ovirt-engine[master]: core: Fix dao tests not to print lookup error

yzaslavs at redhat.com yzaslavs at redhat.com
Wed Feb 20 06:21:37 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Fix dao tests not to print lookup error
......................................................................


core: Fix dao tests not to print lookup error

Although this does not break tests, seeing an exception
when running the first test in a bunch of tests is not something nice.
This patch solve this by introducing an EJBUtilsStrategy that uses the
createDataSource static method to "lookup" the dataSource, and set it
before DbFacadadeLocator.getDbFacade is called.

Change-Id: I9f37acaea5c21463c4e126799524a3c41b6071a3
Signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/BaseDAOTestCase.java
1 file changed, 10 insertions(+), 3 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/12176
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9f37acaea5c21463c4e126799524a3c41b6071a3
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list