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