Summary: | Web Inspector: move LayoutTests/inspector-protocol/ tests to LayoutTests/inspector/ | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | BJ Burg <bburg> | ||||
Component: | Web Inspector | Assignee: | BJ Burg <bburg> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bburg, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=147739 https://bugs.webkit.org/show_bug.cgi?id=147787 |
||||||
Attachments: |
|
Description
BJ Burg
2015-08-06 10:08:48 PDT
I would go with unit-tests. unit tests just seems odd. Created attachment 258441 [details]
Proposed Fix
Comment on attachment 258441 [details]
Proposed Fix
I think this can lead to confusion. Now LayoutTests/inspector has some tests that use 1 path (protocol-test) and other tests that use another path (inspector-test) both have "InspectorTest.foo" that behave differently.
Committed r188142: <http://trac.webkit.org/changeset/188142> (In reply to comment #3) > Comment on attachment 258441 [details] > Proposed Fix > > I think this can lead to confusion. Now LayoutTests/inspector has some tests > that use 1 path (protocol-test) and other tests that use another path > (inspector-test) both have "InspectorTest.foo" that behave differently. I would be happy to rename InspectorTest from protocol-test.js to ProtocolTest. There is plenty of de-duplication that we should do, and now it's more obvious where we need to improve. |