Michael Blow has submitted this change and it was merged. Change subject: [NO ISSUE][TEST] Log exceptions on test failures ......................................................................
[NO ISSUE][TEST] Log exceptions on test failures Change-Id: I5a2713a4430eeece3fcb5a9549a6e06b0ce01bec Reviewed-on: https://asterix-gerrit.ics.uci.edu/3155 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Contrib: Jenkins <[email protected]> Reviewed-by: Murtadha Hubail <[email protected]> --- M asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Anon. E. Moose #1000171: Jenkins: Verified; No violations found; ; Verified Murtadha Hubail: Looks good to me, approved diff --git a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java index b762517..7e29ed5 100644 --- a/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java +++ b/asterixdb/asterix-common/src/test/java/org/apache/asterix/test/base/TestMethodTracer.java @@ -56,7 +56,7 @@ @Override protected void failed(Throwable e, Description description) { - LOGGER.log(level, "### {} FAILED ({})", description.getMethodName(), e.getClass().getName()); + LOGGER.log(level, "### {} FAILED", description.getMethodName(), e); } @Override -- To view, visit https://asterix-gerrit.ics.uci.edu/3155 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a2713a4430eeece3fcb5a9549a6e06b0ce01bec Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]>
