Commit:    68173202fd328debe9a96b2705d32b6e0572af5f
Author:    Christoph M. Becker <cmbecke...@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];


--
PHP Quality Assurance Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to