com web/qa: Fix recognition of test filenames: reports/parserfunc.php
| From: | Christoph Michael Becker | Date: | Tue, 24 Jul 2018 13:47:33 +0000 |
| Subject: | com web/qa: Fix recognition of test filenames: reports/parserfunc.php | ||
| Groups: | php.qa | ||
| Request: | Send a blank email to php-qa+get-68137@lists.php.net to get a copy of this message | ||
Commit: 68173202fd328debe9a96b2705d32b6e0572af5f
Author: Christoph M. Becker <cmbecker69@gmx.de> Tue, 24 Jul 2018 15:47:33 +0200
Parents: 404890adeaf37f751147ecc8cc96f2d7e6ca859b
Branches: master
Link: http://git.php.net/?p=web/qa.git;a=commitdiff;h=68173202fd328debe9a96b2705d32b6e0572af5f
Log:
Fix recognition of test filenames
The filenames of submitted test runs are determined from each line of
the test results, which has the format
testname [filename]. If the
testname contains a bracketed phrase, the filename is not correctly
recognized. We fix this by anchoring the pattern to the end of the
line (but allow for trailing whitespace).
This is supposed to fix the resulting mess that currently can be seen
on <http://qa.php.net/reports/run_tests.php?version=7.2.0>
for new bug
reports.
Changed paths:
M reports/parserfunc.php
Diff:
diff --git a/reports/parserfunc.php b/reports/parserfunc.php
index 519ef0a..7cd4004 100644
--- a/reports/parserfunc.php
+++ b/reports/parserfunc.php
@@ -241,7 +241,7 @@ function parse_phpmaketest($version, $status=null, $file)
$currentTest = '';
} elseif ($currentPart == 'failedTest' || $currentPart ==
'expectedFailedTest') {
- preg_match('@ \[([^\]]{1,})\]@', $row, $tab);
+ preg_match('@ \[([^\]]{1,})\]\s*$@', $row, $tab);
if (count($tab) == 2)
if (!isset($extract[$currentPart]) || !in_array($tab[1], $extract[$currentPart]))
$extract[$currentPart][] = $tab[1];