diff --git a/core/src/main/java/eu/tsystems/mms/tic/testframework/execution/testng/worker/finish/MethodContextUpdateWorker.java b/core/src/main/java/eu/tsystems/mms/tic/testframework/execution/testng/worker/finish/MethodContextUpdateWorker.java index 701d9ff92..2b6f5d3f9 100644 --- a/core/src/main/java/eu/tsystems/mms/tic/testframework/execution/testng/worker/finish/MethodContextUpdateWorker.java +++ b/core/src/main/java/eu/tsystems/mms/tic/testframework/execution/testng/worker/finish/MethodContextUpdateWorker.java @@ -58,19 +58,6 @@ public void onMethodEnd(MethodEndEvent event) { testResult.setStatus(ITestResult.FAILURE); methodContext.setStatus(Status.FAILED); } -// StringBuilder sb = new StringBuilder(); -// sb.append("The following assertions failed in dataprovider method "); -// sb.append(testResult.getMethod().getDataProviderMethod().getMethod().getName()); -// sb.append(":"); -// AtomicInteger i = new AtomicInteger(); -// methodContext.readErrors() -//// .filter(ErrorContext::isNotOptional) -// .forEach(errorContext -> { -// i.incrementAndGet(); -// sb.append("\n").append(i).append(") ").append(errorContext.getThrowable().getMessage()); -// }); -// AssertionError testMethodContainerError = new AssertionError(sb.toString()); -// testResult.setThrowable(testMethodContainerError); } else // Handle collected assertions if we have more than one if (testResult.isSuccess() && methodContext.readErrors().anyMatch(ErrorContext::isNotOptional)) { @@ -129,7 +116,7 @@ public void onMethodEnd(MethodEndEvent event) { TestStep failedStep = methodContext.getCurrentTestStep(); methodContext.setFailedStep(failedStep); } else if (event.isSkipped()) { - // Can be caused by + // Can be caused by failed dataprovider or config methods methodContext.setStatus(Status.SKIPPED); } else if (testResult.isSuccess()) { methodContext.setStatus(Status.PASSED);