<div dir="ltr">Hi Luciano<div> great changes to the runner!! I like them!</div><div> I have a few comments:</div><div>1) The changes in TestCase>>#debug and TestCase>>#debugAsFailureIfCanNot: can generate invalid results. The idea when running/debugging a test is not to reuse the previous instance of the test but to create a new one. Doing so the tests run isolated. Reusing the same instance can generate erratic results.</div><div>2) I fixed runSuiteProfiled:. It was doing: </div><div>...</div><div>self changed: #runTests ]]] newProcess.<br></div><div>...</div><div>And it should do:</div><div>...</div><div>   self changed: #runTests; changed: #isRunning; changed: #isNotRunning. ]]] newProcess.<br></div><div>....</div><div>as runSuite:</div><div>(I also extracted those methods to one)</div><div>3) I fixed a bug when running profiled an empty list of tests.</div><div><br></div><div>I'm attaching a .cs for those changes. I hope you agree with them. </div><div><br></div><div>Also, why did you remove the SUnit from the laben/menu? I'd like it to say "SUnit" for historical reasons ... and there could be runners of other types of tests... Do you mind if we go back to "SUnit ..."?</div><div><br></div><div>Cheers!</div><div>Hernan.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 24, 2023 at 1:01 PM Luciano Notarfrancesco via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Here's some changes for your consideration.</div><div>1) Updated the commands to enter unicode characters with \;</div><div>2) Added cmd-A shortcut to deselect all in the change list (analogous to cmd-a for select all);</div><div>3) Sort the messages in the browser message list with binary messages at the top;</div><div>4) Reworked the test runner.</div><div><br></div><div>The biggest change is the last one. I simplified the test runner window, added shortcuts for everything (cmd-a/A for select/deselect all, cmd-r for run, cmd-o for run one, cmd-l for canceling a run, etc), and fixed redrawing/update problems and some other bugs. Now the lists of error and failing tests are updated while the tests are running, and clicking on an item from those lists stops the current run and debugs the item immediately.</div><div>I use the test runner a lot, I run 5k tests on my project every time I change anything and it takes some minutes. I think these changes make the test runner more useful and less painful, let me know what you think.<br></div></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div><br clear="all"><div><br></div><span class="gmail_signature_prefix">-- </span><br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div style="font-size:small"><div dir="ltr"><div dir="ltr"><div style="font-size:12.8px"><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal"><span style="font-weight:bold">Hernán Wilkinson</span><br>Agile Software Development, Teaching & Coaching</span></font></span></span></strong></span></div><div style="font-size:12.8px"><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Phone: +54-011</span></font></span></span></strong></span><font face="tahoma, sans-serif" size="2">-4893-2057</font></div><div style="font-size:12.8px"><strong style="font-family:tahoma,sans-serif;font-size:xx-small"><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div style="font-size:12.8px"><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" target="_blank">http://www.10Pines.com</a></span></font></span></span></strong></span></div><div style="font-size:12.8px"><font face="tahoma, sans-serif"><span style="border-collapse:collapse">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div>