Change in ovirt-engine[master]: webadmin: Add ErrorMessagesTest to frontend module

From Allon Mureinik <amureini@redhat.com>:
Allon Mureinik has submitted this change and it was merged. Change subject: webadmin: Add ErrorMessagesTest to frontend module ...................................................................... webadmin: Add ErrorMessagesTest to frontend module Commit 25603e [rightfully!] removed ErrorMessagesTest from the webadmin module, as it no longer has an AppErrors.properties file, after it was removed in 6d4008. That patch, however, neglected to add an ErrorMessagesTest to the new frontend location. This patch adds back the missing test and removes the redundant key in the properties file found by the said test. Change-Id: I570eb1d7f48a4b53e5408e5d670f6d4dc6b607d9 Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M frontend/webadmin/modules/frontend/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties A frontend/webadmin/modules/frontend/src/test/java/org/ovirt/engine/ui/frontend/ErrorMessagesTest.java 2 files changed, 11 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/78228 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I570eb1d7f48a4b53e5408e5d670f6d4dc6b607d9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review