In Pharo 7 it works. 2017-11-09 18:29 GMT+01:00 Tim Mackinnon <tim@testit.works>:
> Thanks for looking at this - there is an issue however - when you apply > that change (at least in a Pharo 6.1 image) - it shows green even when a > test fails? So I think its turned one problem into the opposite one. > > Unfortunately I haven’t got a chance to look a bit deeper to help - but it > might be worth rolling back this change for now. We should fix it though - > and the answer must be in the area you have identified. > > tim > > On 9 Nov 2017, at 12:43, Denis Kudriashov <dionisi...@gmail.com> wrote: > > And now it is in latest Pharo > > 2017-11-09 12:16 GMT+01:00 Denis Kudriashov <dionisi...@gmail.com>: > >> Hi Tim. >> >> Fix is here 20661-Fixing-test-from-debugger-should-mark-test-as-gre >> en-when-proceed <https://github.com/pharo-project/pharo/pull/456> . >> Thank's for reporting. It forces me to fix. I always noticed it but never >> take it seriously :) >> >> 2017-11-09 11:32 GMT+01:00 Tim Mackinnon <tim@testit.works>: >> >>> Hi - I really like the build in test runner in the Pharo browsers, and I >>> was preparing a talk to show how great TDD is in Pharo and how we aren’t >>> ashamed of our debugger when testing (it augments the experience in fact - >>> letting you poke around and get your thoughts straight). >>> >>> However - if I pick rerun in the test runner debugger - and step through >>> a test and then correct the failing code, and hit resume - the browser >>> always shows a red failure, even though the execution is now correct. I >>> have to run the test again. >>> >>> This doesn’t seem right to me - are we missing a success event or >>> something? >>> >>> >>> Tim >>> >> >> > >