<br><br><div class="gmail_quote">On Thu, May 12, 2011 at 11:33 AM, David Levin <span dir="ltr"><<a href="mailto:levin@chromium.org">levin@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br><br><div class="gmail_quote"><div class="im">On Thu, May 12, 2011 at 10:36 AM, Adam Barth <span dir="ltr"><<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



Here's a straw-man proposal:<br>
<br>
Tests/ </blockquote></div><div>†-- All testing code. †Do test harnesses go here too?</div><div class="im"><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


 †Layout/ †# Would be nice to have a better name since these test much<br>
more than Layout<br></blockquote><div>†</div></div><div>DumpRenderTreeTests?</div><div>Integration?</div><div>AutomatedHtmlTests?</div><div>†</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




 †Performance/<br>
 †Unit/<br></blockquote><div><br></div><div>with a directory structure that mirrors Source for the tests</div><div>†</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




 †Manual/<br>
Source/</blockquote><div>†--†Only the source code necessary for building WebKit.</div><div class="im"><div>†</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 †JavaScriptCore/<br>
 †WebCore/<br>
 †WebKit/<br>
 †ThirdParty/<br>
 † †ANGLE/<br>
Tools/<br>
 †WebKitTestRunner/†</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 †MiniBrowser/<br>
 †Scripts/<br></blockquote><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In this organization, gtest probably doesn't belong in Source because<br>
it's not a dependency of WebKit.</blockquote><div><br></div></div><div>I agree but I felt like it lacked explicit definition, so I added some.</div><div class="im"><div>†</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



 †It probably belongs somewhere in<br>
Tests. †Maybe inside the Unit directory somewhere?<br></blockquote><div><br></div></div><div>I think so. I wonder why we wouldn't put all tests related code under Tests. For example†WebKitTestRunner, DumpRenderTree,†CSSTestSuiteHarness seem to fit in that category and be similar in some ways to gtest in that they are the test harness.</div>

</div></blockquote><div><br></div><div>Dmitry Lomov pointed out to me that DRT is a tool on its own. Now, I see a reasonable distinction here.†</div><div><br></div><div>In short, test harnesses are tools.</div><div>oth, gtest is part of the test code itself.</div>

<div><br></div><div>†</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="gmail_quote"><div><div></div><div class="h5">

<div><br></div><div>†</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<font color="#888888"><br>
Adam<br>
</font><div><div></div><div><br>
<br>
On Thu, May 12, 2011 at 10:24 AM, Dmitry Lomov <<a href="mailto:dslomov@google.com" target="_blank">dslomov@google.com</a>> wrote:<br>
> I think more important issue to consider is should WebKit unit-tests<br>
> (TestWebKitAPI) live under Tools.<br>
> Unit-tests evolve with the product and an organic part of it - it doesn't<br>
> feel that they constitute a "tool".<br>
> Three possible locations come to mind:<br>
> - ./Tests - separate top-level directory, sibling to Source and LayoutTests<br>
> - ./Source/Test - separate directory, but under Source to emphasize that<br>
> tests are part of the source code<br>
> - Scattered around by the files that tests test, as David suggests.<br>
> In terms of gtest directory placement, I think it is nice to have all<br>
> third-party libraries live in the same place.<br>
> What do people think?<br>
> Kind regards,<br>
> Dmitry<br>
><br>
> On Thu, May 12, 2011 at 9:38 AM, David Levin <<a href="mailto:levin@chromium.org" target="_blank">levin@chromium.org</a>> wrote:<br>
>><br>
>> It sounds like you have a helpful mental mapping for what belongs in each<br>
>> directory that we haven't written anywhere. †Perhaps you could write it down<br>
>> and send it to webkit-dev so that we can make it a common model.<br>
>> It feels like a related topic is why are the unit tests under Tools as<br>
>> opposed to being by the files that they test. Being by the files they tests<br>
>> would make them easier to find and change as needed. †(For an example of<br>
>> where this is done,<br>
>> see†<a href="http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common" target="_blank">http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common</a> where<br>
>> there is file.py and file_unittest.py)<br>
>> dave<br>
>><br>
>> On Thu, May 12, 2011 at 8:36 AM, Dan Bernstein <<a href="mailto:mitz@apple.com" target="_blank">mitz@apple.com</a>> wrote:<br>
>>><br>
>>> Is gtest required to build any of the WebKit ports? If not, can it be<br>
>>> moved out of Source and into Tools?<br>
>>><br>
>>> Thanks,<br>
>>> óDan<br>
>>> _______________________________________________<br>
>>> webkit-dev mailing list<br>
>>> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
>>> <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
>><br>
>><br>
>> _______________________________________________<br>
>> webkit-dev mailing list<br>
>> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
>> <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
>><br>
><br>
><br>
> _______________________________________________<br>
> webkit-dev mailing list<br>
> <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
> <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
><br>
><br>
</div></div></blockquote></div></div></div><br>
</blockquote></div><br>