fix: propagates exception from running runner.run() method (#184)

If someone omits required runtime class in a deployment archive there will be ClassDefNotFoundError when executing run() method in TestNGRunner. Arquillian servlet will return HTTP code 500 which is ignored in servlet protocol runner and consequently returned as a test which passed. But in reality the test did not even run and should be marked as failed.

testng/container/src/main/java/org/jboss/arquillian/testng/container/TestNGTestRunner.java

@@ -46,7 +46,12 @@ public TestResult execute(Class<?> testClass, String methodName) {
         runner.setXmlSuites(
             Collections.singletonList(createSuite(testClass, methodName)));
 
-        runner.run();
+        //we catch problems in executing run method, e.g. ClassDefNotFoundError and wrap it in TestResult
+        try {
+            runner.run();
+        } catch (Throwable ex) {
+            return TestResult.failed(ex);
+        }
 
         return resultListener.getTestResult();
     }