> I will integrate it.
> I was thinking that WindowOpened is not that english but it has the advantage 
> that all the events of a certain kinds are grouped.

Yes, this is why I first used SUAnnouncement and SUTestBegin...

Alexandre


> 
> Stef
> 
> 
> On Aug 2, 2010, at 12:08 PM, Alexandre Bergel wrote:
> 
>> hi!
>> 
>> In PharoInbox there is: 
>> SLICE-Issue-2721-AnnouncementsForSUnit-Alexandre_Bergel.2
>> The comment of this slice is: 
>> -=-=-=-=-=-=-=-=-=-=-=-=
>> Renamed the event classes into:
>> TestCaseAnnouncement
>> -- TestCaseStarted
>> -- TestCaseEnded
>> 
>> Event are also emitted in TestResult>>runCase: like this, the result of the 
>> test execution is accessible from the event.
>> (Thanks Laurent Laffon for these suggestions)
>> -=-=-=-=-=-=-=-=-=-=-=-=
>> 
>> Cheers,
>> Alexandre
>> 
>> On 29 Jul 2010, at 22:22, laurent laffont wrote:
>> 
>>> On Thu, Jul 29, 2010 at 7:42 PM, Henrik Sperre Johansen 
>>> <henrik.s.johan...@veloxit.no> wrote:
>>> On 29.07.2010 17:45, Alexandre Bergel wrote:
>>> Wow ! Fast !.
>>> 
>>> That means it should be easy now to do a lot of exciting stuff. Show tests 
>>> progression, errors/failures as soon as they occur, log several things, 
>>> drink cocktails,  meet beautiful girls ....that's really cool.
>>> I will be in winter time in 1 week time. We have to enjoy now
>>> 
>>> Alexandre
>>> Nice!
>>> Maybe it's just me, or could we not add Announcement to the end of every 
>>> Announcement subclass name though?
>>> Like with the window announcement, I like them better as verbs describing 
>>> the event, f.ex.
>>> someAnnouncer on: TestStarted do: []
>>> someAnnouncer on: TestEnded send: #x to: y
>>> 
>>> +1. I propose
>>> TestCaseAnnouncement
>>> -- TestCaseStarted
>>> -- TestCaseEnded
>>> 
>>> 
>>> Shouldn't we put announce in TestResult>>runCase: instead of 
>>> TestCase>>runCase ? This way we can pass the result to the Annoucement 
>>> object.
>>> 
>>> 
>>> Actually, self announce: SUTestEndAnnouncement. is not in the ensure: 
>>> block. So if the test fails there's no annoucement sent.
>>> 
>>> 
>>> Cheers,
>>> 
>>> Laurent
>>> 
>>> 
>>> 
>>> Cheers,
>>> Henry
>>> 
>>> 
>>> _______________________________________________
>>> Pharo-project mailing list
>>> Pharo-project@lists.gforge.inria.fr
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>> 
>>> _______________________________________________
>>> Pharo-project mailing list
>>> Pharo-project@lists.gforge.inria.fr
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>> 
>> -- 
>> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
>> Alexandre Bergel  http://www.bergel.eu
>> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
>> 
>> 
>> 
>> 
>> 
>> 
>> _______________________________________________
>> Pharo-project mailing list
>> Pharo-project@lists.gforge.inria.fr
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
> 
> 
> _______________________________________________
> Pharo-project mailing list
> Pharo-project@lists.gforge.inria.fr
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

-- 
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.






_______________________________________________
Pharo-project mailing list
Pharo-project@lists.gforge.inria.fr
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Reply via email to