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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
