Bug 78283 - some tests in webkitpy are being run three times
Summary: some tests in webkitpy are being run three times
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dirk Pranke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-09 14:21 PST by Dirk Pranke
Modified: 2012-02-09 15:21 PST (History)
6 users (show)

See Also:


Attachments
Patch (3.62 KB, patch)
2012-02-09 14:23 PST, Dirk Pranke
rniwa: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Pranke 2012-02-09 14:21:41 PST
some tests in webkitpy are being run three times
Comment 1 Dirk Pranke 2012-02-09 14:23:20 PST
Created attachment 126371 [details]
Patch
Comment 2 Dirk Pranke 2012-02-09 14:25:08 PST
people who often write unittests in python ... fyi.
Comment 3 Adam Barth 2012-02-09 14:28:15 PST
Comment on attachment 126371 [details]
Patch

Good to know!
Comment 4 Dirk Pranke 2012-02-09 14:33:26 PST
Committed r107293: <http://trac.webkit.org/changeset/107293>
Comment 5 Eric Seidel (no email) 2012-02-09 14:49:17 PST
Comment on attachment 126371 [details]
Patch

Why are we even importing these into other unittests in the first place...

How did you find these?  I assume by using the -v output?

How much faster did this make test-webkitpy? :))
Comment 6 Dirk Pranke 2012-02-09 15:21:23 PST
(In reply to comment #5)
> (From update of attachment 126371 [details])
> Why are we even importing these into other unittests in the first place...
> 
> How did you find these?  I assume by using the -v output?
> 

-v plus debugging a script I wrote that buckets the test times ...

> How much faster did this make test-webkitpy? :))

Not much, maybe a second.

For more details on test speed, see bug 78289, where I summarize what I've dug up recently in service of whether or not parallel testing would be worthwhile.