[Webkit-unassigned] [Bug 168979] Add ability for webkitpy to run bindings tests

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=168979

--- Comment #3 from Sri Vijayaraghavan <svijayaraghavan at apple.com> ---
(In reply to comment #2)
> Comment on attachment 302955 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=302955&action=review
> 
> looks good to me.
> 
> > Tools/Scripts/webkitpy/tool/steps/runtests.py:168
> > +        self._tool.filesystem.maybe_make_directory(results_directory)
> 
> do we need similar line in _run_javascriptcore_tests as well above.

JSC tests cannot run without building, so if the WebKitBuild/[Release/Debug] directory doesn't exist at the time of writing the test results to a file, something would have gone very wrong.

Adding the line enables some code to be abstracted out, but I'd prefer to not abstract things out until we have another group of tests.

Some things, like the results directory, might change by then. Placing results inside WebKitBuild/ for consistency is OK, but it might make sense to move as we add more tests that don't require a build to run (bindings is one of them, also webkitpy, webkitperl etc).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170228/43c8f6c6/attachment.html>