Bug 79602

Summary:

Extract the logic to merge tests from MergeTestsHandler and add unit tests

Product:

WebKit

Reporter:

Ryosuke Niwa <rniwa>

Component:

Tools / Tests

Assignee:

Ryosuke Niwa <rniwa>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, morrita

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Removed unnecessary handler rules in app.yaml	none

Description Ryosuke Niwa 2012-02-26 01:22:49 PST

The logic to merge tests in MergeTestsHandler is mission critical, and should be thoroughly unit-tested.

Comment 1 Ryosuke Niwa 2012-02-26 01:25:43 PST

Also, we can't be running this logic in frontend since merging a test with lots of TestResult would hit the process deadline.

Comment 2 Ryosuke Niwa 2012-02-26 01:35:11 PST

Created attachment 128910 [details]
Patch

Comment 3 Ryosuke Niwa 2012-02-26 01:38:53 PST

Created attachment 128911 [details]
Removed unnecessary handler rules in app.yaml

Comment 4 Ryosuke Niwa 2012-02-27 18:14:39 PST

Comment on attachment 128911 [details]
Removed unnecessary handler rules in app.yaml

Clearing flags on attachment: 128911

Committed r109057: <http://trac.webkit.org/changeset/109057>

Comment 5 Ryosuke Niwa 2012-02-27 18:14:43 PST

All reviewed patches have been landed.  Closing bug.