Bug 63350 - Short bug titles from TestFailures page give no indication which tests are failing
Summary: Short bug titles from TestFailures page give no indication which tests are fa...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://build.webkit.org/TestFailures/
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-24 13:32 PDT by Adam Roben (:aroben)
Modified: 2011-06-24 13:56 PDT (History)
1 user (show)

See Also:


Attachments
Include the directory containing the failing tests in titles of bugs filed from TestFailures when all the tests won't fit (4.17 KB, patch)
2011-06-24 13:38 PDT, Adam Roben (:aroben)
ddkilzer: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-06-24 13:32:28 PDT
When so many tests are failing that they all can't be listed in the bug title, the TestFailures page generates a bug title of the form:

REGRESSION (r12345): 8 tests failing on Windows 7 Release (Tests)

This gives no indication of which tests are failing. It would be nice if we could include some info about which tests are failing, even if we can't name all the tests.
Comment 1 Adam Roben (:aroben) 2011-06-24 13:38:45 PDT
Created attachment 98530 [details]
Include the directory containing the failing tests in titles of bugs filed from TestFailures when all the tests won't fit
Comment 2 David Kilzer (:ddkilzer) 2011-06-24 13:44:55 PDT
Comment on attachment 98530 [details]
Include the directory containing the failing tests in titles of bugs filed from TestFailures when all the tests won't fit

View in context: https://bugs.webkit.org/attachment.cgi?id=98530&action=review

r=me

> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Utilities.js:96
> +        if (!splitPaths.every(function(splitPath) { return splitPath[i] === firstSplitPath[i] }))

That's a neat way to write the test!
Comment 3 Adam Roben (:aroben) 2011-06-24 13:56:02 PDT
Committed r89702: <http://trac.webkit.org/changeset/89702>