fixed a problem with nested testsuites

* the reporter now selects every `testsuites` that has `testcases` and ignores all `testsuites` with only `sub-testsuites`
* it iterates through all errors and adds them only to the resultset if the set is not empty -> that is necessary because there are still testsuites that includes `testcases` and `testsuites`

Author: pfefferle

functions.php

@@ -105,10 +105,8 @@ function process_junit_xml( $xml_string )
 
 	$results['testsuites'] = array();
 
-	$testsuites = $xml->xpath( '//testsuites//testsuite[ ( @file ) and ( @errors > 0 or @failures > 0 ) ]' );
+	$testsuites = $xml->xpath( '//testsuites//testsuite[ ( count( testcase ) > 0 ) and ( @errors > 0 or @failures > 0 ) ]' );
 	foreach ( $testsuites as $testsuite ) {
-		// Handle nested testsuites like tests with data providers.
-		$testsuite = isset( $testsuite->testsuite ) ? $testsuite->testsuite : $testsuite;
 		$result = array(
 			'name'     => (string) $testsuite['name'],
 			'tests'    => (string) $testsuite['tests'],
@@ -118,19 +116,22 @@ function process_junit_xml( $xml_string )
 		if ( empty( $result['failures'] ) && empty( $result['errors'] ) ) {
 			continue;
 		}
-		$results['testsuites'][ (string) $testsuite['name'] ] = $result;
-		$results['testsuites'][ (string) $testsuite['name'] ]['testcases'] = array();
+		$failures = array();
 		foreach ( $testsuite->testcase as $testcase ) {
 			// Capture both failure and error children.
 			foreach ( array( 'failure', 'error') as $key ) {
 				if ( isset( $testcase->{$key} ) ) {
-					$results['testsuites'][ (string) $testsuite['name'] ]['testcases'][ (string) $testcase['name'] ] = array(
+					$failures[ (string) $testcase['name'] ] = array(
 						'name' => (string) $testcase['name'],
 						$key   => (string) $testcase->{$key},
 					);
 				}
 			}
 		}
+		if ( $failures ) {
+			$results['testsuites'][ (string) $testsuite['name'] ] = $result;
+			$results['testsuites'][ (string) $testsuite['name'] ]['testcases'] = $failures;
+		}
 	}
 
 	return json_encode( $results );