<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On May 8, 2017, at 11:15 PM, Ryosuke Niwa <<a href="mailto:rniwa@webkit.org" class="">rniwa@webkit.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">On Mon, May 8, 2017 at 11:01 PM, Brady Eidson <</span><a href="mailto:beidson@apple.com" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">beidson@apple.com</a><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">> wrote:</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><div class="apple-mail-abbreviatedContentWrapper" style="background-color: rgb(255, 255, 255); -webkit-user-select: none; font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><a class="apple-mail-expandLink" href="x-redundant-cluster-toggle://0" style="text-decoration: none; color: inherit;"></a><div class="apple-mail-topPreviewContent" style="overflow: hidden;"><a class="apple-mail-expandLink" href="x-redundant-cluster-toggle://0" style="text-decoration: none; color: inherit;"></a><blockquote type="cite" class=""><a class="apple-mail-expandLink" href="x-redundant-cluster-toggle://0" style="text-decoration: none; color: inherit;"></a><blockquote type="cite" class=""><a class="apple-mail-expandLink" href="x-redundant-cluster-toggle://0" style="text-decoration: none; color: inherit;">On May 8, 2017, at 10:44 PM, Ryosuke Niwa <</a><a href="mailto:rniwa@webkit.org" class="">rniwa@webkit.org</a>> wrote:<br class="">On Mon, May 8, 2017 at 10:17 PM, Brady Eidson <<a href="mailto:beidson@apple.com" class="">beidson@apple.com</a>> wrote:<br class=""><blockquote type="cite" class="">But now talking about testharness.js directly, I object on the grounds of "a<br class="">file:// regression test is dirt easy to hack on and work with, whereas<br class="">anything that requires me to have an httpd running is a PITA"<br class=""></blockquote>I think whether we use file:// or http:// is orthogonal point to using<br class="">testharness.js.  Many of the tests Chris and I have written using<br class="">testharness.js are checked into regular LayoutTests/ directories, and<br class="">they work just fine.<br class=""></blockquote>Yes, I misunderstood this in Youenn's original message. Good to know!<br class=""><blockquote type="cite" class=""><blockquote type="cite" class="">I just object to making it the "recommended way" of writing tests.<br class=""></blockquote>Would you equally object to making js-test.js / js-test-pre.js the<br class="">recommended way of writing tests?<br class=""></blockquote>Yes.<br class=""><blockquote type="cite" class="">If not, why?<br class=""></blockquote>N/A<br class=""><blockquote type="cite" class="">What we're suggesting is to give preferential treatments to<br class="">testharness.js over js-test.js / js-test-pre.js when you were already<br class="">planning to write a test with the latter two scripts.<br class=""></blockquote>"It's okay to write your test however you'd like. If you were considering using js-test, maybe you should consider using testharness instead."<br class="">Is that's what's being proposed?<br class=""></blockquote></div><div class="apple-mail-seeMoreText"></div><div class="apple-mail-bottomPreviewContent" style="overflow: hidden;"></div></div><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class=""><br class=""><blockquote type="cite" class="">On May 8, 2017, at 10:44 PM, Ryosuke Niwa <<a href="mailto:rniwa@webkit.org" class="">rniwa@webkit.org</a>> wrote:<br class=""><br class="">On Mon, May 8, 2017 at 10:17 PM, Brady Eidson <<a href="mailto:beidson@apple.com" class="">beidson@apple.com</a>> wrote:<br class=""><br class=""><blockquote type="cite" class="">But now talking about testharness.js directly, I object on the grounds of "a<br class="">file:// regression test is dirt easy to hack on and work with, whereas<br class="">anything that requires me to have an httpd running is a PITA"<br class=""></blockquote><br class="">I think whether we use file:// or http:// is orthogonal point to using<br class="">testharness.js.  Many of the tests Chris and I have written using<br class="">testharness.js are checked into regular LayoutTests/ directories, and<br class="">they work just fine.<br class=""></blockquote><br class="">Yes, I misunderstood this in Youenn's original message. Good to know!<br class=""><blockquote type="cite" class=""><br class=""><blockquote type="cite" class="">I just object to making it the "recommended way" of writing tests.<br class=""></blockquote><br class="">Would you equally object to making js-test.js / js-test-pre.js the<br class="">recommended way of writing tests?<br class=""></blockquote><br class="">Yes.<br class=""><br class=""><blockquote type="cite" class="">If not, why?<br class=""></blockquote><br class="">N/A<br class=""><br class=""><blockquote type="cite" class="">What we're suggesting is to give preferential treatments to<br class="">testharness.js over js-test.js / js-test-pre.js when you were already<br class="">planning to write a test with the latter two scripts.<br class=""></blockquote><br class="">"It's okay to write your test however you'd like. If you were considering using js-test, maybe you should consider using testharness instead."<br class=""><br class="">Is that's what's being proposed?<br class=""></blockquote></div></blockquote><div><br class=""></div><div>Besides other issues mentioned, testharness tends to result in more verbose tests compared to js-test, at least for simple cases.</div><br class=""><blockquote type="cite" class=""><div class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">The thing I specifically asked Youenn to ask is, whether we should</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">place a test inside LayoutTests/wpt like LayoutTests/http/tests when</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">we want to write a test using testharness.js which requires some sort</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">of network code.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Since people have had some opinions about directory structures in the past.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""></div></blockquote></div><br class=""><div class="">It seems like we need a few different directories, here are my opinions on them:</div><div class=""><br class=""></div><div class="">(1) Imported web platform tests that don't need a server</div><div class="">    Currently LayoutTests/imported/w3c/web-platform-tests, which seems fine.</div><div class="">(2) Imported web platform tests that do need a server</div><div class="">    Probably should be under LayoutTests/imported/w3c/ somewhere, or maybe under http/tests/ per point (4)</div><div class="">(3) Custom testharness.js tests that don't need a server </div><div class="">    Probably these should just go in their normal topic-specific directories and should not need a special directory</div><div class="">(4) Custom testharness.js tests that do need a server</div><div class="">    Can these just be a subdirectory of http/tests/? We have websocket and ssl/tls tests in there too. Would be nice to not need a separate directory for networking tests that to use a particular test framework.</div><div class=""><br class=""></div><div class="">Regards,</div><div class="">Maciej</div><div class=""><br class=""></div></body></html>