commit 4dd8f4768197b043fdb3876ec203e8618c69015b
author Vladislav Rassokhin <vladrassokhin@gmail.com> Sun Nov 23 02:55:43 2014 +0300
committer Vladislav Rassokhin <vladrassokhin@gmail.com> Sun Nov 23 03:36:50 2014 +0300
tree df15495ff1deb6ad3838692797955da84adbb9f5
parent a85a2ac54f23fc8a01c2d59a56997f7eb7d40cdb

Invoker: simplify exception in parameters provider case of method invocation.

src/main/java/org/testng/internal/Invoker.java

diff --git a/src/main/java/org/testng/internal/Invoker.java b/src/main/java/org/testng/internal/Invoker.java
index f5d9d9b..4b46ac9 100644
--- a/src/main/java/org/testng/internal/Invoker.java
+++ b/src/main/java/org/testng/internal/Invoker.java
@@ -1067,7 +1067,7 @@
    *
    * Note (alex): this method can be refactored to use a SingleTestMethodWorker that
    * directly invokes
-   * {@link #invokeTestMethod(Object[], ITestNGMethod, Object[], XmlSuite, Map, ITestClass, ITestNGMethod[], ITestNGMethod[], ConfigurationGroupMethods)}
+   * {@link #invokeTestMethod(Object, ITestNGMethod, Object[], int, XmlSuite, Map, ITestClass, ITestNGMethod[], ITestNGMethod[], ConfigurationGroupMethods, FailureContext)}
    * and this would simplify the implementation (see how DataTestMethodWorker is used)
    */
   @Override
@@ -1141,12 +1141,10 @@
             parameters, allParameterNames, suite, testContext, instance);
 
         if (bag.hasErrors()) {
-          handleInvocationResults(testMethod,
-              Lists.newArrayList(bag.errorResult), expectedExceptionHolder, true,
-              true /* collect results */, failure);
-          ITestResult tr = registerSkippedTestResult(testMethod, instance,
-              System.currentTimeMillis(),
-              bag.errorResult.getThrowable());
+          final ITestResult tr = bag.errorResult;
+          tr.setStatus(ITestResult.SKIP);
+          runTestListeners(tr);
+          m_notifier.addSkippedTest(testMethod, tr);
           result.add(tr);
           continue;
         }