[Pharo-project] Issue 3984 in pharo: Announcements announced when a test is runned
Status: FixProposed Owner: Benjamin...@gmail.com Labels: Milestone-1.3 Nautilus New issue 3984 by Benjamin...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 More precisely, when the history of the test case is updated Attachments: AnnouncementsInTestCase.1.cs 2.8 KB
Comment #1 on issue 3984 by ryd...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 Please... no Announcement at the end of class name :) Announcements are action notifications, and their names should indicate that. Also, Runned is not English, either Ran, or Finished, or Completed, etc.
Comment #2 on issue 3984 by ryd...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 Also, should this be done by an announcer of a TestRunner, or is SystemAnnouncer ok? To me, that seems a bit strange of a place to be announcing them :)
Comment #3 on issue 3984 by Benjamin...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 Oops sorry for my bad english :s I used SystemAnnouncer because for me, tests are part of the system... But we can implement a TestAnnouncer, it's not a big deal
Updates: Status: Started Comment #4 on issue 3984 by stephane...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 (No comment was entered for this change.)
Updates: Status: FixProposed Comment #5 on issue 3984 by Benjamin...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 A new version of announcements for TestCase The first one can be forgotten Attachments: TestCaseAnnouncements.1.cs 3.6 KB
Comment #6 on issue 3984 by Benjamin...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 A little fix in addition of the previous one Attachments: FixTestCaseAnnouncement.1.cs 974 bytes
Updates: Status: Started Comment #7 on issue 3984 by marcus.d...@gmail.com: Announcements announced when a test is runned http://code.google.com/p/pharo/issues/detail?id=3984 The changeset contains changes to NautilusUI and can not be loaded.
participants (1)
-
pharo＠googlecode.com