Summary: | Separate gtk unit tests | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Christian Dywan <christian> | ||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | Keywords: | Gtk | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Christian Dywan
2009-02-19 12:35:29 PST
Created attachment 28395 [details]
Separate the tests
This patch also includes the build support needed to build the separate tests.
Could you explain the reasoning behind creating multiple programs. The only benefit is that one run of a test is not influencing other parts. The downsides are runtime overhead to start the app, repeated boilerplate code, people driving the tests need to know the multiple binary names... I think the different files/areas should be separate GTestSuite's. (In reply to comment #2) > Could you explain the reasoning behind creating multiple programs. The only > benefit is that one run of a test is not influencing other parts. The downsides > are runtime overhead to start the app, repeated boilerplate code, people > driving the tests need to know the multiple binary names... > > I think the different files/areas should be separate GTestSuite's. Well, the main reason is modularity which makes it easy to maintain, run particular tests independently, add new tests, avoid having to edit a file that is several megabytes large... The overhead isn't really noticible. Arguably WebCore should live in a single file because that decreases linking overhead, what do you think? ;) It is not about the linking overhead. But think of automatic execution of the test executables, collecting the results, generating coverage data... having a single executable makes it a lot more easy to have a stable test setup. Comment on attachment 28395 [details] Separate the tests > +2009-03-07 Jan Michael Alonzo <jmalonzo@webkit.org> > + > + Reviewed by NOBODY (OOPS!). > + > + Separate gtk unit tests > + https://bugs.webkit.org/show_bug.cgi?id=24039 Could you please describe the reasoning of the split up here? and please use svn cp to copy the file when landing. (In reply to comment #5) > (From update of attachment 28395 [details] [review]) > > > +2009-03-07 Jan Michael Alonzo <jmalonzo@webkit.org> > > + > > + Reviewed by NOBODY (OOPS!). > > + > > + Separate gtk unit tests > > + https://bugs.webkit.org/show_bug.cgi?id=24039 > > Could you please describe the reasoning of the split up here? > > and please use svn cp to copy the file when landing. > Thanks. Landed in r41864 |