Eli Mesika has submitted this change and it was merged. Change subject: tools: notifier: fix NotificationPropertiesTest. ......................................................................
tools: notifier: fix NotificationPropertiesTest. Call NotificationProperties.release() so we do not get a stale copy of Notification Properties. (Set default and get instance are not enough if the current class loader has a copy of NotificationProperties with an old instance we will get that one!) also enable NotificationPropertiesTest to run both under maven and straight from source thus speeding up testing during development. Change-Id: Ia3488856323577e3db6c779d86c190e064e15c97 Signed-off-by: Mooli Tayer <mta...@redhat.com> --- M backend/manager/tools/src/test/java/org/ovirt/engine/core/notifier/utils/NotificationPropertiesTest.java 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved mooli tayer: Verified -- To view, visit http://gerrit.ovirt.org/24387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia3488856323577e3db6c779d86c190e064e15c97 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <mta...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: mooli tayer <mta...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
