<div class="gmail_quote">On Fri, Jul 13, 2012 at 1:24 AM, Thiago Marcos P. Santos <span dir="ltr">&lt;<a href="mailto:tmpsantos@gmail.com" target="_blank">tmpsantos@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="gmail_quote"><div><div class="h5">On Tue, Jul 3, 2012 at 7:56 PM, Thiago Marcos P. Santos <span dir="ltr">&lt;<a href="mailto:tmpsantos@gmail.com" target="_blank">tmpsantos@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi WebKit EFL,<div><br></div><div>Thanks to the great work done by Krzysztof at the bug 68509, we have now an API Unit Test Framework for EWK.</div><div><br></div><div>You can run the tests individually or all tests at once by calling the runner script:</div>



<div><br></div><div>$ ./Tools/Scripts/run-efl-tests [--debug|--release]</div><div><br></div><div>The runner was integrated as a step of the build bots running tests. The only thing missing now is test coverage, and I hope the EFL community can work together at the meta bug tracking the missing tests [1]. I strongly recommend if you are working on a test, to add as dependency to this bug to give visibility to all the developers involved on the task.</div>



<div><br></div><div>Another suggestion would be that we do not accept any addition/modification to the APIs if there is no test backing the change. IMO this is aligned to the WebKit best practices and can only bring benefits to the port.</div>



<div><br></div><div>[1] <a href="https://bugs.webkit.org/show_bug.cgi?id=90451" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=90451</a></div><div><br></div><div>Cheers!</div></blockquote><div><br></div></div></div>

<div>And the unit test framework for WebKit 2 API has just landed! Let&#39;s make unit tests also a requirement for any new API&#39;s and changes.</div>
<div><br></div><div><a href="https://bugs.webkit.org/show_bug.cgi?id=90606" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=90606</a></div><div><br></div><div>Cheers,</div></div>
</blockquote></div><div><br></div><div>Hi,</div><div><br></div><div>We have now WebKit2 C API and EFL API tests running on the bots [1] and instructions for adding/running new EFL WebKit2 API tests [2].</div><div><br></div>

<div>Can we make it official that we need unit tests as acceptance criteria for any changes/additions to EFL WebKit2 API and the WebKit2 C API?</div><div><br></div><div>In any case, please remember to run the tests before submitting patches. In case of failure, the bots will turn red.</div>

<div><br></div><div><div>[1] <a href="http://build.webkit.org/builders/EFL%20Linux%2064-bit%20Debug/builds/2967/steps/API%20tests/logs/stdio">http://build.webkit.org/builders/EFL%20Linux%2064-bit%20Debug/builds/2967/steps/API%20tests/logs/stdio</a></div>

<div><br></div><div>[2] <a href="http://trac.webkit.org/wiki/EFLWebKitTests">http://trac.webkit.org/wiki/EFLWebKitTests</a></div></div><div><br></div><div>Cheers,</div>