[Cuis-dev] Some enhancements

[Hernán Wilkinson]

Hi Luciano
 great changes to the runner!! I like them!
 I have a few comments:
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.
2) I fixed runSuiteProfiled:. It was doing:
...
self changed: #runTests ]]] newProcess.
...
And it should do:
...
   self changed: #runTests; changed: #isRunning; changed: #isNotRunning.
]]] newProcess.
....
as runSuite:
(I also extracted those methods to one)
3) I fixed a bug when running profiled an empty list of tests.

I'm attaching a .cs for those changes. I hope you agree with them.

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 ..."?

Cheers!
Hernan.

On Wed, May 24, 2023 at 1:01 PM Luciano Notarfrancesco via Cuis-dev <
cuis-dev at lists.cuis.st> wrote:

> Here's some changes for your consideration.
> 1) Updated the commands to enter unicode characters with \;
> 2) Added cmd-A shortcut to deselect all in the change list (analogous to
> cmd-a for select all);
> 3) Sort the messages in the browser message list with binary messages at
> the top;
> 4) Reworked the test runner.
>
> 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.
> 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.
> --
> Cuis-dev mailing list
> Cuis-dev at lists.cuis.st
> https://lists.cuis.st/mailman/listinfo/cuis-dev
>


-- 

*Hernán WilkinsonAgile Software Development, Teaching & Coaching*
*Phone: +54-011*-4893-2057
*Twitter: @HernanWilkinson*
*site: http://www.10Pines.com <http://www.10pines.com/>*
Address: Alem 896, Floor 6, Buenos Aires, Argentina
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20230524/62604458/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 5806-SUnitImprovementsAndFixes-HernanWilkinson-2023May24-16h56m-HAW.001.cs.st
Type: application/octet-stream
Size: 2990 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20230524/62604458/attachment.obj>