From: Nikita Popov <nikita.ppv@gmail.com>
Date: Thu, 11 Jul 2019 10:45:49 +0000 (+0200)
Subject: Include test name in JUnit testcase name
X-Git-Tag: php-7.4.0beta1~166
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c02dfddda3c4e267c99d301f5f042f36da50a658;p=php

Include test name in JUnit testcase name

Azure does not show the classname attribute in any convenient way,
and the file name is the part we usually care about.
---

diff --git a/run-tests.php b/run-tests.php
index 96df073a84..d83669b706 100755
--- a/run-tests.php
+++ b/run-tests.php
@@ -3444,8 +3444,8 @@ function junit_mark_test_as($type, $file_name, $test_name, $time = null, $messag
 	}, $escaped_details);
 	$escaped_message = htmlspecialchars($message, ENT_QUOTES, 'UTF-8');
 
-	$escaped_test_name = htmlspecialchars($test_name, ENT_QUOTES);
-	$JUNIT['files'][$file_name]['xml'] = "<testcase classname='$file_name' name='$escaped_test_name' time='$time'>\n";
+	$escaped_test_name = htmlspecialchars($file_name . ' (' . $test_name . ')', ENT_QUOTES);
+	$JUNIT['files'][$file_name]['xml'] = "<testcase name='$escaped_test_name' time='$time'>\n";
 
 	if (is_array($type)) {
 		$output_type = $type[0] . 'ED';