Allon Mureinik has submitted this change and it was merged. Change subject: core: QuotaManagerTest throws declarations ...................................................................... core: QuotaManagerTest throws declarations Allow methods to declare a thrown exception (which is never actually thrown) instead of having to catch it in order to clean up the code. Besides cleaning up the code, this patch has an added bonus of actually making the test fail if the underlying implementation is ever broken in a way that a CloneNotSupportedException is thrown. Change-Id: Ieddb8c728258be8176b24fb97109dda79e60e47e Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/quota/QuotaManagerTest.java 1 file changed, 10 insertions(+), 16 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Allon Mureinik: Verified; Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63573 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieddb8c728258be8176b24fb97109dda79e60e47e Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>