Alon Bar-Lev has posted comments on this change.

Change subject: tools: notifier: fix NotificationPropertiesTest.
......................................................................


Patch Set 5: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/24387/5/backend/manager/tools/src/test/java/org/ovirt/engine/core/notifier/utils/NotificationPropertiesTest.java
File 
backend/manager/tools/src/test/java/org/ovirt/engine/core/notifier/utils/NotificationPropertiesTest.java:

Line 13:     @BeforeClass
Line 14:     static public void beforeClass() {
Line 15:         NotificationProperties.release();
Line 16:         NotificationProperties.setDefaults(
Line 17:                 
ClassLoader.getSystemResource("conf/notifier-prop-test.conf").getPath(),
should be:

                 
URLDecoder.decode(ClassLoader.getSystemResource("conf/notifier-prop-test.conf").getPath(),
 "UTF-8"),
Line 18:                 ""
Line 19:         );
Line 20:         prop = NotificationProperties.getInstance();
Line 21:         assertNotNull(prop);


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia3488856323577e3db6c779d86c190e064e15c97
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: mooli tayer <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to