November 2008 - Pharo-dev - lists.pharo.org

Re: [Pharo-project] Sunit Extensions breaking the TestRunner

by Keith Hodges

I apologize for my previous emails being somewhat brash. I admit that am not in the best of spirits having been hit by a firework (no lasting damage done). Keith

Nov. 10, 2008

Re: [Pharo-project] SUnit extensions and icons OB support

by Simon Denier

On 10 nov. 08, at 19:51, Alexandre Bergel wrote: > Yes, this is the case. When you run the test, the result are stored > into the class that has been run. > Maybe we could store them in a location outside the package. But > test results will not be persistent. Yes and no. The package becomes dirty only if the test results change - ie if a test goes from failure to pass for example. > > > Alexandre > > On 10 Nov 2008, at 14:40, StÃ©phane Ducasse wrote: > >> what would be nice if that the packages do not get dirty because >> the tests run. >> I do not know if this is the case but this is important. >> >> On Nov 10, 2008, at 4:58 PM, Alexandre Bergel wrote: >> >>> I have seen this. We should continue this discussion offline, then >>> propose a robust and stable version to the pharo community. >>> >>> Cheers, >>> Alexandre >>> >>> >>> On 10 Nov 2008, at 12:53, Simon Denier wrote: >>> >>>> >>>> On 10 nov. 08, at 11:07, David RÃ¶thlisberger wrote: >>>> >>>>> >>>>>> So let me know when I can load again and integrate your >>>>>> extension. >>>>> >>>>> Could you please check with SUnit-DavidRoethlisberger.42 in http://www.squeaksource.com/Pharo >>>>> and execute ScriptLoader class >> loadOB? >>>> >>>> It is not enough >>>> >>>> Starting from a fresh 10144 image, loading OB with scriptloader, >>>> then your latest commit for sunit, I do get the icons in the >>>> browser, but cant update their state: not lastRun method is >>>> generated and it fails when I try to save the results in the >>>> TestRunner. Best guess: the code generating lastRun on the class >>>> side is lost. >>>> >>>>> >>>>> >>>>>> Then after it would be great to have a button to run the tests >>>>>> and >>>>> >>>>> yes, we should have something like that in the browser. >>>>> >>>>> David >>>>> >>>>> >>>>>>> Hi Stef, >>>>>>> >>>>>>> as you as the SUnit extensions are integrated, you just need >>>>>>> to load OB. Any OB load method in ScriptLoader can be used for >>>>>>> this as this extension is also in the stable OB version now. >>>>>>> >>>>>>> David >>>>>>> >>>>>>>> can you tell me what I should do to get the icons showing the >>>>>>>> status of tests in your browser? >>>>>>>> I should harvest the Sunit extensions and then? >>>>>>>> Stef >>>>>>>> _______________________________________________ >>>>>>>> Pharo-project mailing list >>>>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>>> >>>>>>> >>>>>>> >>>>>>> _______________________________________________ >>>>>>> Pharo-project mailing list >>>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>>> >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> Pharo-project(a)lists.gforge.inria.fr >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo- >>>>> project >>>>> >>>> >>>> -- >>>> Simon >>>> >>>> >>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> Pharo-project(a)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(a)lists.gforge.inria.fr >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)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(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- Simon

Nov. 10, 2008

Re: [Pharo-project] New pharo-dev based on latest version

by StÃ©phane Ducasse

On Nov 10, 2008, at 7:08 PM, Igor Stasenko wrote: > Some of my observations of glitches found: > > - open world menu System->About. Now instead of clicking Ok, try drag > the window. Leads to DNU when you release a mouse button. thanks. I may be the result of my etoy surgery. I will check. > - a submenu gradient in OB (maybe not only in OB) is sporadically > loses its colors: > when you first open a context menu, and first open a submenu in it, > all is drawn ok. But then try hover over different menu items to force > hiding/showing submenus, you will notice that newly opened submenus > start losing gradient background, until finally background becomes > completely white. This is a feature :) Stef > > > 2008/11/10 Damien Cassou <damien.cassou(a)gmail.com>: >> http://gforge.inria.fr/frs/download.php/10931/pharo0.1-10142dev08.11.2.zip >> >> always available at: http://code.google.com/p/pharo/wiki/Downloads?tm=2 >> >> -- >> Damien Cassou >> http://damiencassou.seasidehosting.st >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > > -- > Best regards, > Igor Stasenko AKA sig. > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

Nov. 10, 2008

Re: [Pharo-project] SUnit extensions and icons OB support

by Alexandre Bergel

Yes, this is the case. When you run the test, the result are stored into the class that has been run. Maybe we could store them in a location outside the package. But test results will not be persistent. Alexandre On 10 Nov 2008, at 14:40, StÃ©phane Ducasse wrote: > what would be nice if that the packages do not get dirty because the > tests run. > I do not know if this is the case but this is important. > > On Nov 10, 2008, at 4:58 PM, Alexandre Bergel wrote: > >> I have seen this. We should continue this discussion offline, then >> propose a robust and stable version to the pharo community. >> >> Cheers, >> Alexandre >> >> >> On 10 Nov 2008, at 12:53, Simon Denier wrote: >> >>> >>> On 10 nov. 08, at 11:07, David RÃ¶thlisberger wrote: >>> >>>> >>>>> So let me know when I can load again and integrate your extension. >>>> >>>> Could you please check with SUnit-DavidRoethlisberger.42 in http://www.squeaksource.com/Pharo >>>> and execute ScriptLoader class >> loadOB? >>> >>> It is not enough >>> >>> Starting from a fresh 10144 image, loading OB with scriptloader, >>> then your latest commit for sunit, I do get the icons in the >>> browser, but cant update their state: not lastRun method is >>> generated and it fails when I try to save the results in the >>> TestRunner. Best guess: the code generating lastRun on the class >>> side is lost. >>> >>>> >>>> >>>>> Then after it would be great to have a button to run the tests and >>>> >>>> yes, we should have something like that in the browser. >>>> >>>> David >>>> >>>> >>>>>> Hi Stef, >>>>>> >>>>>> as you as the SUnit extensions are integrated, you just need to >>>>>> load OB. Any OB load method in ScriptLoader can be used for >>>>>> this as this extension is also in the stable OB version now. >>>>>> >>>>>> David >>>>>> >>>>>>> can you tell me what I should do to get the icons showing the >>>>>>> status of tests in your browser? >>>>>>> I should harvest the Sunit extensions and then? >>>>>>> Stef >>>>>>> _______________________________________________ >>>>>>> Pharo-project mailing list >>>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>> >>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> Pharo-project(a)lists.gforge.inria.fr >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo- >>>>> project >>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> Pharo-project(a)lists.gforge.inria.fr >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>> >>> >>> -- >>> Simon >>> >>> >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> Pharo-project(a)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(a)lists.gforge.inria.fr >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >> > > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.

Nov. 10, 2008

Re: [Pharo-project] Sunit Extensions breaking the TestRunner

by Keith Hodges

Igor Stasenko wrote: > 2008/11/10 StÃ©phane Ducasse <stephane.ducasse(a)inria.fr>: > >> Hi keith >> >> >>>>> We had Kerala loaded when we committed this SUnit version, but Kerala >>>>> >>>> overrides method in SUnit such as TestResult>>runCase:, so those >>>> methods didn't get committed. Very bad. >>>> >>> Not bad, typical, and you get used to it winding you up if you are an >>> MC1.0 user. So you have to adopt a strategy for maintaining each >>> package, such as maintainng an image specifically for each package that >>> you may have overrides in. >>> >>> So that is what MC1.5 was written for, to fix this bug! But people have >>> failed to see the need for it. >>> >>> Because many of you folks are package writers, rather than package >>> users/tweakers you don't find this problem an issue. But if the package >>> writer will not take on board requests for modifications what can you do. >>> >> You lost me. >> Could reexplained it to me> May be I'm just too tired too :) >> >> > > Let me elaborate. Suppose there is two persons A and B. > Both are using package C. > And both want to override some methods in package C to make their > packages working. > But author of package C refusing to include modifications of both persons. > Now, the only option they have is to ship their MC package each with > own overrides to original 'maintained' version of C, and hope that if > someone would want to load both A and B packages, their overrides will > not interfere with each other. > As an alternative users A and B can club together and make a package called C-extras, which avoids the problem of overrides clashing. This is exactly the situation we have with Seaside, package maintainers refusing to add even simple accessors, so... Seaside28Jetsam was created as "Seaside-Extras". This is exactly the role of the Kernel-Extensions package (not that it includes many overrides). Several have turned their noses up at Kernel-Extensions because it includes things that they dont necessarily like, but in the process you are pooring scorn on a social process of collecting all extensions in a managed place where collisions can be avoided. Taking this on board... a package developer could recognise that perhaps they haven't got it all right and recognise a need to harness and guide such extras. Thus encouraging users of their package to collaborate in order to make the extras well documented and worthy of inclusion into future releases of the main package. Presently, the culture of "overrides are evil" is maintained because of the danger of clashes as Igor explained. But they dont have to be evil if they are managed in a social collaborative process which avoids such collisions, i.e. Seaside28Jetsam, and KernelExtensions. Those who write the packages can pronounce the doctrine of "overrides are evil", because they are not forced to use them, however those of us who want to actually. The solution for SUnit is for those who are interested in improving SUnit to actually TALK to others who have been already doing the same, and to collaborate on one such SUnit-improved rather than 10 separate ones! regards Keith

Nov. 10, 2008

Re: [Pharo-project] Sunit Extensions breaking the TestRunner

by Igor Stasenko

2008/11/10 StÃ©phane Ducasse <stephane.ducasse(a)inria.fr>: > Hi keith > >>>> We had Kerala loaded when we committed this SUnit version, but Kerala >>> >>> overrides method in SUnit such as TestResult>>runCase:, so those >>> methods didn't get committed. Very bad. >> >> Not bad, typical, and you get used to it winding you up if you are an >> MC1.0 user. So you have to adopt a strategy for maintaining each >> package, such as maintainng an image specifically for each package that >> you may have overrides in. >> >> So that is what MC1.5 was written for, to fix this bug! But people have >> failed to see the need for it. >> >> Because many of you folks are package writers, rather than package >> users/tweakers you don't find this problem an issue. But if the package >> writer will not take on board requests for modifications what can you do. > > You lost me. > Could reexplained it to me> May be I'm just too tired too :) > Let me elaborate. Suppose there is two persons A and B. Both are using package C. And both want to override some methods in package C to make their packages working. But author of package C refusing to include modifications of both persons. Now, the only option they have is to ship their MC package each with own overrides to original 'maintained' version of C, and hope that if someone would want to load both A and B packages, their overrides will not interfere with each other. -- Best regards, Igor Stasenko AKA sig.

Nov. 10, 2008

Re: [Pharo-project] New pharo-dev based on latest version

by Igor Stasenko

Some of my observations of glitches found: - open world menu System->About. Now instead of clicking Ok, try drag the window. Leads to DNU when you release a mouse button. - a submenu gradient in OB (maybe not only in OB) is sporadically loses its colors: when you first open a context menu, and first open a submenu in it, all is drawn ok. But then try hover over different menu items to force hiding/showing submenus, you will notice that newly opened submenus start losing gradient background, until finally background becomes completely white. 2008/11/10 Damien Cassou <damien.cassou(a)gmail.com>: > http://gforge.inria.fr/frs/download.php/10931/pharo0.1-10142dev08.11.2.zip > > always available at: http://code.google.com/p/pharo/wiki/Downloads?tm=2 > > -- > Damien Cassou > http://damiencassou.seasidehosting.st > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- Best regards, Igor Stasenko AKA sig.

Nov. 10, 2008

Re: [Pharo-project] Sunit Extensions breaking the TestRunner

by StÃ©phane Ducasse

Hi keith >>> We had Kerala loaded when we committed this SUnit version, but >>> Kerala >> overrides method in SUnit such as TestResult>>runCase:, so those >> methods didn't get committed. Very bad. > Not bad, typical, and you get used to it winding you up if you are an > MC1.0 user. So you have to adopt a strategy for maintaining each > package, such as maintainng an image specifically for each package > that > you may have overrides in. > > So that is what MC1.5 was written for, to fix this bug! But people > have > failed to see the need for it. > > Because many of you folks are package writers, rather than package > users/tweakers you don't find this problem an issue. But if the > package > writer will not take on board requests for modifications what can > you do. You lost me. Could reexplained it to me> May be I'm just too tired too :) > >> I committed a new version to Pharo: SUnit-DavidRoethlisberger.42 in >> which I also merged changes of stephane_ducassse.denker.41. >> Let me know if you need anything else to make SUnit work. >> >> David > the only reason you would need to override runCase: is if your SUnit > doesnt include runCaseWith: aBlock as a specific place intended as a > hook for test cases to wrap all tests in contexts/exception handlers. > (e.g. WACurrentSession) > > SUnit in www.squeaksource.com/Testing includes runCaseWith: > > Keith > > _______________________________________________ > Pharo-project mailing list > Pharo-project(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

Nov. 10, 2008

Re: [Pharo-project] SUnit extensions and icons OB support

by StÃ©phane Ducasse

what would be nice if that the packages do not get dirty because the tests run. I do not know if this is the case but this is important. On Nov 10, 2008, at 4:58 PM, Alexandre Bergel wrote: > I have seen this. We should continue this discussion offline, then > propose a robust and stable version to the pharo community. > > Cheers, > Alexandre > > > On 10 Nov 2008, at 12:53, Simon Denier wrote: > >> >> On 10 nov. 08, at 11:07, David RÃ¶thlisberger wrote: >> >>> >>>> So let me know when I can load again and integrate your extension. >>> >>> Could you please check with SUnit-DavidRoethlisberger.42 in http://www.squeaksource.com/Pharo >>> and execute ScriptLoader class >> loadOB? >> >> It is not enough >> >> Starting from a fresh 10144 image, loading OB with scriptloader, >> then your latest commit for sunit, I do get the icons in the >> browser, but cant update their state: not lastRun method is >> generated and it fails when I try to save the results in the >> TestRunner. Best guess: the code generating lastRun on the class >> side is lost. >> >>> >>> >>>> Then after it would be great to have a button to run the tests and >>> >>> yes, we should have something like that in the browser. >>> >>> David >>> >>> >>>>> Hi Stef, >>>>> >>>>> as you as the SUnit extensions are integrated, you just need to >>>>> load OB. Any OB load method in ScriptLoader can be used for this >>>>> as this extension is also in the stable OB version now. >>>>> >>>>> David >>>>> >>>>>> can you tell me what I should do to get the icons showing the >>>>>> status of tests in your browser? >>>>>> I should harvest the Sunit extensions and then? >>>>>> Stef >>>>>> _______________________________________________ >>>>>> Pharo-project mailing list >>>>>> Pharo-project(a)lists.gforge.inria.fr >>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>>>> >>>>> >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> Pharo-project(a)lists.gforge.inria.fr >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo- >>>>> project >>>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> Pharo-project(a)lists.gforge.inria.fr >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> Pharo-project(a)lists.gforge.inria.fr >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> -- >> Simon >> >> >> >> >> _______________________________________________ >> Pharo-project mailing list >> Pharo-project(a)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(a)lists.gforge.inria.fr > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >

Nov. 10, 2008

[Pharo-project] New pharo-dev based on latest version

by Damien Cassou

http://gforge.inria.fr/frs/download.php/10931/pharo0.1-10142dev08.11.2.zip always available at: http://code.google.com/p/pharo/wiki/Downloads?tm=2 -- Damien Cassou http://damiencassou.seasidehosting.st

Nov. 10, 2008