<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/12/13 Code Raguet <span dir="ltr"><<a href="mailto:iraguet@activitycentral.com" target="_blank">iraguet@activitycentral.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Only on the first save().<br>Using getEnv().defaultTimeoutInterval = something we could increase the timeout for all the datastore tests or even for all the tests. I'm worried it would get a bit annoying for development to always have a big timeout...</blockquote>


<div> </div></div><div>Perhaps we should do as in env.js...</div><div>Adding anachronistic unit-tests (tests that check the logic in isolation, mocking and stubbing the external services and deps) that would run faster, hence lighting the development process.</div>

</div></blockquote><div><br></div><div>+1</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div dir="ltr"><div>Also, moving those heavier tests (whose check through the actual implementation) to another file that, developers would run after a long round of development (before pushing, for example) or just leaving that job to buildbot.</div>

</div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">
<div>Therefore, we would have the benefit that the current tests give (keeping big timeouts, may be) without delaying developers.</div></div></blockquote><div><br></div><div>+1. This point is an advantage to run tests on any basic browser if is necessary in any time.<br>

</div><div><br></div><div>Then we can do:</div><div>- separate this functional tests on another file and run it on full-master maybe for first approach?</div><div>- solve on temporary way using increasing timeout (if we see that this approach works...)</div>

<div>- doing unit tests for datastore.js</div><div><br></div><div>I think... now it works (always assuming that higher time, we reduce or avoid completely the probability of failure of the tests, to see...), just that it is sometimes a bit slow datastore labor, we should take note to solve in the future, but inevitably we should have unit tests here and be sure that the functional tests on full-master run well, even a little slow the running time of the tests.<br>

</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5"><div class="gmail_extra">

<br><br><div class="gmail_quote">On Fri, Dec 13, 2013 at 2:48 PM, Rogelio Mita <span dir="ltr"><<a href="mailto:rogeliomita@activitycentral.com" target="_blank">rogeliomita@activitycentral.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div><br><div class="gmail_quote">

2013/12/12 Daniel Narvaez <span dir="ltr"><<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<pre><span>Only on the first save().<br></span></pre><pre><span>Using </span><span><em>getEnv</em>().<em>defaultTimeoutInterval</em> = something we could increase the timeout for all the datastore tests or even for all the tests. I'm worried it would get a bit annoying for development to always have a big timeout... An alternative could be to have a speficic test at the top to get through this, with a title explaining what it's doing, so that we wouldn't keep breaking it. I dunno... thoughts? Patches welcome too, if anyone has more time than me!</span></pre>




</blockquote></div><br></div>oh!, now we have a lot of information about, let me collect and understand better the situation before send a more complete answer<span><font color="#888888"><br><br clear="all">
<div><br></div>-- <br>Roger<div><br><div><a href="http://activitycentral.com/" target="_blank">Activity Central</a></div>

</div>
</font></span></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Roger<div><br><div><a href="http://activitycentral.com/" target="_blank">Activity Central</a></div></div>
</div></div>