Bug 61660 - New bug links on TestFailures page often contain titles that are so long they are rejected by Bugzilla
Summary: New bug links on TestFailures page often contain titles that are so long they...
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-05-27 12:58 PDT by Adam Roben (:aroben)
Modified: 2011-06-23 10:56 PDT (History)
1 user (show)

See Also:


Attachments
Make finding existing bugs and filing new bugs work on TestFailures even when lots of tests are failing (11.76 KB, patch)
2011-06-23 10:06 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-05-27 12:58:29 PDT
Bugzilla apparently has a 255-character limit on bug titles. The TestFailures page's new bug links include the names of all the failing tests, which can often be much more than 255 characters. Trying to submit one of these bugs results in an obscure Bugzilla error like this:

DBD::Pg::db selectrow_array failed: ERROR:  current transaction is aborted, commands ignored until end of transaction block [for Statement "SELECT CURRVAL('bugs_bug_id_seq')"] at Bugzilla/DB/Pg.pm line 90
	Bugzilla::DB::Pg::bz_last_key('Bugzilla::DB::Pg=HASH(0x100ebfdd8)', 'bugs', 'bug_id') called at Bugzilla/Object.pm line 367
	Bugzilla::Object::insert_create_data('Bugzilla::Bug', 'HASH(0x10096bf98)') called at Bugzilla/Bug.pm line 352
	Bugzilla::Bug::create('Bugzilla::Bug', 'HASH(0x100e244c0)') called at /www/hosts/bugs.webkit.org/post_bug.cgi line 177

We should make the TestFailures page never create a bug title that's longer than 255 characters.
Comment 1 Adam Roben (:aroben) 2011-06-23 10:06:57 PDT
Created attachment 98363 [details]
Make finding existing bugs and filing new bugs work on TestFailures even when lots of tests are failing
Comment 2 David Kilzer (:ddkilzer) 2011-06-23 10:13:02 PDT
Comment on attachment 98363 [details]
Make finding existing bugs and filing new bugs work on TestFailures even when lots of tests are failing

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

r=me

> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:272
> +            title = titlePrefix + failingTests.length + ' tests' + titleSuffix;

Would be nice to have some kind of ASSERT(title.length < Bugzilla.maximumBugTitleLength).
Comment 3 Adam Roben (:aroben) 2011-06-23 10:42:24 PDT
Comment on attachment 98363 [details]
Make finding existing bugs and filing new bugs work on TestFailures even when lots of tests are failing

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

>> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:272
>> +            title = titlePrefix + failingTests.length + ' tests' + titleSuffix;
> 
> Would be nice to have some kind of ASSERT(title.length < Bugzilla.maximumBugTitleLength).

I'll add a console.assert that does just that.
Comment 4 Adam Roben (:aroben) 2011-06-23 10:56:13 PDT
Committed r89588: <http://trac.webkit.org/changeset/89588>